lwIP - A Lightweight TCP/IP stack - Bugs: bug #36471, ppp_oe.c is using sizeof(struct...
You are not allowed to post comments on this tracker with your current authentication level.
bug #36471: ppp_oe.c is using sizeof(struct eth_hdr) instead of SIZEOF_ETH_HDR
Submitter: | Sylvain Rochet <gradator> | ||
Submitted: | Tue 15 May 2012 10:35:02 AM UTC | ||
Category: | None | Severity: | 3 - Normal |
Item Group: | Faulty Behaviour | Status: | Invalid |
Privacy: | Public | Assigned to: | None |
Open/Closed: | Closed | Planned Release: | None |
lwIP version: | CVS Head |
Tue 15 May 2012 08:49:18 PM UTC, comment #2: |
Sylvain Rochet <gradator> |
Tue 15 May 2012 07:55:42 PM UTC, comment #1: I don't think it makes a difference. If you look at the definition of struct eth_hdr in etharp.h, you see it already includes an u8_t array of ETH_PAD_SIZE (before the actual header). Being like that, sizeof(struct eth_hdr) and ETH_PAD_SIZE should yield the same values.
|
Simon Goldschmidt <goldsimon> |
Tue 15 May 2012 10:35:02 AM UTC, original submission:
Hi,
|
Sylvain Rochet <gradator> |
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 |
---|---|---|---|---|---|
2012-05-15 | gradator | Status | None | Invalid | |
Open/Closed | Open | Closed |
Powered by Savane 3.13-4b48.
Corresponding source code
Oh!, I did not see the padding at the -beginning- of the structure definition, of course it makes sense to be front-padded in this case :-)
Of course, lwIP won't work with compilers that do not support structure packing.
Sorry about this superflous bug report.