autofs4: catatonic_mode vs. notify_daemon race
authorAl Viro <viro@zeniv.linux.org.uk>
Wed, 11 Jan 2012 03:24:48 +0000 (22:24 -0500)
committerGrazvydas Ignotas <notasas@gmail.com>
Sat, 16 Sep 2017 20:20:39 +0000 (23:20 +0300)
commit0d94423bff1c02e4207397884a01ec550f9f1955
treecbe7179682eecbedf6171afe53ea4df7abc1db20
parenta15f7d59e122ac9bf84a72315a44e18ce35143a9
autofs4: catatonic_mode vs. notify_daemon race

we need to hold ->wq_mutex while we are forming the packet to send,
lest we have autofs4_catatonic_mode() setting wq->name.name to NULL
just as autofs4_notify_daemon() decides to memcpy() from it...

We do have check for catatonic mode immediately after that (under
->wq_mutex, as it ought to be) and packet won't be actually sent,
but it'll be too late for us if we oops on that memcpy() from NULL...

Fix is obvious - just extend the area covered by ->wq_mutex over
that switch and check whether it's catatonic *before* doing anything
else.

Acked-by: Ian Kent <raven@themaw.net>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
fs/autofs4/waitq.c