[kbuild-all] [android-common:android-4.19 9/9] security/selinux/ss/services.c:2304:17: warning: variable 'sidtab' set but not used

tree: https://android.googlesource.com/kernel/common android-4.19 head: 999765145e535816fdad1a7949e202cd8784cf1e commit: 999765145e535816fdad1a7949e202cd8784cf1e [9/9] UPSTREAM: selinux: sidtab reverse lookup hash table config: i386-randconfig-r015-20210209 (attached as .config) compiler: gcc-9 (Debian 9.3.0-15) 9.3.0 reproduce (this is a W=1 build): git remote add android-common https://android.googlesource.com/kernel/common git fetch --no-tags android-common android-4.19 git checkout 999765145e535816fdad1a7949e202cd8784cf1e # save the attached .config to linux build tree make W=1 ARCH=i386 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp(a)intel.com&gt; All warnings (new ones prefixed by >>): security/selinux/ss/services.c: In function 'security_port_sid': 2304 | struct sidtab *sidtab; | ^~~~~~ security/selinux/ss/services.c: In function 'security_ib_endport_sid': security/selinux/ss/services.c:2393:17: warning: variable 'sidtab' set but not used [-Wunused-but-set-variable] 2393 | struct sidtab *sidtab; | ^~~~~~ security/selinux/ss/services.c: In function 'security_netif_sid': security/selinux/ss/services.c:2438:17: warning: variable 'sidtab' set but not used [-Wunused-but-set-variable] 2438 | struct sidtab *sidtab; | ^~~~~~ security/selinux/ss/services.c: In function 'security_fs_use': security/selinux/ss/services.c:2789:17: warning: variable 'sidtab' set but not used [-Wunused-but-set-variable] 2789 | struct sidtab *sidtab; | ^~~~~~ security/selinux/ss/services.c:1113: warning: Function parameter or member 'state' not described in 'security_compute_av' security/selinux/ss/services.c:1113: warning: Function parameter or member 'orig_tclass' not described in 'security_compute_av' security/selinux/ss/services.c:1113: warning: Excess function parameter 'tclass' description in 'security_compute_av' security/selinux/ss/services.c:1361: warning: Function parameter or member 'state' not described in 'security_sid_to_context' security/selinux/ss/services.c:1567: warning: Function parameter or member 'state' not described in 'security_context_to_sid' security/selinux/ss/services.c:1600: warning: Function parameter or member 'state' not described in 'security_context_to_sid_default' security/selinux/ss/services.c:1600: warning: Function parameter or member 'gfp_flags' not described in 'security_context_to_sid_default' security/selinux/ss/services.c:1867: warning: Function parameter or member 'state' not described in 'security_transition_sid' security/selinux/ss/services.c:1867: warning: Function parameter or member 'qstr' not described in 'security_transition_sid' security/selinux/ss/services.c:1900: warning: Function parameter or member 'state' not described in 'security_member_sid' security/selinux/ss/services.c:1924: warning: Function parameter or member 'state' not described in 'security_change_sid' security/selinux/ss/services.c:2126: warning: Function parameter or member 'state' not described in 'security_load_policy' security/selinux/ss/services.c:2302: warning: Function parameter or member 'state' not described in 'security_port_sid' security/selinux/ss/services.c:2347: warning: Function parameter or member 'state' not described in 'security_ib_pkey_sid' security/selinux/ss/services.c:2391: warning: Function parameter or member 'state' not described in 'security_ib_endport_sid' security/selinux/ss/services.c:2391: warning: Function parameter or member 'port_num' not described in 'security_ib_endport_sid' security/selinux/ss/services.c:2391: warning: Excess function parameter 'port' description in 'security_ib_endport_sid' security/selinux/ss/services.c:2436: warning: Function parameter or member 'state' not described in 'security_netif_sid' security/selinux/ss/services.c:2499: warning: Function parameter or member 'state' not described in 'security_node_sid' security/selinux/ss/services.c:2586: warning: Function parameter or member 'state' not described in 'security_get_user_sids' security/selinux/ss/services.c:2711: warning: Function parameter or member 'state' not described in '__security_genfs_sid' security/selinux/ss/services.c:2711: warning: Function parameter or member 'orig_sclass' not described in '__security_genfs_sid' security/selinux/ss/services.c:2711: warning: Excess function parameter 'sclass' description in '__security_genfs_sid' security/selinux/ss/services.c:2773: warning: Function parameter or member 'state' not described in 'security_genfs_sid' security/selinux/ss/services.c:2773: warning: Function parameter or member 'orig_sclass' not described in 'security_genfs_sid' security/selinux/ss/services.c:2773: warning: Excess function parameter 'sclass' description in 'security_genfs_sid' security/selinux/ss/services.c:2787: warning: Function parameter or member 'state' not described in 'security_fs_use' security/selinux/ss/services.c:3092: warning: Function parameter or member 'state' not described in 'security_net_peersid_resolve' security/selinux/ss/services.c:3092: warning: Function parameter or member 'peer_sid' not described in 'security_net_peersid_resolve' security/selinux/ss/services.c:3284: warning: Function parameter or member 'state' not described in 'security_policycap_supported' security/selinux/ss/services.c:3616: warning: Function parameter or member 'state' not described in 'security_netlbl_secattr_to_sid' security/selinux/ss/services.c:3685: warning: Function parameter or member 'state' not described in 'security_netlbl_sid_to_secattr' security/selinux/ss/services.c:3724: warning: Function parameter or member 'state' not described in 'security_read_policy' vim +/sidtab +2304 security/selinux/ss/services.c cee74f47a6baba Eric Paris 2010-10-13 2293 ^1da177e4c3f41 Linus Torvalds 2005-04-16 2294 /** ^1da177e4c3f41 Linus Torvalds 2005-04-16 2295 * security_port_sid - Obtain the SID for a port. ^1da177e4c3f41 Linus Torvalds 2005-04-16 2296 * @protocol: protocol number ^1da177e4c3f41 Linus Torvalds 2005-04-16 2297 * @port: port number ^1da177e4c3f41 Linus Torvalds 2005-04-16 2298 * @out_sid: security identifier ^1da177e4c3f41 Linus Torvalds 2005-04-16 2299 */ aa8e712cee93d5 Stephen Smalley 2018-03-01 2300 int security_port_sid(struct selinux_state *state, aa8e712cee93d5 Stephen Smalley 2018-03-01 2301 u8 protocol, u16 port, u32 *out_sid) ^1da177e4c3f41 Linus Torvalds 2005-04-16 2302 { aa8e712cee93d5 Stephen Smalley 2018-03-01 2303 struct policydb *policydb; aa8e712cee93d5 Stephen Smalley 2018-03-01 @2304 struct sidtab *sidtab; ^1da177e4c3f41 Linus Torvalds 2005-04-16 2305 struct ocontext *c; ^1da177e4c3f41 Linus Torvalds 2005-04-16 2306 int rc = 0; ^1da177e4c3f41 Linus Torvalds 2005-04-16 2307 aa8e712cee93d5 Stephen Smalley 2018-03-01 2308 read_lock(&state->ss->policy_rwlock); ^1da177e4c3f41 Linus Torvalds 2005-04-16 2309 aa8e712cee93d5 Stephen Smalley 2018-03-01 2310 policydb = &state->ss->policydb; 9ed9e913b79972 Ondrej Mosnacek 2018-11-30 2311 sidtab = state->ss->sidtab; aa8e712cee93d5 Stephen Smalley 2018-03-01 2312 aa8e712cee93d5 Stephen Smalley 2018-03-01 2313 c = policydb->ocontexts[OCON_PORT]; ^1da177e4c3f41 Linus Torvalds 2005-04-16 2314 while (c) { ^1da177e4c3f41 Linus Torvalds 2005-04-16 2315 if (c->u.port.protocol == protocol && ^1da177e4c3f41 Linus Torvalds 2005-04-16 2316 c->u.port.low_port <= port && ^1da177e4c3f41 Linus Torvalds 2005-04-16 2317 c->u.port.high_port >= port) ^1da177e4c3f41 Linus Torvalds 2005-04-16 2318 break; ^1da177e4c3f41 Linus Torvalds 2005-04-16 2319 c = c->next; ^1da177e4c3f41 Linus Torvalds 2005-04-16 2320 } ^1da177e4c3f41 Linus Torvalds 2005-04-16 2321 ^1da177e4c3f41 Linus Torvalds 2005-04-16 2322 if (c) { ^1da177e4c3f41 Linus Torvalds 2005-04-16 2323 if (!c->sid[0]) { 999765145e5358 Jeff Vander Stoep 2019-11-22 2324 rc = context_struct_to_sid(state, &c->context[0], ^1da177e4c3f41 Linus Torvalds 2005-04-16 2325 &c->sid[0]); ^1da177e4c3f41 Linus Torvalds 2005-04-16 2326 if (rc) ^1da177e4c3f41 Linus Torvalds 2005-04-16 2327 goto out; ^1da177e4c3f41 Linus Torvalds 2005-04-16 2328 } ^1da177e4c3f41 Linus Torvalds 2005-04-16 2329 *out_sid = c->sid[0]; ^1da177e4c3f41 Linus Torvalds 2005-04-16 2330 } else { ^1da177e4c3f41 Linus Torvalds 2005-04-16 2331 *out_sid = SECINITSID_PORT; ^1da177e4c3f41 Linus Torvalds 2005-04-16 2332 } ^1da177e4c3f41 Linus Torvalds 2005-04-16 2333 ^1da177e4c3f41 Linus Torvalds 2005-04-16 2334 out: aa8e712cee93d5 Stephen Smalley 2018-03-01 2335 read_unlock(&state->ss->policy_rwlock); ^1da177e4c3f41 Linus Torvalds 2005-04-16 2336 return rc; ^1da177e4c3f41 Linus Torvalds 2005-04-16 2337 } ^1da177e4c3f41 Linus Torvalds 2005-04-16 2338 :::::: The code at line 2304 was first introduced by commit :::::: aa8e712cee93d520e96a2ca8e3a20f807c937e3f selinux: wrap global selinux state :::::: TO: Stephen Smalley <sds(a)tycho.nsa.gov&gt; :::::: CC: Paul Moore <paul(a)paul-moore.com&gt; --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org