lwIP - A Lightweight TCP/IP stack - Bugs: bug #50914, TCP_CHECKSUM_ON_COPY when adding...
You are not allowed to post comments on this tracker with your current authentication level.
bug #50914: TCP_CHECKSUM_ON_COPY when adding data to retransmission
Submitter: | shilei yu <kyliuxing> | ||
Submitted: | Tue 02 May 2017 11:58:33 AM UTC | ||
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: | 2.0.0 |
Jump to the original submission
Mon 12 Feb 2018 11:39:01 AM UTC, comment #17: |
Simon Goldschmidt <goldsimon> |
Thu 11 May 2017 07:31:11 PM UTC, comment #16: In any case, this seems like yet another bug in calculating the a header checksum (at least #43840 fised in db4844 on 2014-12-17 was also in this region somewhere)
|
Simon Goldschmidt <goldsimon> |
Thu 11 May 2017 07:03:56 PM UTC, comment #15:
|
Simon Goldschmidt <goldsimon> |
Thu 11 May 2017 01:45:47 PM UTC, comment #14:
|
Joel Cunningham <jcunningham> |
Thu 11 May 2017 09:06:10 AM UTC, comment #13: I added the log as attachment, it happened twice but i didn't get wireshark capture.i feel if the PC's ack between the same segment's seq and seq+len, problem will be reproduced.
|
shilei yu <kyliuxing> |
Thu 11 May 2017 07:25:55 AM UTC, comment #12: LWIP_CHECKSUM_ON_COPY
|
Simon Goldschmidt <goldsimon> |
Thu 11 May 2017 06:10:58 AM UTC, comment #11: @goldsimon
|
shilei yu <kyliuxing> |
Mon 08 May 2017 07:15:44 PM UTC, comment #10: Right. (Were you waiting for that?)
|
Simon Goldschmidt <goldsimon> |
Fri 05 May 2017 09:49:50 AM UTC, comment #9: TCP_CHECKSUM_ON_COPY_SANITY_CHECK seems not work if i set LWIP_CHECKSUM_ON_COPY==0(TCP_CHECKSUM_ON_COPY will be 0 too),
|
shilei yu <kyliuxing> |
Fri 05 May 2017 07:37:17 AM UTC, comment #8: Oh, and you might also want to try:
|
Simon Goldschmidt <goldsimon> |
Fri 05 May 2017 07:33:19 AM UTC, comment #7:
|
Simon Goldschmidt <goldsimon> |
Fri 05 May 2017 07:20:05 AM UTC, comment #6: @goldsimon
|
shilei yu <kyliuxing> |
Thu 04 May 2017 11:02:50 AM UTC, comment #5: Yes. There are many "[TCP CHECKSUM INCORRECT]" packets from the PC as checksum offloading is done after wireshark captures the local packets.
|
Simon Goldschmidt <goldsimon> |
Wed 03 May 2017 02:09:59 AM UTC, comment #4: @goldsimon
|
shilei yu <kyliuxing> |
Tue 02 May 2017 07:23:43 PM UTC, comment #3: And after another look, did you notice both sides produce invalid checksums? Normally, only the PC running wireshark should have invalid checksums. |
Simon Goldschmidt <goldsimon> |
Tue 02 May 2017 07:11:46 PM UTC, comment #2: Reading the pcap, I can only assume you have a lot of lost packets, possibly due to recvmbox limitations (or other limitations, as you don't say what lwIP API you are using).
|
Simon Goldschmidt <goldsimon> |
Tue 02 May 2017 07:02:52 PM UTC, comment #1: I assume the PC is running lwIP? ;-)
|
Simon Goldschmidt <goldsimon> |
Tue 02 May 2017 11:58:33 AM UTC, original submission:
I met a issue when made TCP test. I built a TCP server on PC and make my device to connect it, they sent data to each other.
|
shilei yu <kyliuxing> |
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 8 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2018-02-12 | goldsimon | Status | None | Fixed | |
Assigned to | None | goldsimon | |||
Open/Closed | Open | Closed | |||
2018-01-02 | goldsimon | Planned Release | None | 2.1.0 | |
2017-05-11 | goldsimon | Summary | TCP server don't update the ack num | TCP_CHECKSUM_ON_COPY when adding data to retransmission | |
2017-05-11 | kyliuxing | Attached File | - | Added SANITY_CHECK_log.7z, #40663 | |
2017-05-02 | kyliuxing | Attached File | - | Added UART_client.log, #40555 | |
Attached File | - | Added Wireshark_server.pcap, #40556 |
Powered by Savane 3.13-f8d8.
Corresponding source code
To fix this, tcp_output_segment has to restore seg->chksum_swapped
before returning.
Fixed, thanks for reporting.