| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Thu, 9 Jun 2022 03:36:21 +0800
From: kernel test robot <lkp@...el.com>
To: Chenbo Feng <fengc@...gle.com>
Cc: kbuild-all@...ts.01.org, linux-kernel@...r.kernel.org,
Guenter Roeck <groeck@...omium.org>
Subject: [jsarha:topic/cros-sof-v4.19 110/6555]
net/netfilter/xt_qtaguid.c:1835:27: warning: function
'prdebug_full_state_locked' might be a candidate for 'gnu_printf' format
attribute
tree: https://github.com/jsarha/linux topic/cros-sof-v4.19
head: d7a3e91d8d16d1ef8653deec5a1fffc4de034a0c
commit: fe6bc43a5e8411303b6096e8976d87ae052926fd [110/6555] ANDROID: netfilter: xt_qtaguid: fix the deadlock when enable DDEBUG
config: sh-randconfig-r005-20220608 (https://download.01.org/0day-ci/archive/20220609/202206090347.2RtNCOCc-lkp@intel.com/config)
compiler: sh4-linux-gcc (GCC) 11.3.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/jsarha/linux/commit/fe6bc43a5e8411303b6096e8976d87ae052926fd
git remote add jsarha https://github.com/jsarha/linux
git fetch --no-tags jsarha topic/cros-sof-v4.19
git checkout fe6bc43a5e8411303b6096e8976d87ae052926fd
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.3.0 make.cross W=1 O=build_dir ARCH=sh SHELL=/bin/bash net/netfilter/
If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@...el.com>
All warnings (new ones prefixed by >>):
1812 | MT_DEBUG("qtaguid[%d]: left %d\n", par->hooknum, res);
| ^~~~~~~~
include/linux/compiler.h:25:39: warning: ignoring attribute 'section ("_ftrace_annotated_branch")' because it conflicts with previous 'section ("_ftrace_branch")' [-Wattributes]
25 | static struct ftrace_likely_data \
| ^~~~~~~~~~~~~~~~~~
include/linux/compiler.h:69:30: note: in definition of macro '__trace_if'
69 | ______r = !!(cond); \
| ^~~~
include/linux/dynamic_debug.h:126:9: note: in expansion of macro 'if'
126 | if (DYNAMIC_DEBUG_BRANCH(descriptor)) \
| ^~
include/linux/compiler.h:45:26: note: in expansion of macro '__branch_check__'
45 | # define likely(x) (__branch_check__(x, 1, __builtin_constant_p(x)))
| ^~~~~~~~~~~~~~~~
include/linux/dynamic_debug.h:115:9: note: in expansion of macro 'likely'
115 | likely(descriptor.flags & _DPRINTK_FLAGS_PRINT)
| ^~~~~~
include/linux/dynamic_debug.h:126:13: note: in expansion of macro 'DYNAMIC_DEBUG_BRANCH'
126 | if (DYNAMIC_DEBUG_BRANCH(descriptor)) \
| ^~~~~~~~~~~~~~~~~~~~
include/linux/printk.h:340:9: note: in expansion of macro 'dynamic_pr_debug'
340 | dynamic_pr_debug(fmt, ##__VA_ARGS__)
| ^~~~~~~~~~~~~~~~
net/netfilter/xt_qtaguid_internal.h:44:25: note: in expansion of macro 'pr_debug'
44 | pr_debug(__VA_ARGS__); \
| ^~~~~~~~
net/netfilter/xt_qtaguid_internal.h:52:23: note: in expansion of macro 'MSK_DEBUG'
52 | #define MT_DEBUG(...) MSK_DEBUG(MDEBUG_MASK, __VA_ARGS__)
| ^~~~~~~~~
net/netfilter/xt_qtaguid.c:1812:9: note: in expansion of macro 'MT_DEBUG'
1812 | MT_DEBUG("qtaguid[%d]: left %d\n", par->hooknum, res);
| ^~~~~~~~
include/linux/compiler.h:64:25: note: previous declaration here
64 | ______f = { \
| ^~~~~~~
include/linux/compiler.h:56:23: note: in expansion of macro '__trace_if'
56 | #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
| ^~~~~~~~~~
include/linux/dynamic_debug.h:126:9: note: in expansion of macro 'if'
126 | if (DYNAMIC_DEBUG_BRANCH(descriptor)) \
| ^~
include/linux/printk.h:340:9: note: in expansion of macro 'dynamic_pr_debug'
340 | dynamic_pr_debug(fmt, ##__VA_ARGS__)
| ^~~~~~~~~~~~~~~~
net/netfilter/xt_qtaguid_internal.h:44:25: note: in expansion of macro 'pr_debug'
44 | pr_debug(__VA_ARGS__); \
| ^~~~~~~~
net/netfilter/xt_qtaguid_internal.h:52:23: note: in expansion of macro 'MSK_DEBUG'
52 | #define MT_DEBUG(...) MSK_DEBUG(MDEBUG_MASK, __VA_ARGS__)
| ^~~~~~~~~
net/netfilter/xt_qtaguid.c:1812:9: note: in expansion of macro 'MT_DEBUG'
1812 | MT_DEBUG("qtaguid[%d]: left %d\n", par->hooknum, res);
| ^~~~~~~~
In file included from include/linux/printk.h:336,
from include/linux/kernel.h:14,
from include/linux/bitmap.h:10,
from include/linux/inetdevice.h:7,
from net/netfilter/xt_qtaguid.c:18:
net/netfilter/xt_qtaguid.c:1812:47: error: 'struct xt_action_param' has no member named 'hooknum'
1812 | MT_DEBUG("qtaguid[%d]: left %d\n", par->hooknum, res);
| ^~
include/linux/dynamic_debug.h:128:38: note: in definition of macro 'dynamic_pr_debug'
128 | ##__VA_ARGS__); \
| ^~~~~~~~~~~
net/netfilter/xt_qtaguid_internal.h:44:25: note: in expansion of macro 'pr_debug'
44 | pr_debug(__VA_ARGS__); \
| ^~~~~~~~
net/netfilter/xt_qtaguid_internal.h:52:23: note: in expansion of macro 'MSK_DEBUG'
52 | #define MT_DEBUG(...) MSK_DEBUG(MDEBUG_MASK, __VA_ARGS__)
| ^~~~~~~~~
net/netfilter/xt_qtaguid.c:1812:9: note: in expansion of macro 'MT_DEBUG'
1812 | MT_DEBUG("qtaguid[%d]: left %d\n", par->hooknum, res);
| ^~~~~~~~
In file included from include/linux/file.h:9,
from net/netfilter/xt_qtaguid.c:17:
net/netfilter/xt_qtaguid.c: In function 'prdebug_full_state_locked':
include/linux/compiler.h:25:39: warning: ignoring attribute 'section ("_ftrace_annotated_branch")' because it conflicts with previous 'section ("_ftrace_branch")' [-Wattributes]
25 | static struct ftrace_likely_data \
| ^~~~~~~~~~~~~~~~~~
include/linux/compiler.h:69:30: note: in definition of macro '__trace_if'
69 | ______r = !!(cond); \
| ^~~~
net/netfilter/xt_qtaguid.c:1827:9: note: in expansion of macro 'if'
1827 | if (!unlikely(qtaguid_debug_mask & DDEBUG_MASK))
| ^~
include/linux/compiler.h:48:26: note: in expansion of macro '__branch_check__'
48 | # define unlikely(x) (__branch_check__(x, 0, __builtin_constant_p(x)))
| ^~~~~~~~~~~~~~~~
net/netfilter/xt_qtaguid.c:1827:14: note: in expansion of macro 'unlikely'
1827 | if (!unlikely(qtaguid_debug_mask & DDEBUG_MASK))
| ^~~~~~~~
include/linux/compiler.h:64:25: note: previous declaration here
64 | ______f = { \
| ^~~~~~~
include/linux/compiler.h:56:23: note: in expansion of macro '__trace_if'
56 | #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
| ^~~~~~~~~~
net/netfilter/xt_qtaguid.c:1827:9: note: in expansion of macro 'if'
1827 | if (!unlikely(qtaguid_debug_mask & DDEBUG_MASK))
| ^~
>> net/netfilter/xt_qtaguid.c:1835:27: warning: function 'prdebug_full_state_locked' might be a candidate for 'gnu_printf' format attribute [-Wsuggest-attribute=format]
1835 | fmt_buff, args);
| ^~~~~~~~
In file included from include/linux/file.h:9,
from net/netfilter/xt_qtaguid.c:17:
include/linux/compiler.h:25:39: warning: ignoring attribute 'section ("_ftrace_annotated_branch")' because it conflicts with previous 'section ("_ftrace_branch")' [-Wattributes]
25 | static struct ftrace_likely_data \
| ^~~~~~~~~~~~~~~~~~
include/linux/compiler.h:69:30: note: in definition of macro '__trace_if'
69 | ______r = !!(cond); \
| ^~~~
include/linux/dynamic_debug.h:126:9: note: in expansion of macro 'if'
126 | if (DYNAMIC_DEBUG_BRANCH(descriptor)) \
| ^~
include/linux/compiler.h:45:26: note: in expansion of macro '__branch_check__'
45 | # define likely(x) (__branch_check__(x, 1, __builtin_constant_p(x)))
| ^~~~~~~~~~~~~~~~
include/linux/dynamic_debug.h:115:9: note: in expansion of macro 'likely'
115 | likely(descriptor.flags & _DPRINTK_FLAGS_PRINT)
| ^~~~~~
include/linux/dynamic_debug.h:126:13: note: in expansion of macro 'DYNAMIC_DEBUG_BRANCH'
126 | if (DYNAMIC_DEBUG_BRANCH(descriptor)) \
| ^~~~~~~~~~~~~~~~~~~~
include/linux/printk.h:340:9: note: in expansion of macro 'dynamic_pr_debug'
340 | dynamic_pr_debug(fmt, ##__VA_ARGS__)
| ^~~~~~~~~~~~~~~~
net/netfilter/xt_qtaguid.c:1837:9: note: in expansion of macro 'pr_debug'
1837 | pr_debug("%s", buff);
| ^~~~~~~~
include/linux/compiler.h:64:25: note: previous declaration here
64 | ______f = { \
| ^~~~~~~
include/linux/compiler.h:56:23: note: in expansion of macro '__trace_if'
56 | #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
| ^~~~~~~~~~
include/linux/dynamic_debug.h:126:9: note: in expansion of macro 'if'
126 | if (DYNAMIC_DEBUG_BRANCH(descriptor)) \
| ^~
include/linux/printk.h:340:9: note: in expansion of macro 'dynamic_pr_debug'
340 | dynamic_pr_debug(fmt, ##__VA_ARGS__)
| ^~~~~~~~~~~~~~~~
net/netfilter/xt_qtaguid.c:1837:9: note: in expansion of macro 'pr_debug'
1837 | pr_debug("%s", buff);
| ^~~~~~~~
include/linux/compiler.h:25:39: warning: ignoring attribute 'section ("_ftrace_annotated_branch")' because it conflicts with previous 'section ("_ftrace_branch")' [-Wattributes]
25 | static struct ftrace_likely_data \
| ^~~~~~~~~~~~~~~~~~
include/linux/compiler.h:69:30: note: in definition of macro '__trace_if'
69 | ______r = !!(cond); \
| ^~~~
include/linux/dynamic_debug.h:126:9: note: in expansion of macro 'if'
126 | if (DYNAMIC_DEBUG_BRANCH(descriptor)) \
| ^~
include/linux/compiler.h:45:26: note: in expansion of macro '__branch_check__'
45 | # define likely(x) (__branch_check__(x, 1, __builtin_constant_p(x)))
| ^~~~~~~~~~~~~~~~
include/linux/dynamic_debug.h:115:9: note: in expansion of macro 'likely'
115 | likely(descriptor.flags & _DPRINTK_FLAGS_PRINT)
| ^~~~~~
include/linux/dynamic_debug.h:126:13: note: in expansion of macro 'DYNAMIC_DEBUG_BRANCH'
126 | if (DYNAMIC_DEBUG_BRANCH(descriptor)) \
| ^~~~~~~~~~~~~~~~~~~~
include/linux/printk.h:340:9: note: in expansion of macro 'dynamic_pr_debug'
340 | dynamic_pr_debug(fmt, ##__VA_ARGS__)
| ^~~~~~~~~~~~~~~~
net/netfilter/xt_qtaguid.c:1853:9: note: in expansion of macro 'pr_debug'
1853 | pr_debug("qtaguid: %s(): }\n", __func__);
| ^~~~~~~~
include/linux/compiler.h:64:25: note: previous declaration here
64 | ______f = { \
| ^~~~~~~
include/linux/compiler.h:56:23: note: in expansion of macro '__trace_if'
56 | #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
| ^~~~~~~~~~
include/linux/dynamic_debug.h:126:9: note: in expansion of macro 'if'
126 | if (DYNAMIC_DEBUG_BRANCH(descriptor)) \
| ^~
include/linux/printk.h:340:9: note: in expansion of macro 'dynamic_pr_debug'
340 | dynamic_pr_debug(fmt, ##__VA_ARGS__)
| ^~~~~~~~~~~~~~~~
net/netfilter/xt_qtaguid.c:1853:9: note: in expansion of macro 'pr_debug'
1853 | pr_debug("qtaguid: %s(): }\n", __func__);
| ^~~~~~~~
net/netfilter/xt_qtaguid.c: In function 'qtaguid_ctrl_proc_start':
include/linux/compiler.h:25:39: warning: ignoring attribute 'section ("_ftrace_annotated_branch")' because it conflicts with previous 'section ("_ftrace_branch")' [-Wattributes]
25 | static struct ftrace_likely_data \
| ^~~~~~~~~~~~~~~~~~
include/linux/compiler.h:69:30: note: in definition of macro '__trace_if'
69 | ______r = !!(cond); \
| ^~~~
net/netfilter/xt_qtaguid.c:1895:9: note: in expansion of macro 'if'
1895 | if (unlikely(module_passive))
| ^~
include/linux/compiler.h:48:26: note: in expansion of macro '__branch_check__'
48 | # define unlikely(x) (__branch_check__(x, 0, __builtin_constant_p(x)))
| ^~~~~~~~~~~~~~~~
net/netfilter/xt_qtaguid.c:1895:13: note: in expansion of macro 'unlikely'
1895 | if (unlikely(module_passive))
| ^~~~~~~~
include/linux/compiler.h:64:25: note: previous declaration here
64 | ______f = { \
vim +1835 net/netfilter/xt_qtaguid.c
cead6bffca84b2 JP Abgrall 2011-06-20 1815
cead6bffca84b2 JP Abgrall 2011-06-20 1816 #ifdef DDEBUG
fe6bc43a5e8411 Chenbo Feng 2017-03-23 1817 /*
fe6bc43a5e8411 Chenbo Feng 2017-03-23 1818 * This function is not in xt_qtaguid_print.c because of locks visibility.
fe6bc43a5e8411 Chenbo Feng 2017-03-23 1819 * The lock of sock_tag_list must be aquired before calling this function
fe6bc43a5e8411 Chenbo Feng 2017-03-23 1820 */
fe6bc43a5e8411 Chenbo Feng 2017-03-23 1821 static void prdebug_full_state_locked(int indent_level, const char *fmt, ...)
cead6bffca84b2 JP Abgrall 2011-06-20 1822 {
cead6bffca84b2 JP Abgrall 2011-06-20 1823 va_list args;
cead6bffca84b2 JP Abgrall 2011-06-20 1824 char *fmt_buff;
cead6bffca84b2 JP Abgrall 2011-06-20 1825 char *buff;
cead6bffca84b2 JP Abgrall 2011-06-20 1826
cead6bffca84b2 JP Abgrall 2011-06-20 @1827 if (!unlikely(qtaguid_debug_mask & DDEBUG_MASK))
cead6bffca84b2 JP Abgrall 2011-06-20 1828 return;
cead6bffca84b2 JP Abgrall 2011-06-20 1829
cead6bffca84b2 JP Abgrall 2011-06-20 1830 fmt_buff = kasprintf(GFP_ATOMIC,
cead6bffca84b2 JP Abgrall 2011-06-20 1831 "qtaguid: %s(): %s {\n", __func__, fmt);
cead6bffca84b2 JP Abgrall 2011-06-20 1832 BUG_ON(!fmt_buff);
cead6bffca84b2 JP Abgrall 2011-06-20 1833 va_start(args, fmt);
cead6bffca84b2 JP Abgrall 2011-06-20 1834 buff = kvasprintf(GFP_ATOMIC,
cead6bffca84b2 JP Abgrall 2011-06-20 @1835 fmt_buff, args);
cead6bffca84b2 JP Abgrall 2011-06-20 1836 BUG_ON(!buff);
cead6bffca84b2 JP Abgrall 2011-06-20 1837 pr_debug("%s", buff);
cead6bffca84b2 JP Abgrall 2011-06-20 1838 kfree(fmt_buff);
cead6bffca84b2 JP Abgrall 2011-06-20 1839 kfree(buff);
cead6bffca84b2 JP Abgrall 2011-06-20 1840 va_end(args);
cead6bffca84b2 JP Abgrall 2011-06-20 1841
cead6bffca84b2 JP Abgrall 2011-06-20 1842 prdebug_sock_tag_tree(indent_level, &sock_tag_tree);
cead6bffca84b2 JP Abgrall 2011-06-20 1843
cead6bffca84b2 JP Abgrall 2011-06-20 1844 spin_lock_bh(&uid_tag_data_tree_lock);
cead6bffca84b2 JP Abgrall 2011-06-20 1845 prdebug_uid_tag_data_tree(indent_level, &uid_tag_data_tree);
cead6bffca84b2 JP Abgrall 2011-06-20 1846 prdebug_proc_qtu_data_tree(indent_level, &proc_qtu_data_tree);
cead6bffca84b2 JP Abgrall 2011-06-20 1847 spin_unlock_bh(&uid_tag_data_tree_lock);
cead6bffca84b2 JP Abgrall 2011-06-20 1848
cead6bffca84b2 JP Abgrall 2011-06-20 1849 spin_lock_bh(&iface_stat_list_lock);
cead6bffca84b2 JP Abgrall 2011-06-20 1850 prdebug_iface_stat_list(indent_level, &iface_stat_list);
cead6bffca84b2 JP Abgrall 2011-06-20 1851 spin_unlock_bh(&iface_stat_list_lock);
cead6bffca84b2 JP Abgrall 2011-06-20 1852
cead6bffca84b2 JP Abgrall 2011-06-20 1853 pr_debug("qtaguid: %s(): }\n", __func__);
cead6bffca84b2 JP Abgrall 2011-06-20 1854 }
cead6bffca84b2 JP Abgrall 2011-06-20 1855 #else
fe6bc43a5e8411 Chenbo Feng 2017-03-23 1856 static void prdebug_full_state_locked(int indent_level, const char *fmt, ...) {}
cead6bffca84b2 JP Abgrall 2011-06-20 1857 #endif
cead6bffca84b2 JP Abgrall 2011-06-20 1858
:::::: The code at line 1835 was first introduced by commit
:::::: cead6bffca84b298691c58da076022a82af7e6be ANDROID: netfilter: xt_qtaguid: add qtaguid matching module
:::::: TO: JP Abgrall <jpa@...gle.com>
:::::: CC: Guenter Roeck <groeck@...omium.org>
--
0-DAY CI Kernel Test Service
https://01.org/lkp
Powered by blists - more mailing lists