dm cache: Fix UAF in destroy()
authorLuo Meng <luomeng12@huawei.com>
Tue, 29 Nov 2022 02:48:49 +0000 (10:48 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 7 Jan 2023 11:07:36 +0000 (12:07 +0100)
commit 6a459d8edbdbe7b24db42a5a9f21e6aa9e00c2aa upstream.

Dm_cache also has the same UAF problem when dm_resume()
and dm_destroy() are concurrent.

Therefore, cancelling timer again in destroy().

Cc: stable@vger.kernel.org
Fixes: c6b4fcbad044e ("dm: add cache target")
Signed-off-by: Luo Meng <luomeng12@huawei.com>
Signed-off-by: Mike Snitzer <snitzer@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/md/dm-cache-target.c

index 1b7d77080d6b920894c748cfafeea32bfb19d86f..03dcb4ec422522e19a7d1a84d371d043b1244027 100644 (file)
@@ -2321,6 +2321,7 @@ static void destroy(struct cache *cache)
        if (cache->prison)
                dm_bio_prison_destroy(cache->prison);
 
+       cancel_delayed_work_sync(&cache->waker);
        if (cache->wq)
                destroy_workqueue(cache->wq);