]> nv-tegra.nvidia Code Review - linux-2.6.git/commit
[NET]: Make sure l_linger is unsigned to avoid negative timeouts
authorEric Dumazet <dada1@cosmosbay.com>
Tue, 6 Sep 2005 21:51:39 +0000 (14:51 -0700)
committerDavid S. Miller <davem@davemloft.net>
Tue, 6 Sep 2005 21:51:39 +0000 (14:51 -0700)
commit9261c9b042547d01eeb206cf0e21ce72832245ec
treee6dbce47404e1d40dff0fb5814463d89370a6642
parentfe2d5295a11e2ab2d6f4e7ea074816000b32eba3
[NET]: Make sure l_linger is unsigned to avoid negative timeouts

One of my x86_64 (linux 2.6.13) server log is filled with :

schedule_timeout: wrong timeout value ffffffffffffff06 from ffffffff802e63ca
schedule_timeout: wrong timeout value ffffffffffffff06 from ffffffff802e63ca
schedule_timeout: wrong timeout value ffffffffffffff06 from ffffffff802e63ca
schedule_timeout: wrong timeout value ffffffffffffff06 from ffffffff802e63ca
schedule_timeout: wrong timeout value ffffffffffffff06 from ffffffff802e63ca

This is because some application does a

struct linger li;
li.l_onoff = 1;
li.l_linger = -1;
setsockopt(sock, SOL_SOCKET, SO_LINGER, &li, sizeof(li));

And unfortunatly l_linger is defined as a 'signed int' in
include/linux/socket.h:

struct linger {
         int             l_onoff;        /* Linger active                */
         int             l_linger;       /* How long to linger for       */
};

I dont know if it's safe to change l_linger to 'unsigned int' in the
include file (It might be defined as int in ABI specs)

Signed-off-by: Eric Dumazet <dada1@cosmosbay.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/core/sock.c