On Wed, May 30, 2012 at 02:24:12PM +0400, Dmitry Monakhov wrote:
> 
> If remount failed due to some reason we have to fallback quota to it's 
> original state
> otherwise it will stay in suspended state which is incorrect.
> 
> #TEST_CASE
> dd if=/dev/zero of=/tmp/imgfile bs=1M count=100
> # any fs with quota are affected
> mkfs.ext4 -F /tmp/imgfile
> mount /tmp/imgfile  /mnt -oloop,quota
> quotacheck -cug /mnt
> quotaon /mnt
> # remount to RO state , and explicitly provide bad option
> mount /mnt -oremount,ro,some-bad-option
> # At this point fs is in broken state because it is RW, but quota is in
> # suspended state. Later activity will trigger NULL pointer dereference
> mount /mnt -oremount,ro,some-bad-option
> 
> Bug was silently fixed in: e0ccfd959cd890 v2.6.34-7101-ge0ccfd9, All previous
> kernel versions are affected.
> 
> 
> 
> Signed-off-by: Dmitry Monakhov <[email protected]>
> ---
>  fs/super.c |    7 ++++++-
>  1 files changed, 6 insertions(+), 1 deletions(-)
> 

<formletter>

This is not the correct way to submit patches for inclusion in the
stable kernel tree.  Please read Documentation/stable_kernel_rules.txt
for how to do this properly.

</formletter>
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to