espressif / esp-lwip

Fork of lwIP (https://savannah.nongnu.org/projects/lwip/) with ESP-IDF specific patches
Other
79 stars 126 forks source link

napt: fix checksum of UDP (IDFGH-5425) #31

Closed bl4kraven closed 3 years ago

bl4kraven commented 3 years ago

According to the UDP protocol, the checksum is optional. so if the checksum is 0, this meaning the checksum is turn off, and no need to correct checksum.

Signed-off-by: leo chung gewalalb@gmail.com

david-cermak commented 3 years ago

@bl4kraven Thanks for this fix! LGTM.

Could you please rebase your branch onto the head of espressif:2.1.2-esp and correct the formatting, so we could merge this PR directly? Otherwise we would apply the changes internally, and cherry-pick your commit to close the PR.

bl4kraven commented 3 years ago

@david-cermak OK, i just rebase to head of espressif:2.1.2-esp and fix the code formatting. Thanks!