fs: ext4: Unmount FS in do_fs_type()
authorMarek Vasut <marek.vasut+renesas@gmail.com>
Wed, 6 Feb 2019 12:19:29 +0000 (13:19 +0100)
committerVagrant Cascadian <vagrant@debian.org>
Fri, 1 Mar 2019 04:42:31 +0000 (04:42 +0000)
commit7edc5d5fe11c0a0c25e22d4315100f4e8b4ece7e
tree7560269a16b784d6c96c375306811ffb49892d3d
parent9d8ee90c1046e7fe4eaab863683975cb844be63d
fs: ext4: Unmount FS in do_fs_type()

Unlike other generic FS accessors, fs_get_info() does not call fs_close()
at the end of it's operation. Thus, using fs_get_info() in do_fs_type()
without calling fs_close() causes potential memory leak by creating new
filesystem structures on each call of do_fs_type().

The test case to trigger this problem is as follows. It is required to
have ext4 filesystem on the first partition of the SDMMC device, since
ext4 requires stateful mount and causes memory allocation.
=> while true ; do mmc rescan ; fstype mmc 1 ; done
Eventually, the mounting of ext4 will fail due to malloc failures
and the filesystem will not be correctly detected.

This patch fixes the problem by adding the missing fs_close().

Signed-off-by: Marek Vasut <marek.vasut+renesas@gmail.com>
Cc: Simon Glass <sjg@chromium.org>
Cc: Tom Rini <trini@konsulko.com>
Gbp-Pq: Topic upstream
Gbp-Pq: Name fs-ext4-Unmount-FS-in-do_fs_type.patch
fs/fs.c