lwIP - A Lightweight TCP/IP stack - Bugs: bug #37959, ip6_debug_print prints plen,...
You are not allowed to post comments on this tracker with your current authentication level.
bug #37959: ip6_debug_print prints plen, nexth, hoplim in wrong byte order
Submitter: | Valery Ushakov <uwe> | ||
Submitted: | Wed 19 Dec 2012 03:36:25 PM UTC | ||
Category: | IPv6 | Severity: | 3 - Normal |
Item Group: | Faulty Behaviour | Status: | Fixed |
Privacy: | Public | Assigned to: | idelamer |
Open/Closed: | Closed | Planned Release: | None |
lwIP version: | CVS Head |
Wed 19 Dec 2012 11:39:05 PM UTC, comment #1: |
Ivan Delamer <idelamer> |
Wed 19 Dec 2012 03:36:25 PM UTC, original submission:
ip6_debug_print prints plen, netxh, hoplim with an extra ntohs, e.g. ntohs(IP6H_PLEN(ip6hdr)), etc - but IP6H_* macros already do ntohs() internally (as hinted by 'H'ost in their names).
|
Valery Ushakov <uwe> |
Attached Files
Depends on the following items: None found
Items that depend on this one: None found
Carbon-Copy List
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 4 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2012-12-19 | idelamer | Status | None | Fixed | |
2012-12-19 | idelamer | Assigned to | None | idelamer | |
Open/Closed | Open | Closed | |||
2012-12-19 | uwe | Attached File | - | Added ip6_debug_print.diff, #27125 |
Powered by Savane 3.13-02a9.
Corresponding source code
Good catch, ntohs() is not necessary, and seems to be there due to an ugly copy-paste error.
Will push a fix for this.
Thanks for reporting
PS: 'H' is for 'H'eader ;-)