Commit 798a9cad authored by Brian Foster's avatar Brian Foster Committed by Darrick J. Wong

xfs: fix mount failure crash on invalid iclog memory access

syzbot (via KASAN) reports a use-after-free in the error path of
xlog_alloc_log(). Specifically, the iclog freeing loop doesn't
handle the case of a fully initialized ->l_iclog linked list.
Instead, it assumes that the list is partially constructed and NULL
terminated.

This bug manifested because there was no possible error scenario
after iclog list setup when the original code was added.  Subsequent
code and associated error conditions were added some time later,
while the original error handling code was never updated. Fix up the
error loop to terminate either on a NULL iclog or reaching the end
of the list.

Reported-by: syzbot+c732f8644185de340492@syzkaller.appspotmail.com
Signed-off-by: default avatarBrian Foster <bfoster@redhat.com>
Reviewed-by: default avatarDarrick J. Wong <darrick.wong@oracle.com>
Signed-off-by: default avatarDarrick J. Wong <darrick.wong@oracle.com>
parent 69ffe596
...@@ -1542,6 +1542,8 @@ xlog_alloc_log( ...@@ -1542,6 +1542,8 @@ xlog_alloc_log(
prev_iclog = iclog->ic_next; prev_iclog = iclog->ic_next;
kmem_free(iclog->ic_data); kmem_free(iclog->ic_data);
kmem_free(iclog); kmem_free(iclog);
if (prev_iclog == log->l_iclog)
break;
} }
out_free_log: out_free_log:
kmem_free(log); kmem_free(log);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment