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

Fix for buffer overflow #10

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@invisi-zz
Copy link

invisi-zz commented Aug 3, 2016

This is a somewhat naive fix for a possible buffer overflow when handling
packets that span more than one packet. The existing codebase doesn't
do any sanity checking that the packet is within the required bounds
of the buffer (see DEFBUF), so this really should be a part of a more
robust handling mechanism. However, this is sufficient for now.

Fixes #9.

Tim Nordell
Fix for buffer overflow
This is a somewhat naive fix for a possible buffer overflow when handling
packets that span more than one packet.  The existing codebase doesn't
do any sanity checking that the packet is within the required bounds
of the buffer (see DEFBUF), so this really should be a part of a more
robust handling mechanism.  However, this is sufficient for now.
@gartnera

This comment has been minimized.

Copy link
Owner

gartnera commented Sep 18, 2016

Merged

@gartnera gartnera closed this Sep 18, 2016

Trevelopment added a commit that referenced this pull request Jan 14, 2019

Wi fi (#10)
HUD
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment