[SLOF] [PATCH] Fix remaining compiler warnings in sloffs.c
Segher Boessenkool
segher at kernel.crashing.org
Wed Aug 10 05:41:12 AEST 2016
On Tue, Aug 09, 2016 at 12:15:42PM +0530, Nikunj A Dadhania wrote:
> > Don't you need to rather check for " i < udp_hdr->uh_ulen / 2" now?
>
> Right, missed that. And I think we need to take care for odd length as
> well.
Yes, both those. The original code doesn't handle odd length correctly
either it seems.
The checksum magically works out with either BE or LE 16-bit accesses,
but you might want to make that explicit, too.
Segher
More information about the SLOF
mailing list