ubifs: drop false positive assertion
authorSascha Hauer <s.hauer@pengutronix.de>
Wed, 12 Sep 2018 12:51:38 +0000 (14:51 +0200)
committerRichard Weinberger <richard@nod.at>
Thu, 20 Sep 2018 19:37:07 +0000 (21:37 +0200)
commitd3bdc016c598e09a4ddf17805d17d43759b0a582
treea35b6a33a9d3abf650b87e4aa722c5727864591a
parent37f31b6ca4311b94d985fb398a72e5399ad57925
ubifs: drop false positive assertion

The following sequence triggers

ubifs_assert(c, c->lst.taken_empty_lebs > 0);

at the end of ubifs_remount_fs():

mount -t ubifs /dev/ubi0_0 /mnt
echo 1 > /sys/kernel/debug/ubifs/ubi0_0/ro_error
umount /mnt
mount -t ubifs -o ro /dev/ubix_y /mnt
mount -o remount,ro /mnt

The resulting

UBIFS assert failed in ubifs_remount_fs at 1878 (pid 161)

is a false positive. In the case above c->lst.taken_empty_lebs has
never been changed from its initial zero value. This will only happen
when the deferred recovery is done.

Fix this by doing the assertion only when recovery has been done
already.

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Signed-off-by: Richard Weinberger <richard@nod.at>
fs/ubifs/super.c