Skip to content
  • Eric W. Biederman's avatar
    kobject: Fix kobject_rename and !CONFIG_SYSFS · 030c1d2b
    Eric W. Biederman authored
    
    
    When looking at kobject_rename I found two bugs with
    that exist when sysfs support is disabled in the kernel.
    
    kobject_rename does not change the name on the kobject when
    sysfs support is not compiled in.
    
    kobject_rename without locking attempts to check the
    validity of a rename operation, which the kobject layer
    simply does not have the infrastructure to do.
    
    This patch documents the previously unstated requirement of
    kobject_rename that is the responsibility of the caller to
    provide mutual exclusion and to be certain that the new_name
    for the kobject is valid.
    
    This patch modifies sysfs_rename_dir in !CONFIG_SYSFS case
    to call kobject_set_name to actually change the kobject_name.
    
    This patch removes the bogus and misleading check in kobject_rename
    that attempts to see if a rename is valid.  The check is bogus
    because we do not have the proper locking.  The check is misleading
    because it looks like we can and do perform checking at the kobject
    level that we don't.
    
    Signed-off-by: default avatarEric W. Biederman <ebiederm@xmission.com>
    Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
    030c1d2b