lwIP - A Lightweight TCP/IP stack - Bugs: bug #29970, DHCP endian issues parsing option...
You are not allowed to post comments on this tracker with your current authentication level.
bug #29970: DHCP endian issues parsing option responses (w/ patch)
Submitter: | Matt Darland <mdarland> | ||
Submitted: | Wed 26 May 2010 02:34:52 PM UTC | ||
Category: | IPv4 | Severity: | 3 - Normal |
Item Group: | Faulty Behaviour | Status: | Fixed |
Privacy: | Public | Assigned to: | goldsimon |
Open/Closed: | Closed | Planned Release: | None |
lwIP version: | CVS Head |
Tue 15 Jun 2010 08:20:24 PM UTC, comment #2: |
Simon Goldschmidt <goldsimon> |
Sat 12 Jun 2010 06:57:01 PM UTC, comment #1: Sorry for needing so long to reply (I've been on holiday for some time..).
|
Simon Goldschmidt <goldsimon> |
Wed 26 May 2010 02:34:52 PM UTC, original submission:
DHCP options parsing makes assumptions about endianess that are incorrect and leads to faulty options parsing. In particular, reading a raw byte sequence directly into a u32_t* and then reading the integer as a 1-byte value does not work for all platforms. Attached is the patch.
|
Matt Darland <mdarland> |
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 7 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2010-06-15 | goldsimon | Open/Closed | Open | Closed | |
2010-06-15 | goldsimon | Status | In Progress | Fixed | |
2010-06-12 | goldsimon | Status | None | In Progress | |
Assigned to | None | goldsimon | |||
Planned Release | 1.4.0 | ||||
2010-05-26 | mdarland | Attached File | - | Added dhcp.c.diff, #20627 | |
Carbon-Copy | - | Added -email is unavailable- |
Powered by Savane 3.13-02a9.
Corresponding source code
Fixed like suggested in comment #1, please reopen if it didn't work like I expected.
Thanks for reporting, Matt!