diff options
| author | Linus Torvalds <torvalds@linux-foundation.org> | 2019-10-10 08:16:44 -0700 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2019-10-10 08:16:44 -0700 |
| commit | 015c21ba59fcbc522d7a9d1e0ab0f0e6c0895ff0 (patch) | |
| tree | 4fa0e91287d927e9df8e1658f93c40ca9efd60bc | |
| parent | 5e48e55fb57a9026e6b3b6961def6d2aeb210659 (diff) | |
| parent | 6fcf0c72e4b9360768cf5ef543c4f14c34800ee8 (diff) | |
| download | linux-015c21ba59fcbc522d7a9d1e0ab0f0e6c0895ff0.tar.gz | |
Merge branch 'work.mount3' of git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs
Pull mount fixes from Al Viro:
"A couple of regressions from the mount series"
* 'work.mount3' of git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs:
vfs: add missing blkdev_put() in get_tree_bdev()
shmem: fix LSM options parsing
| -rw-r--r-- | fs/super.c | 5 | ||||
| -rw-r--r-- | mm/shmem.c | 6 |
2 files changed, 10 insertions, 1 deletions
diff --git a/fs/super.c b/fs/super.c index f627b7c53d2b7f..cfadab2cbf35fd 100644 --- a/fs/super.c +++ b/fs/super.c @@ -1300,6 +1300,7 @@ int get_tree_bdev(struct fs_context *fc, mutex_lock(&bdev->bd_fsfreeze_mutex); if (bdev->bd_fsfreeze_count > 0) { mutex_unlock(&bdev->bd_fsfreeze_mutex); + blkdev_put(bdev, mode); warnf(fc, "%pg: Can't mount, blockdev is frozen", bdev); return -EBUSY; } @@ -1308,8 +1309,10 @@ int get_tree_bdev(struct fs_context *fc, fc->sget_key = bdev; s = sget_fc(fc, test_bdev_super_fc, set_bdev_super_fc); mutex_unlock(&bdev->bd_fsfreeze_mutex); - if (IS_ERR(s)) + if (IS_ERR(s)) { + blkdev_put(bdev, mode); return PTR_ERR(s); + } if (s->s_root) { /* Don't summarily change the RO/RW state. */ diff --git a/mm/shmem.c b/mm/shmem.c index cd570cc79c76ab..220be9fa2c417c 100644 --- a/mm/shmem.c +++ b/mm/shmem.c @@ -3482,6 +3482,12 @@ static int shmem_parse_options(struct fs_context *fc, void *data) { char *options = data; + if (options) { + int err = security_sb_eat_lsm_opts(options, &fc->security); + if (err) + return err; + } + while (options != NULL) { char *this_char = options; for (;;) { |