Re: [PATCH 1/1] fs-sysfs bugfix of sysfs_hash_and_removedereferencing before checking for NULL

From: Greg KH
Date: Mon Jun 25 2007 - 17:57:21 EST


On Fri, Jun 22, 2007 at 10:13:24AM -0700, Doug Thompson wrote:
> From: Douglas Thompson <dougthompson@xxxxxxxxxxxx>
>
> In tracking down one of my bugs in using sysfs, I found the kernel doing a NULL de-reference
> in function fs/sysfs/inode.c:sysfs_hash_and_remove(), when I (incorrectly) passed
> in a dentry value of NULL.
>
> The check for NULL should occur BEFORE the dereference of 'dir'.
> This patch modifies the sequence and does the check FIRST
>
>
> Signed-off-by: Douglas Thompson <dougthompson@xxxxxxxxxxxx>
> ---
>
> Index: linux-2.6.22-rc4-mm2/fs/sysfs/inode.c

There's been a lot of churn and change in this area and this patch
doens't apply anymore. Can you try respinning it against the next -mm
release and resending it if it is still needed?

thanks,

greg k-h

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/