• Milton D. Miller II's avatar
    [PATCH] fix sysfs backing store error path confusion · 2cb7ab3e
    Milton D. Miller II authored
    On Nov 3, 2004, at 3:42 PM, Greg KH wrote:
    
    |On Tue, Nov 02, 2004 at 10:03:34AM -0600, Maneesh Soni wrote:
    ||On Tue, Nov 02, 2004 at 02:46:58AM -0600, Milton Miller wrote:
    |||sysfs_new_dirent returns ERR_PTR(-ENOMEM) if kmalloc fails but the callers
    |||were expecting NULL.
    ||
    ||Thanks for spotting this. But as you said, I will prefer to change the callee.
    ||How about this patch?
    ..
    ||-		return -ENOMEM;
    ||+		return NULL;
    |
    |Actually, this needs to be a 0, not NULL, otherwise the compiler
    |complains with a warning.  I've fixed it up and applied it.
    |
    |thanks,
    |
    |greg k-h
    
    I wondered why greg thought the type was wrong.   After it was merged I
    realized that the wrong function was changed.  Here's an attempt to fix
    both errors.
    Signed-off-by: default avatarGreg Kroah-Hartman <greg@kroah.com>
    2cb7ab3e
dir.c 10.2 KB