race of lockd inetaddr notifiers vs nlmsvc_rqst change
authorVasily Averin <[email protected]>
Fri, 10 Nov 2017 07:19:26 +0000 (10:19 +0300)
committerJ. Bruce Fields <[email protected]>
Mon, 27 Nov 2017 21:45:11 +0000 (16:45 -0500)
commit6b18dd1c03e07262ea0866084856b2a3c5ba8d09
tree0e9780bbbd7ae2aa0913a5ebbff8822d5fa7edd4
parentee24eac3ebb781c12a654985e33ecaa07f4d0f95
race of lockd inetaddr notifiers vs nlmsvc_rqst change

lockd_inet[6]addr_event use nlmsvc_rqst without taken nlmsvc_mutex,
nlmsvc_rqst can be changed during execution of notifiers and crash the host.

Patch enables access to nlmsvc_rqst only when it was correctly initialized
and delays its cleanup until notifiers are no longer in use.

Note that nlmsvc_rqst can be temporally set to ERR_PTR, so the "if
(nlmsvc_rqst)" check in notifiers is insufficient on its own.

Signed-off-by: Vasily Averin <[email protected]>
Tested-by: Scott Mayhew <[email protected]>
Signed-off-by: J. Bruce Fields <[email protected]>
fs/lockd/svc.c