Bluetooth: serialize calls to sco_sock_{set,clear}_timer
authorDesmond Cheong Zhi Xi <desmondcheongzx@gmail.com>
Tue, 10 Aug 2021 04:14:08 +0000 (12:14 +0800)
committerLuiz Augusto von Dentz <luiz.von.dentz@intel.com>
Tue, 10 Aug 2021 17:40:48 +0000 (10:40 -0700)
commit3f2c89fb465f63fe14da804a8afcd7ac6776508e
tree77f90cabe318488ff0270b3ab87d7961d19de7a1
parent27c24fda62b601d6f9ca5e992502578c4310876f
Bluetooth: serialize calls to sco_sock_{set,clear}_timer

Currently, calls to sco_sock_set_timer are made under the locked
socket, but this does not apply to all calls to sco_sock_clear_timer.

Both sco_sock_{set,clear}_timer should be serialized by lock_sock to
prevent unexpected concurrent clearing/setting of timers.

Additionally, since sco_pi(sk)->conn is only cleared under the locked
socket, this change allows us to avoid races between
sco_sock_clear_timer and the call to kfree(conn) in sco_conn_del.

Signed-off-by: Desmond Cheong Zhi Xi <desmondcheongzx@gmail.com>
Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>
net/bluetooth/sco.c