Skip to content

Commit d7fad2c

Browse files
jhovoldgregkh
authored andcommitted
net: hso: fix NULL-deref on disconnect regression
commit 2ad5692 upstream. 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: stable@vger.kernel.org Cc: Anirudh Rayabharam <mail@anirudhrb.com> Reported-by: Leonardo Antoniazzi <leoanto@aruba.it> Signed-off-by: Johan Hovold <johan@kernel.org> Reviewed-by: Anirudh Rayabharam <mail@anirudhrb.com> Signed-off-by: David S. Miller <davem@davemloft.net> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
1 parent 699017f commit d7fad2c

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

drivers/net/usb/hso.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3105,7 +3105,7 @@ static void hso_free_interface(struct usb_interface *interface)
31053105
cancel_work_sync(&serial_table[i]->async_put_intf);
31063106
cancel_work_sync(&serial_table[i]->async_get_intf);
31073107
hso_serial_tty_unregister(serial);
3108-
kref_put(&serial_table[i]->ref, hso_serial_ref_free);
3108+
kref_put(&serial->parent->ref, hso_serial_ref_free);
31093109
}
31103110
}
31113111

0 commit comments

Comments
 (0)