lwIP - A Lightweight TCP/IP stack - Bugs: bug #50829, [CPPCHECK] Array index used before...
You are not allowed to post comments on this tracker with your current authentication level.
bug #50829: [CPPCHECK] Array index used before limits check.
Submitter: | user name <username> | ||
Submitted: | Wed 19 Apr 2017 01:58:16 AM UTC | ||
Category: | IPv4 | Severity: | 3 - Normal |
Item Group: | None | Status: | Fixed |
Privacy: | Public | Assigned to: | None |
Open/Closed: | Closed | Planned Release: | None |
lwIP version: | git head |
Thu 20 Apr 2017 07:47:18 PM UTC, comment #2: |
Simon Goldschmidt <goldsimon> |
Wed 19 Apr 2017 04:36:15 AM UTC, comment #1: Yes, thanks for the patch! |
Dirk Ziegelmeier <dziegel> |
Wed 19 Apr 2017 01:58:16 AM UTC, original submission:
http://git.savannah.nongnu.org/cgit/lwip.git/tree/src/core/ipv4/dhcp.c#n1502
|
user name <username> |
No files currently attached
Depends on the following items: None found
Items that depend on this one: None found
There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.
Follow 2 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2017-04-19 | dziegel | Status | None | Fixed | |
Open/Closed | Open | Closed |
Powered by Savane 3.13-d3ae.
Corresponding source code
I'll have to add that this is a false positive.
'offset_max' is not a limit of the 'options' "array". Instead, 'options' is a pointer into a pbuf, the length is checked at the end of the array.
This change doesn't hurt, but it does not fix anything!