gfs2: bypass signal_our_withdraw if no journal
authorBob Peterson <rpeterso@redhat.com>
Fri, 12 Mar 2021 12:58:54 +0000 (07:58 -0500)
committerAndreas Gruenbacher <agruenba@redhat.com>
Fri, 12 Mar 2021 13:55:23 +0000 (14:55 +0100)
commitd5bf630f355d8c532bef2347cf90e8ae60a5f1bd
treecb7cdc1c95c123fa07ded5b6f017cf68a70f53be
parent1a5a2cfd34c17db73c53ef127272c8c1ae220485
gfs2: bypass signal_our_withdraw if no journal

Before this patch, function signal_our_withdraw referenced the journal
inode immediately. But corrupt file systems may have some invalid
journals, in which case our attempt to read it in will withdraw and the
resulting signal_our_withdraw would dereference the NULL value.

This patch adds a check to signal_our_withdraw so that if the journal
has not yet been initialized, it simply returns and does the old-style
withdraw.

Thanks, Andy Price, for his analysis.

Reported-by: syzbot+50a8a9cf8127f2c6f5df@syzkaller.appspotmail.com
Fixes: 601ef0d52e96 ("gfs2: Force withdraw to replay journals and wait for it to finish")
Signed-off-by: Bob Peterson <rpeterso@redhat.com>
Signed-off-by: Andreas Gruenbacher <agruenba@redhat.com>
fs/gfs2/util.c