Re: [PATCH v6 1/2] audit: add audit_log_nf_skb helper function

Hi Ricardo,

kernel test robot noticed the following build warnings:

[auto build test WARNING on pcmoore-audit/next]
[also build test WARNING on netfilter-nf/main nf-next/master linus/master v6.18-rc5 next-20251113]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Ricardo-Robaina/audit-include-source-and-destination-ports-to-NETFILTER_PKT/20251113-223721
base:   https://git.kernel.org/pub/scm/linux/kernel/git/pcmoore/audit.git next
patch link:    https://lore.kernel.org/r/589b485078a65c766bcdee2fd9881c540813f8c5.1763036807.git.rrobaina%40redhat.com
patch subject: [PATCH v6 1/2] audit: add audit_log_nf_skb helper function
config: x86_64-rhel-9.4 (https://download.01.org/0day-ci/archive/20251114/[email protected]/config)
compiler: gcc-14 (Debian 14.2.0-19) 14.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20251114/[email protected]/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 <[email protected]>
| Closes: https://lore.kernel.org/oe-kbuild-all/[email protected]/

All warnings (new ones prefixed by >>):

   net/netfilter/nft_log.c: In function 'nft_log_eval_audit':
>> net/netfilter/nft_log.c:33:13: warning: unused variable 'fam' [-Wunused-variable]
      33 |         int fam = -1;
         |             ^~~


vim +/fam +33 net/netfilter/nft_log.c

96518518cc417bb Patrick McHardy 2013-10-14  28  
1a893b44de45288 Phil Sutter     2018-05-30  29  static void nft_log_eval_audit(const struct nft_pktinfo *pkt)
1a893b44de45288 Phil Sutter     2018-05-30  30  {
1a893b44de45288 Phil Sutter     2018-05-30  31  	struct sk_buff *skb = pkt->skb;
1a893b44de45288 Phil Sutter     2018-05-30  32  	struct audit_buffer *ab;
1a893b44de45288 Phil Sutter     2018-05-30 @33  	int fam = -1;
1a893b44de45288 Phil Sutter     2018-05-30  34  
1a893b44de45288 Phil Sutter     2018-05-30  35  	if (!audit_enabled)
1a893b44de45288 Phil Sutter     2018-05-30  36  		return;
1a893b44de45288 Phil Sutter     2018-05-30  37  
1a893b44de45288 Phil Sutter     2018-05-30  38  	ab = audit_log_start(NULL, GFP_ATOMIC, AUDIT_NETFILTER_PKT);
1a893b44de45288 Phil Sutter     2018-05-30  39  	if (!ab)
1a893b44de45288 Phil Sutter     2018-05-30  40  		return;
1a893b44de45288 Phil Sutter     2018-05-30  41  
1a893b44de45288 Phil Sutter     2018-05-30  42  	audit_log_format(ab, "mark=%#x", skb->mark);
1a893b44de45288 Phil Sutter     2018-05-30  43  
832662a8b1d3d70 Ricardo Robaina 2025-11-13  44  	audit_log_nf_skb(ab, skb, nft_pf(pkt));
1a893b44de45288 Phil Sutter     2018-05-30  45  
1a893b44de45288 Phil Sutter     2018-05-30  46  	audit_log_end(ab);
1a893b44de45288 Phil Sutter     2018-05-30  47  }
1a893b44de45288 Phil Sutter     2018-05-30  48  

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