* [PATCH] ext4: don't free an error pointer
@ 2017-06-30 7:56 Dan Carpenter
2017-06-30 11:17 ` Tahsin Erdogan
2017-07-04 1:37 ` Theodore Ts'o
0 siblings, 2 replies; 3+ messages in thread
From: Dan Carpenter @ 2017-06-30 7:56 UTC (permalink / raw)
To: Theodore Ts'o, Andreas Dilger; +Cc: linux-ext4, kernel-janitors
"goto out;" calls brelse(bh); which will oops on this this path where
"bh" is an error pointer.
Fixes: 2851a40a5d71 ("ext4: xattr-in-inode support")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
---
Oops. I wrote a bug report and a patch for this... Probably we only
need the patch.
diff --git a/fs/ext4/xattr.c b/fs/ext4/xattr.c
index ce12c3fb7e59..f7d6845e147b 100644
--- a/fs/ext4/xattr.c
+++ b/fs/ext4/xattr.c
@@ -1305,10 +1305,8 @@ static int ext4_xattr_inode_write(handle_t *handle, struct inode *ea_inode,
csize = (bufsize - wsize) > blocksize ? blocksize :
bufsize - wsize;
bh = ext4_getblk(handle, ea_inode, block, 0);
- if (IS_ERR(bh)) {
- ret = PTR_ERR(bh);
- goto out;
- }
+ if (IS_ERR(bh))
+ return PTR_ERR(bh);
ret = ext4_journal_get_write_access(handle, bh);
if (ret)
goto out;
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] ext4: don't free an error pointer
2017-06-30 7:56 [PATCH] ext4: don't free an error pointer Dan Carpenter
@ 2017-06-30 11:17 ` Tahsin Erdogan
2017-07-04 1:37 ` Theodore Ts'o
1 sibling, 0 replies; 3+ messages in thread
From: Tahsin Erdogan @ 2017-06-30 11:17 UTC (permalink / raw)
To: Dan Carpenter
Cc: Theodore Ts'o, Andreas Dilger, Ext4 Developers List,
kernel-janitors
Reviewed-by: Tahsin Erdogan <tahsin@google.com>
On Fri, Jun 30, 2017 at 12:56 AM, Dan Carpenter
<dan.carpenter@oracle.com> wrote:
> "goto out;" calls brelse(bh); which will oops on this this path where
> "bh" is an error pointer.
>
> Fixes: 2851a40a5d71 ("ext4: xattr-in-inode support")
> Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
> ---
> Oops. I wrote a bug report and a patch for this... Probably we only
> need the patch.
>
>
> diff --git a/fs/ext4/xattr.c b/fs/ext4/xattr.c
> index ce12c3fb7e59..f7d6845e147b 100644
> --- a/fs/ext4/xattr.c
> +++ b/fs/ext4/xattr.c
> @@ -1305,10 +1305,8 @@ static int ext4_xattr_inode_write(handle_t *handle, struct inode *ea_inode,
> csize = (bufsize - wsize) > blocksize ? blocksize :
> bufsize - wsize;
> bh = ext4_getblk(handle, ea_inode, block, 0);
> - if (IS_ERR(bh)) {
> - ret = PTR_ERR(bh);
> - goto out;
> - }
> + if (IS_ERR(bh))
> + return PTR_ERR(bh);
> ret = ext4_journal_get_write_access(handle, bh);
> if (ret)
> goto out;
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] ext4: don't free an error pointer
2017-06-30 7:56 [PATCH] ext4: don't free an error pointer Dan Carpenter
2017-06-30 11:17 ` Tahsin Erdogan
@ 2017-07-04 1:37 ` Theodore Ts'o
1 sibling, 0 replies; 3+ messages in thread
From: Theodore Ts'o @ 2017-07-04 1:37 UTC (permalink / raw)
To: Dan Carpenter; +Cc: Andreas Dilger, linux-ext4, kernel-janitors
On Fri, Jun 30, 2017 at 10:56:53AM +0300, Dan Carpenter wrote:
> "goto out;" calls brelse(bh); which will oops on this this path where
> "bh" is an error pointer.
>
> Fixes: 2851a40a5d71 ("ext4: xattr-in-inode support")
> Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
> ---
> Oops. I wrote a bug report and a patch for this... Probably we only
> need the patch.
Thanks, I've folded your this to the xattr-in-inode patch and added a
credit for the bugfix to the commit log.
- Ted
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-07-04 1:37 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-06-30 7:56 [PATCH] ext4: don't free an error pointer Dan Carpenter
2017-06-30 11:17 ` Tahsin Erdogan
2017-07-04 1:37 ` Theodore Ts'o
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox