All of lore.kernel.org
 help / color / mirror / Atom feed
* [driver-core:debugfs_cleanup 7/7] fs/sysfs/file.c:397:8: warning: variable 'uid' is uninitialized when used here
@ 2023-08-31 13:39 kernel test robot
  0 siblings, 0 replies; only message in thread
From: kernel test robot @ 2023-08-31 13:39 UTC (permalink / raw)
  To: Greg Kroah-Hartman; +Cc: llvm, oe-kbuild-all, devel, Alistair Francis

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/driver-core.git debugfs_cleanup
head:   2929d17b58d02dcf52d0345fa966c616e09a5afa
commit: 2929d17b58d02dcf52d0345fa966c616e09a5afa [7/7] sysfs: do not create empty directories if no attributes are present
config: i386-randconfig-012-20230831 (https://download.01.org/0day-ci/archive/20230831/202308312145.bv4S52ys-lkp@intel.com/config)
compiler: clang version 16.0.4 (https://github.com/llvm/llvm-project.git ae42196bc493ffe877a7e3dff8be32035dea4d07)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20230831/202308312145.bv4S52ys-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202308312145.bv4S52ys-lkp@intel.com/

All warnings (new ones prefixed by >>):

>> fs/sysfs/file.c:397:8: warning: variable 'uid' is uninitialized when used here [-Wuninitialized]
                                             uid, gid, kobj, NULL);
                                             ^~~
   fs/sysfs/file.c:383:2: note: variable 'uid' is declared here
           kuid_t uid;
           ^
>> fs/sysfs/file.c:397:13: warning: variable 'gid' is uninitialized when used here [-Wuninitialized]
                                             uid, gid, kobj, NULL);
                                                  ^~~
   fs/sysfs/file.c:384:2: note: variable 'gid' is declared here
           kgid_t gid;
           ^
   2 warnings generated.


vim +/uid +397 fs/sysfs/file.c

   372	
   373	/**
   374	 * sysfs_add_file_to_group - add an attribute file to a pre-existing group.
   375	 * @kobj: object we're acting for.
   376	 * @attr: attribute descriptor.
   377	 * @group: group name.
   378	 */
   379	int sysfs_add_file_to_group(struct kobject *kobj,
   380			const struct attribute *attr, const char *group)
   381	{
   382		struct kernfs_node *parent;
   383		kuid_t uid;
   384		kgid_t gid;
   385		int error;
   386	
   387		if (group) {
   388			parent = kernfs_find_and_get(kobj->sd, group);
   389		} else {
   390			parent = kobj->sd;
   391			kernfs_get(parent);
   392		}
   393	
   394		if (!parent) {
   395			parent = kernfs_create_dir_ns(kobj->sd, group,
   396						  S_IRWXU | S_IRUGO | S_IXUGO,
 > 397						  uid, gid, kobj, NULL);
   398			if (IS_ERR(parent)) {
   399				if (PTR_ERR(parent) == -EEXIST)
   400					sysfs_warn_dup(kobj->sd, group);
   401				return PTR_ERR(parent);
   402			}
   403	
   404			kernfs_get(parent);
   405		}
   406	
   407		kobject_get_ownership(kobj, &uid, &gid);
   408		error = sysfs_add_file_mode_ns(parent, attr, attr->mode, uid, gid,
   409					       NULL);
   410		kernfs_put(parent);
   411	
   412		return error;
   413	}
   414	EXPORT_SYMBOL_GPL(sysfs_add_file_to_group);
   415	

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2023-08-31 14:01 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-08-31 13:39 [driver-core:debugfs_cleanup 7/7] fs/sysfs/file.c:397:8: warning: variable 'uid' is uninitialized when used here kernel test robot

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.