lwIP - A Lightweight TCP/IP stack - Bugs: bug #54506, LWIP_CHECKSUM_ON_COPY causes wrong...
You are not allowed to post comments on this tracker with your current authentication level.
bug #54506: LWIP_CHECKSUM_ON_COPY causes wrong checksum (0xFFFF)
Submitter: | Sebastian Maier <basti> | ||
Submitted: | Mon 13 Aug 2018 10:55:16 AM UTC | ||
Votes: | 100 | ||
Category: | TCP | Severity: | 3 - Normal |
Item Group: | Faulty Behaviour | Status: | Fixed |
Privacy: | Public | Assigned to: | goldsimon |
Open/Closed: | Closed | Planned Release: | 2.1.0 |
lwIP version: | git head |
Wed 12 Sep 2018 08:33:27 PM UTC, comment #2: |
Simon Goldschmidt <goldsimon> |
Sat 08 Sep 2018 08:37:12 PM UTC, comment #1: I think the correct version of the fix is to invert back 'acc' (as it is returned inverted by ip_chksum_pseudo_partial()), then do the calculation and fold and then inverting it again. As that is the same what's already done, it should be portable.
|
Simon Goldschmidt <goldsimon> |
Mon 13 Aug 2018 10:55:16 AM UTC, original submission:
I am currently using lwIP (HEAD) on x86-64 with a custom OS and an Intel e1000(e) ethernet adapter to transfer data via TCP.
|
Sebastian Maier <basti> |
Depends on the following items: None found
Items that depend on this one: None found
There are 100 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 9 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2018-09-12 | goldsimon | Status | None | Fixed | |
Assigned to | None | goldsimon | |||
Open/Closed | Open | Closed | |||
Planned Release | None | 2.1.0 | |||
2018-09-08 | goldsimon | Attached File | - | Added patch3.diff, #44958 | |
2018-08-13 | flow | Carbon-Copy | - | Added flow | |
2018-08-13 | basti | Attached File | - | Added patch1.diff, #44770 | |
Attached File | - | Added patch2.diff, #44771 | |||
Attached File | - | Added patch3.diff, #44772 |
Powered by Savane 3.13-d3ae.
Corresponding source code
Fixed as suggested in comment #1.
Thanks for reporting!