Skip to content
New issue

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

There is no way to set the maximum message size #62

Closed
GoogleCodeExporter opened this issue Mar 17, 2015 · 2 comments
Closed

There is no way to set the maximum message size #62

GoogleCodeExporter opened this issue Mar 17, 2015 · 2 comments

Comments

@GoogleCodeExporter
Copy link

In earlier versions of KBUS, there was a fixed maximum message size. When this 
was removed, a settable value should have replaced it, but did not. Without 
this, there is nothing to stop a user from creating an incredibly large message 
as a denial of service attack (or just carelessness).

Two values should be supplied: an absolute maximum message size, determined at 
compile time, above which the maximum message size cannot be set, and a default 
maximum message size.

An ioctl should also be introduced to change the current maximum, on a 
particular KBUS device.

A lower limit of greater than the size of a message header (80 bytes on a 
32-bit machine) should be enforced (e.g., 100).


Original issue reported on code.google.com by t...@tibsnjoan.co.uk on 14 Apr 2011 at 2:46

@GoogleCodeExporter
Copy link
Author

The kernel module has been updated to have the appropriate new ioctl.

The Python module has been updated to support it.

The Python tests now test it.

Outstanding is adding support to the C, C++ and Java libraries.

Original comment by t...@tibsnjoan.co.uk on 14 Apr 2011 at 4:54

  • Changed state: Started

@RMJ10
Copy link
Contributor

RMJ10 commented Jul 21, 2021

The C++ library was done a while ago. I've just done the C library. The Java library is... lacking in so many ways it needs its own issue.

@RMJ10 RMJ10 closed this as completed Jul 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants