Skip to content

Commit b052230

Browse files
YuezhangMonamjaejeon
authored andcommitted
exfat: fix the infinite loop in exfat_find_last_cluster()
In exfat_find_last_cluster(), the cluster chain is traversed until the EOF cluster. If the cluster chain includes a loop due to file system corruption, the EOF cluster cannot be traversed, resulting in an infinite loop. If the number of clusters indicated by the file size is inconsistent with the cluster chain length, exfat_find_last_cluster() will return an error, so if this inconsistency is found, the traversal can be aborted without traversing to the EOF cluster. Reported-by: syzbot+f7d147e6db52b1e09dba@syzkaller.appspotmail.com Closes: https://syzkaller.appspot.com/bug?extid=f7d147e6db52b1e09dba Tested-by: syzbot+f7d147e6db52b1e09dba@syzkaller.appspotmail.com Fixes: 3102386 ("exfat: add fat entry operations") Signed-off-by: Yuezhang Mo <Yuezhang.Mo@sony.com> Signed-off-by: Namjae Jeon <linkinjeon@kernel.org>
1 parent 1bb7ff4 commit b052230

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

fs/exfat/fatent.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -294,7 +294,7 @@ int exfat_find_last_cluster(struct super_block *sb, struct exfat_chain *p_chain,
294294
clu = next;
295295
if (exfat_ent_get(sb, clu, &next))
296296
return -EIO;
297-
} while (next != EXFAT_EOF_CLUSTER);
297+
} while (next != EXFAT_EOF_CLUSTER && count <= p_chain->size);
298298

299299
if (p_chain->size != count) {
300300
exfat_fs_error(sb,

0 commit comments

Comments
 (0)