> Le jeudi 06 mai 2010 à 15:44 +0200, Bjørn Mork a écrit :
>> commit 2783ef23 moved the initialisation of saddr and daddr after
>> pskb_may_pull() to avoid a potential data corruption. Unfortunately
>> also placing it after the short packet and bad checksum error paths,
>> where these variables are used for logging. The result is bogus
>> output like
>>
>> [92238.389505] UDP: short packet: From 2.0.0.0:65535 23715/178 to 0.0.0.0:65535
>>
>> Moving the saddr and daddr initialisation above the error paths, while still
>> keeping it after the pskb_may_pull() to keep the fix from commit 2783ef23.
>>
>> Signed-off-by: Bjørn Mork <bjorn@mork.no>
>> Cc:
stable@kernel.org
>> ---
>> net/ipv4/udp.c | 6 +++---
>> 1 files changed, 3 insertions(+), 3 deletions(-)
>
> Well done :)
>
> Acked-by: Eric Dumazet <eric.dumazet@gmail.com>
>
> To be backported to 2.6.29 and up kernels ;)
