[lkml]   [2005]   [Oct]   [31]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    Patch in this message
    SubjectFix drivers/macintosh/adbhid.c stupid breakage
    Commit c7f7a569d9b4ea7c53ab6fcd1377895312d8372b ("[PATCH] Input:
    convert drivers/macintosh to dynamic input_dev allocation") breaks any
    machine with an ADB keyboard or mouse, which includes my G4
    powerbook. Was it given any testing at all?

    The problem is that adbhid[]->input is NULL, so the kernel oopses with
    a null pointer dereference as soon as I press a key. The following
    patch fixes it.

    Signed-off-by: Paul Mackerras <>
    diff -urN powerpc-merge/drivers/macintosh/adbhid.c merge-hack/drivers/macintosh/adbhid.c
    --- powerpc-merge/drivers/macintosh/adbhid.c 2005-10-31 13:15:26.000000000 +1100
    +++ merge-hack/drivers/macintosh/adbhid.c 2005-10-31 16:30:31.000000000 +1100
    @@ -723,6 +723,7 @@

    sprintf(hid->phys, "adb%d:%d.%02x/input", id, default_id, original_handler_id);

    + hid->input = input_dev;
    hid->id = default_id;
    hid->original_handler_id = original_handler_id;
    hid->current_handler_id = current_handler_id;
    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-10-31 06:41    [W:0.023 / U:0.228 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site