[lkml]   [2003]   [Oct]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: possible bug in tcp_input.c
    On Sun, 26 Oct 2003 07:55:19 +0100
    Tomas Szepe <> wrote:

    > Dave, we've been thinking about this some more and have concluded
    > that since the problem is a relatively non-fatal one, the kernel
    > should just print out an "assertion failed" error similar to the
    > one in tcp_input.c, line 1323 [BUG_TRAP(cnt <= tp->packets_out);]
    > and maybe fix things up a little rather than oops on a NULL pointer
    > dereference; The state in question, although invalid, is possible
    > and should IMHO be checked for as in all the other "if (skb != NULL)
    > ..." places).

    If this condition triggers, the lists are corrupt and the kernel
    should not try to access this socket in any way whatsoever
    past this point.

    Therefore it should OOPS, which is exactly what it does now.
    A BUG_ON() is an equivalent response as far as I am concerned,
    it has the same exact result, and the backtrace shows where the
    problem is occuring.
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2005-03-22 13:58    [W:0.018 / U:400.580 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site