Re: [PATCH v2] ext4: missing !bh check in ext4_xattr_inode_write()

2018-11-08 Thread Theodore Y. Ts'o
On Thu, Nov 08, 2018 at 09:46:30AM +0300, Vasily Averin wrote: > diff --git a/fs/ext4/xattr.c b/fs/ext4/xattr.c > index 0b9688683526..415f73d4c9e6 100644 > --- a/fs/ext4/xattr.c > +++ b/fs/ext4/xattr.c > @@ -1384,6 +1384,12 @@ static int ext4_xattr_inode_write(handle_t *handle, > struct inode *ea_

[PATCH v2] ext4: missing !bh check in ext4_xattr_inode_write()

2018-11-07 Thread Vasily Averin
According to Ted Ts'o ext4_getblk() called in ext4_xattr_inode_write() should not return bh = NULL The only time that bh could be NULL, then, would be in the case of something really going wrong; a programming error elsewhere (perhaps a wild pointer dereference) or I/O error causing on-disk file s