[lkml]   [2006]   [Mar]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: Fw: Re: oops in choose_configuration()
    On Tuesday 07 March 2006 19:57, Linus Torvalds wrote:
    > On Tue, 7 Mar 2006, Chuck Ebbert wrote:
    > >
    > > At least one susbsystem rolls its own method of adding env vars to the
    > > uevent buffer, and it's so broken it triggers the WARN_ON() in
    > > lib/vsprintf.c::vsnprintf() by passing a negative length to that function.
    > Well, snprintf() should be safe, though. It will warn if the caller is
    > lazy, but these days, the thing does
    > max(buf_size - len, 0)
    > which should mean that the input layer passes in 0 instead of a negative
    > number. And snprintf() will then _not_ print anything.
    > So I think input_add_uevent_bm_var() is safe, even if it's not pretty.
    > However, input_devices_read() doesn't do any sanity checking at all, and
    > if that ever ends up printing more than a page, that would be bad. I
    > didn't look very closely, but it looks worrisome.
    > Dmitry?

    I had all this code converted to seq_file, but it depends on converting
    input handlers to class interfaces and it is not possible nowadays
    because with latest Greg's changes to class code we would try to
    register class devices while registering class devices/interfaces
    (psmouse creates input_dev which binds to mousedev interface which in
    turn tries to create mouseX which is also belongs to input class) and
    deadlocking. Greg promised current implementation is only a temporary

    I suppose I could separate those changes...

    BTW, what is ETA for 2.6.16 - I could not affort enought time getting
    psmouse resync logic working in all cases so I'd like to have it disabled
    for 2.6.16 final.

    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: 2006-03-08 02:15    [W:0.026 / U:50.100 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site