Commit ea39e712 authored by Ahmet Eray Karadag's avatar Ahmet Eray Karadag Committed by Greg Kroah-Hartman
Browse files

ext4: guard against EA inode refcount underflow in xattr update



commit 57295e83 upstream.

syzkaller found a path where ext4_xattr_inode_update_ref() reads an EA
inode refcount that is already <= 0 and then applies ref_change (often
-1). That lets the refcount underflow and we proceed with a bogus value,
triggering errors like:

  EXT4-fs error: EA inode <n> ref underflow: ref_count=-1 ref_change=-1
  EXT4-fs warning: ea_inode dec ref err=-117

Make the invariant explicit: if the current refcount is non-positive,
treat this as on-disk corruption, emit ext4_error_inode(), and fail the
operation with -EFSCORRUPTED instead of updating the refcount. Delete the
WARN_ONCE() as negative refcounts are now impossible; keep error reporting
in ext4_error_inode().

This prevents the underflow and the follow-on orphan/cleanup churn.

Reported-by: default avatar <syzbot+0be4f339a8218d2a5bb1@syzkaller.appspotmail.com>
Fixes: https://syzbot.org/bug?extid=0be4f339a8218d2a5bb1


Cc: stable@kernel.org
Co-developed-by: default avatarAlbin Babu Varghese <albinbabuvarghese20@gmail.com>
Signed-off-by: default avatarAlbin Babu Varghese <albinbabuvarghese20@gmail.com>
Signed-off-by: default avatarAhmet Eray Karadag <eraykrdg1@gmail.com>
Message-ID: <20250920021342.45575-1-eraykrdg1@gmail.com>
Signed-off-by: default avatarTheodore Ts'o <tytso@mit.edu>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 7a1286e5
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please to comment