diff options
| author | Andrew Morton <akpm@digeo.com> | 2003-03-08 03:27:42 -0800 |
|---|---|---|
| committer | James Bottomley <jejb@raven.il.steeleye.com> | 2003-03-08 03:27:42 -0800 |
| commit | f6da87caaefffe786e83ca23af7390d7b25456ed (patch) | |
| tree | a46b751005ec12e29194f0c50949353e5b04d82b /fs/ext2/xattr.c | |
| parent | 3ed88a51ae0dd0cbc182d1bf304478cf9c228a0e (diff) | |
| download | history-f6da87caaefffe786e83ca23af7390d7b25456ed.tar.gz | |
[PATCH] Extended attribute sharing and debug macro typo fixes
Patch from Andreas Gruenbacher <agruen@suse.de>
Tony Dziedzic has found two bugs in the extended attributes code. Patches with
explanations are attached.
1: Extended attribute sharing on ext2/ext3 not working
The mb_cache_entry_insert function constantly returns an -EBUSY error
instead of 0, which causes the xattr cache that is needed by the xattr
sharing mechanism on ext2/ext3 to not share anything. This patch fixes
the problem. (It is possible that after applying this fix we will hit
bugs in code that wasn't used before.)
2: Oops in one of the xattr debug statements: The old_bh variable is NULL
if an inode that previously had no EA's assigned would share an EA block
with another inode. (This was hidden by the xattr sharing bug).
Diffstat (limited to 'fs/ext2/xattr.c')
| -rw-r--r-- | fs/ext2/xattr.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/ext2/xattr.c b/fs/ext2/xattr.c index 2383ec92098e2c..046cabba362f54 100644 --- a/fs/ext2/xattr.c +++ b/fs/ext2/xattr.c @@ -731,7 +731,7 @@ ext2_xattr_set2(struct inode *inode, struct buffer_head *old_bh, * The old block will be released after updating * the inode. */ - ea_bdebug(old_bh, "reusing block %ld", + ea_bdebug(new_bh, "reusing block %ld", new_bh->b_blocknr); error = -EDQUOT; |