]> git.ipfire.org Git - thirdparty/kernel/stable.git/commit
ubi: ubi_create_volume: Fix use-after-free when volume creation failed
authorZhihao Cheng <chengzhihao1@huawei.com>
Tue, 10 May 2022 12:31:26 +0000 (20:31 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 14 Jun 2022 14:59:33 +0000 (16:59 +0200)
commitabb67043060f2bf4c03d7c3debb9ae980e2b6db3
tree8202b61dc86edc4f0575f596b636be3e11c8e730
parent28048a4cf3813b7cf5cc8cce629dfdc7951cb1c2
ubi: ubi_create_volume: Fix use-after-free when volume creation failed

[ Upstream commit 8c03a1c21d72210f81cb369cc528e3fde4b45411 ]

There is an use-after-free problem for 'eba_tbl' in ubi_create_volume()'s
error handling path:

  ubi_eba_replace_table(vol, eba_tbl)
    vol->eba_tbl = tbl
out_mapping:
  ubi_eba_destroy_table(eba_tbl)   // Free 'eba_tbl'
out_unlock:
  put_device(&vol->dev)
    vol_release
      kfree(tbl->entries)   // UAF

Fix it by removing redundant 'eba_tbl' releasing.
Fetch a reproducer in [Link].

Fixes: 493cfaeaa0c9b ("mtd: utilize new cdev_device_add helper function")
Link: https://bugzilla.kernel.org/show_bug.cgi?id=215965
Signed-off-by: Zhihao Cheng <chengzhihao1@huawei.com>
Signed-off-by: Richard Weinberger <richard@nod.at>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/mtd/ubi/vmt.c