Skip to content

Commit

Permalink
net: hso: fix NULL-deref on disconnect regression
Browse files Browse the repository at this point in the history
Commit 8a12f88 ("net: hso: fix null-ptr-deref during tty device
unregistration") fixed the racy minor allocation reported by syzbot, but
introduced an unconditional NULL-pointer dereference on every disconnect
instead.

Specifically, the serial device table must no longer be accessed after
the minor has been released by hso_serial_tty_unregister().

Fixes: 8a12f88 ("net: hso: fix null-ptr-deref during tty device unregistration")
Cc: [email protected]
Cc: Anirudh Rayabharam <[email protected]>
Reported-by: Leonardo Antoniazzi <[email protected]>
Signed-off-by: Johan Hovold <[email protected]>
Reviewed-by: Anirudh Rayabharam <[email protected]>
Signed-off-by: David S. Miller <[email protected]>
  • Loading branch information
jhovold authored and davem330 committed Apr 26, 2021
1 parent d0c5d18 commit 2ad5692
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion drivers/net/usb/hso.c
Original file line number Diff line number Diff line change
Expand Up @@ -3104,7 +3104,7 @@ static void hso_free_interface(struct usb_interface *interface)
cancel_work_sync(&serial_table[i]->async_put_intf);
cancel_work_sync(&serial_table[i]->async_get_intf);
hso_serial_tty_unregister(serial);
kref_put(&serial_table[i]->ref, hso_serial_ref_free);
kref_put(&serial->parent->ref, hso_serial_ref_free);
}
}

Expand Down

0 comments on commit 2ad5692

Please sign in to comment.