Re: [PATCH 002 of 6] md: Fix use-after-free bug when dropping an rdev from an md array.

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Al Viro
Date: Monday, January 14, 2008 - 6:56 am

On Mon, Jan 14, 2008 at 12:59:39PM +0000, Al Viro wrote:


More fun questions: what are the locking requirements for ->resize()?
You are calling it with no exclusion whatsoever...  What about
bind_rdev_to_array()?  At the very least, you want to protect
mddev->disks, and AFAICS new_dev_store() has no exclusion at all.
And I suspect that you have other things in there in need of protection
(finding free desc_nr, for one); can all of those be handled by simple
spinlocks?
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Re: [PATCH 002 of 6] md: Fix use-after-free bug when dropp ..., Al Viro, (Mon Jan 14, 6:56 am)