From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: From: Michal Hocko To: Andrew Morton Cc: Al Viro , Tetsuo Handa , Aliaksei Karaliou , , , LKML , Glauber Costa , Michal Hocko Subject: [PATCH 1/2] mm,vmscan: Make unregister_shrinker() no-op if register_shrinker() failed. Date: Tue, 19 Dec 2017 14:28:43 +0100 Message-Id: <20171219132844.28354-2-mhocko@kernel.org> In-Reply-To: <20171219132844.28354-1-mhocko@kernel.org> References: <20171219132844.28354-1-mhocko@kernel.org> Sender: owner-linux-mm@kvack.org List-ID: From: Tetsuo Handa Since allowing register_shrinker() callers to call unregister_shrinker() when register_shrinker() failed can simplify error recovery path, this patch makes unregister_shrinker() no-op when register_shrinker() failed. Let's also make sure that double unregister_shrinker doesn't blow up as well and NULL nr_deferred on successful de-registration to make the clean up even simpler and prevent from potential memory corruptions. [akaraliou.dev@gmail.com: set nr_deferred = NULL to handle double unregister] Signed-off-by: Tetsuo Handa Reported-by: syzbot Cc: Glauber Costa Cc: Al Viro Signed-off-by: Michal Hocko --- mm/vmscan.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/mm/vmscan.c b/mm/vmscan.c index 80dea50f421b..7a5801040fd4 100644 --- a/mm/vmscan.c +++ b/mm/vmscan.c @@ -281,10 +281,13 @@ EXPORT_SYMBOL(register_shrinker); */ void unregister_shrinker(struct shrinker *shrinker) { + if (!shrinker->nr_deferred) + return; down_write(&shrinker_rwsem); list_del(&shrinker->list); up_write(&shrinker_rwsem); kfree(shrinker->nr_deferred); + shrinker->nr_deferred = NULL; } EXPORT_SYMBOL(unregister_shrinker); -- 2.15.1 -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org