Messages in this thread Patch in this message | | | Date | Mon, 19 Nov 2007 12:57:33 +0800 | From | Dave Young <> | Subject | Re: [PATCH][another try]bluetooth rfcomm tty_close before destruct |
| |
Hi, This is the right patch which will not touch original tty logic. Please ignore last one submited by me.
Regards dave
Subject: rfcomm_dev_del could be called twice time in release function. one by rfcomm_tty_hangup, another by rfcomm_release_dev, this will cause the device being destructed before rfcomm_tty_close.
Signed-off-by: Dave Young <hidave.darkstar@gmail.com>
--- tty.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
diff -upr linux/net/bluetooth/rfcomm/tty.c linux.new/net/bluetooth/rfcomm/tty.c --- linux/net/bluetooth/rfcomm/tty.c 2007-11-19 10:48:32.000000000 +0800 +++ linux.new/net/bluetooth/rfcomm/tty.c 2007-11-19 12:48:10.000000000 +0800 @@ -424,8 +424,8 @@ static int rfcomm_release_dev(void __use /* Shut down TTY synchronously before freeing rfcomm_dev */ if (dev->tty) tty_vhangup(dev->tty); - - rfcomm_dev_del(dev); + else if (!test_bit(RFCOMM_RELEASE_ONHUP, &dev->flags)) + rfcomm_dev_del(dev); rfcomm_dev_put(dev); return 0; } - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |