We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
in: https://github.com/davidmoreno/onion/blob/master/src/onion/websocket.c#L207 header[3 + 8 - i] = tlen & 0x0FF;
header[3 + 8 - i] = tlen & 0x0FF;
I am pretty sure the 3 should be a 1:
header[1 + 8 - i] = tlen & 0x0FF;
There may also be a 32 vs 64 bit problem on systems where int isn't 64 bits.
The text was updated successfully, but these errors were encountered:
I think this old issue can be closed.
The assumption that header[3+X] needs to be changed to header[1+X] is wrong: The size will be stored two bytes after the opcode (reference https://developer.mozilla.org/en-US/docs/Web/API/WebSockets_API/Writing_WebSocket_servers )
The 64bit width of size_t is still a problem, so for unrealistic big packages the type of len should be changed to ssize_t, too.
onion/src/onion/websocket.c
Line 205 in 2b3b230
Sorry, something went wrong.
No branches or pull requests
in: https://github.com/davidmoreno/onion/blob/master/src/onion/websocket.c#L207
header[3 + 8 - i] = tlen & 0x0FF;
I am pretty sure the 3 should be a 1:
header[1 + 8 - i] = tlen & 0x0FF;
There may also be a 32 vs 64 bit problem on systems where int isn't 64 bits.
The text was updated successfully, but these errors were encountered: