From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.linuxfoundation.org ([140.211.169.12]:34748 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1162345AbdAIP1h (ORCPT ); Mon, 9 Jan 2017 10:27:37 -0500 Subject: Patch "md: fix refcount problem on mddev when stopping array." has been added to the 4.9-stable tree To: neilb@suse.com, gqjiang@suse.com, gregkh@linuxfoundation.org, marc.smith@mcc.edu, shli@fb.com Cc: , From: Date: Mon, 09 Jan 2017 16:27:03 +0100 Message-ID: <148397562311765@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=ANSI_X3.4-1968 Content-Transfer-Encoding: 8bit Sender: stable-owner@vger.kernel.org List-ID: This is a note to let you know that I've just added the patch titled md: fix refcount problem on mddev when stopping array. to the 4.9-stable tree which can be found at: http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary The filename of the patch is: md-fix-refcount-problem-on-mddev-when-stopping-array.patch and it can be found in the queue-4.9 subdirectory. If you, or anyone else, feels it should not be added to the stable tree, please let know about it. >>From e2342ca832726a840ca6bd196dd2cc073815b08a Mon Sep 17 00:00:00 2001 From: NeilBrown Date: Mon, 5 Dec 2016 16:40:50 +1100 Subject: md: fix refcount problem on mddev when stopping array. From: NeilBrown commit e2342ca832726a840ca6bd196dd2cc073815b08a upstream. md_open() gets a counted reference on an mddev using mddev_find(). If it ends up returning an error, it must drop this reference. There are two error paths where the reference is not dropped. One only happens if the process is signalled and an awkward time, which is quite unlikely. The other was introduced recently in commit af8d8e6f0. Change the code to ensure the drop the reference when returning an error, and make it harded to re-introduce this sort of bug in the future. Reported-by: Marc Smith Fixes: af8d8e6f0315 ("md: changes for MD_STILL_CLOSED flag") Signed-off-by: NeilBrown Acked-by: Guoqing Jiang Signed-off-by: Shaohua Li Signed-off-by: Greg Kroah-Hartman --- drivers/md/md.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) --- a/drivers/md/md.c +++ b/drivers/md/md.c @@ -7092,7 +7092,8 @@ static int md_open(struct block_device * if (test_bit(MD_CLOSING, &mddev->flags)) { mutex_unlock(&mddev->open_mutex); - return -ENODEV; + err = -ENODEV; + goto out; } err = 0; @@ -7101,6 +7102,8 @@ static int md_open(struct block_device * check_disk_change(bdev); out: + if (err) + mddev_put(mddev); return err; } Patches currently in stable-queue which might be from neilb@suse.com are queue-4.9/md-fix-refcount-problem-on-mddev-when-stopping-array.patch queue-4.9/md-md_recovery_needed-is-set-for-mddev-recovery.patch