* [PATCH & RESEND] avoid NULL deference in ext2_xattr_get
@ 2010-07-11 2:16 crosslonelyover
2010-07-11 8:31 ` shenghui
0 siblings, 1 reply; 2+ messages in thread
From: crosslonelyover @ 2010-07-11 2:16 UTC (permalink / raw)
To: kernel-janitors, linux-kernel, linux-ext4, linux-kernel,
kernel-janitors, error27
Hi Dan Carpenter,
Thanks for your instructions. I regenerated the patch.
Please check it.
>From 3149865e0d118b939c27659a4e4c884eaa6b3a4c Mon Sep 17 00:00:00 2001
From: Wang Sheng-Hui <crosslonelyover@gmail.com>
Date: Sat, 10 Jul 2010 22:07:37 -0400
Subject: [PATCH] avoid NULL deference in ext2_xattr_get
Signed-off-by: Wang Sheng-Hui <crosslonelyover@gmail.com>
---
fs/ext2/xattr.c | 5 +++--
1 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/fs/ext2/xattr.c b/fs/ext2/xattr.c
index 7c39157..81ec1c6 100644
--- a/fs/ext2/xattr.c
+++ b/fs/ext2/xattr.c
@@ -156,11 +156,12 @@ ext2_xattr_get(struct inode *inode, int name_index, const char *name,
char *end;
int error;
+ if (name == NULL)
+ return -EINVAL;
+
ea_idebug(inode, "name=%d.%s, buffer=%p, buffer_size=%ld",
name_index, name, buffer, (long)buffer_size);
- if (name == NULL)
- return -EINVAL;
down_read(&EXT2_I(inode)->xattr_sem);
error = -ENODATA;
if (!EXT2_I(inode)->i_file_acl)
--
1.7.1.1
^ permalink raw reply related [flat|nested] 2+ messages in thread* Re: [PATCH & RESEND] avoid NULL deference in ext2_xattr_get
2010-07-11 2:16 [PATCH & RESEND] avoid NULL deference in ext2_xattr_get crosslonelyover
@ 2010-07-11 8:31 ` shenghui
0 siblings, 0 replies; 2+ messages in thread
From: shenghui @ 2010-07-11 8:31 UTC (permalink / raw)
To: kernel-janitors, linux-kernel, linux-ext4, error27
2010/7/11 crosslonelyover <crosslonelyover@gmail.com>:
> Hi Dan Carpenter,
>
> Thanks for your instructions. I regenerated the patch.
> Please check it.
>
> From 3149865e0d118b939c27659a4e4c884eaa6b3a4c Mon Sep 17 00:00:00 2001
> From: Wang Sheng-Hui <crosslonelyover@gmail.com>
> Date: Sat, 10 Jul 2010 22:07:37 -0400
> Subject: [PATCH] avoid NULL deference in ext2_xattr_get
>
>
> Signed-off-by: Wang Sheng-Hui <crosslonelyover@gmail.com>
> ---
> fs/ext2/xattr.c | 5 +++--
> 1 files changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/fs/ext2/xattr.c b/fs/ext2/xattr.c
> index 7c39157..81ec1c6 100644
> --- a/fs/ext2/xattr.c
> +++ b/fs/ext2/xattr.c
> @@ -156,11 +156,12 @@ ext2_xattr_get(struct inode *inode, int name_index, const char *name,
> char *end;
> int error;
>
> + if (name == NULL)
> + return -EINVAL;
> +
> ea_idebug(inode, "name=%d.%s, buffer=%p, buffer_size=%ld",
> name_index, name, buffer, (long)buffer_size);
>
> - if (name == NULL)
> - return -EINVAL;
> down_read(&EXT2_I(inode)->xattr_sem);
> error = -ENODATA;
> if (!EXT2_I(inode)->i_file_acl)
> --
> 1.7.1.1
>
>
>
Please ignore this patch. Just for gmail format check.
Thanks for David Beal's explanation!
--
Thanks and Best Regards,
shenghui
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2010-07-11 8:31 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-07-11 2:16 [PATCH & RESEND] avoid NULL deference in ext2_xattr_get crosslonelyover
2010-07-11 8:31 ` shenghui
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox