[v1] Forwarded: UBSAN: shift-out-of-bounds in dbAllocAG (3)

Forwarded: UBSAN: shift-out-of-bounds in dbAllocAG (3)

syzbot posted 1 patch 3 months, 3 weeks ago

Diff against v1
Download mbox

There is a newer version of this series

fs/jfs/jfs_dmap.c | 6 ++++++
1 file changed, 6 insertions(+)

Expand all Fold all

Forwarded: UBSAN: shift-out-of-bounds in dbAllocAG (3)

Posted by syzbot 3 months, 3 weeks ago

For archival purposes, forwarding an incoming command email to
linux-kernel@vger.kernel.org.

***

Subject: UBSAN: shift-out-of-bounds in dbAllocAG (3)
Author: hsukrut3@gmail.com

#syz test
---
 fs/jfs/jfs_dmap.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/fs/jfs/jfs_dmap.c b/fs/jfs/jfs_dmap.c
index cdfa699cd7c8..477b332763a8 100644
--- a/fs/jfs/jfs_dmap.c
+++ b/fs/jfs/jfs_dmap.c
@@ -1372,6 +1372,12 @@ dbAllocAG(struct bmap * bmp, int agno, s64 nblocks, int l2nb, s64 * results)
 	dcp = (struct dmapctl *) mp->data;
 	budmin = dcp->budmin;
 
+	if (unlikely(budmin < 0) {
+		jfs_err("JFS: dmapctl corruption: budmin=%d", budmin);
+		release_metapage(mp);
+		return -EIO;
+	}
+
 	if (dcp->leafidx != cpu_to_le32(CTLLEAFIND)) {
 		jfs_error(bmp->db_ipbmap->i_sb, "Corrupt dmapctl page\n");
 		release_metapage(mp);
-- 
2.43.0