[U-Boot] [PATCH]Fix checksum to handle odd-length packet
Jerry Van Baren
gerald.vanbaren at ge.com
Thu Dec 10 19:21:54 CET 2009
Greg Ren wrote:
> Jerry Van Baren wrote:
>
>> When you respin your patch, please put the "ping with an odd length
>> causes an incorrect checksum bug" in the commit message for future
>> reference.
>
>> Thanks,
>> gvb
>
> The change was to the general checksum calculation. So there is enough
> reason to believe that any odd-length packet may suffer the same fate.
> Ping is just an easier way to test and verify the fix.
>
> regards
> Greg Ren
Understood. My point was your original commit message did not identify
*how* to exercise the bug (i.e. ping with an odd length packet). That
is very valuable information because it helps us remember what was
broken and how to test both the brokenness and the fix.
Thanks,
gvb
More information about the U-Boot
mailing list