linux-kernel - Re: [PATCH] net/llc/llc_conn.c: fix 4 instances of -Wmissing-variable-declarations

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20230809154104.64963-1-kuniyu@amazon.com>
Date:   Wed, 9 Aug 2023 08:41:04 -0700
From:   Kuniyuki Iwashima <kuniyu@...zon.com>
To:     <horms@...nel.org>
CC:     <davem@...emloft.net>, <edumazet@...gle.com>, <kuba@...nel.org>,
        <kuniyu@...zon.com>, <linux-kernel@...r.kernel.org>,
        <lkp@...el.com>, <llvm@...ts.linux.dev>, <nathan@...nel.org>,
        <ndesaulniers@...gle.com>, <netdev@...r.kernel.org>,
        <pabeni@...hat.com>, <trix@...hat.com>
Subject: Re: [PATCH] net/llc/llc_conn.c: fix 4 instances of -Wmissing-variable-declarations

From: Simon Horman <horms@...nel.org>
Date: Wed, 9 Aug 2023 14:59:51 +0200
> + Kuniyuki Iwashima

Thanks Simon.

> 
> On Tue, Aug 08, 2023 at 09:43:09AM -0700, Nick Desaulniers wrote:
> > I'm looking to enable -Wmissing-variable-declarations behind W=1. 0day
> > bot spotted the following instances:
> > 
> >   net/llc/llc_conn.c:44:5: warning: no previous extern declaration for
> >   non-static variable 'sysctl_llc2_ack_timeout'
> >   [-Wmissing-variable-declarations]
> >   44 | int sysctl_llc2_ack_timeout = LLC2_ACK_TIME * HZ;
> >      |     ^
> >   net/llc/llc_conn.c:44:1: note: declare 'static' if the variable is not
> >   intended to be used outside of this translation unit
> >   44 | int sysctl_llc2_ack_timeout = LLC2_ACK_TIME * HZ;
> >      | ^
> >   net/llc/llc_conn.c:45:5: warning: no previous extern declaration for
> >   non-static variable 'sysctl_llc2_p_timeout'
> >   [-Wmissing-variable-declarations]
> >   45 | int sysctl_llc2_p_timeout = LLC2_P_TIME * HZ;
> >      |     ^
> >   net/llc/llc_conn.c:45:1: note: declare 'static' if the variable is not
> >   intended to be used outside of this translation unit
> >   45 | int sysctl_llc2_p_timeout = LLC2_P_TIME * HZ;
> >      | ^
> >   net/llc/llc_conn.c:46:5: warning: no previous extern declaration for
> >   non-static variable 'sysctl_llc2_rej_timeout'
> >   [-Wmissing-variable-declarations]
> >   46 | int sysctl_llc2_rej_timeout = LLC2_REJ_TIME * HZ;
> >      |     ^
> >   net/llc/llc_conn.c:46:1: note: declare 'static' if the variable is not
> >   intended to be used outside of this translation unit
> >   46 | int sysctl_llc2_rej_timeout = LLC2_REJ_TIME * HZ;
> >      | ^
> >   net/llc/llc_conn.c:47:5: warning: no previous extern declaration for
> >   non-static variable 'sysctl_llc2_busy_timeout'
> >   [-Wmissing-variable-declarations]
> >   47 | int sysctl_llc2_busy_timeout = LLC2_BUSY_TIME * HZ;
> >      |     ^
> >   net/llc/llc_conn.c:47:1: note: declare 'static' if the variable is not
> >   intended to be used outside of this translation unit
> >   47 | int sysctl_llc2_busy_timeout = LLC2_BUSY_TIME * HZ;
> >      | ^
> > 
> > These symbols are referenced by more than one translation unit, so make
> > include the correct header for their declarations. Finally, sort the
> > list of includes to help keep them tidy.
> > 
> > Reported-by: kernel test robot <lkp@...el.com>
> > Closes: https://lore.kernel.org/llvm/202308081000.tTL1ElTr-lkp@intel.com/
> > Signed-off-by: Nick Desaulniers <ndesaulniers@...gle.com>
> 
> Reviewed-by: Simon Horman <horms@...nel.org>

Reviewed-by: Kuniyuki Iwashima <kuniyu@...zon.com>

Thanks!

> 
> > ---
> >  net/llc/llc_conn.c | 11 ++++++-----
> >  1 file changed, 6 insertions(+), 5 deletions(-)
> > 
> > diff --git a/net/llc/llc_conn.c b/net/llc/llc_conn.c
> > index d037009ee10f..0a3f5e0bec00 100644
> > --- a/net/llc/llc_conn.c
> > +++ b/net/llc/llc_conn.c
> > @@ -14,14 +14,15 @@
> >  
> >  #include <linux/init.h>
> >  #include <linux/slab.h>
> > -#include <net/llc_sap.h>
> > -#include <net/llc_conn.h>
> > -#include <net/sock.h>
> > -#include <net/tcp_states.h>
> > -#include <net/llc_c_ev.h>
> > +#include <net/llc.h>
> >  #include <net/llc_c_ac.h>
> > +#include <net/llc_c_ev.h>
> >  #include <net/llc_c_st.h>
> > +#include <net/llc_conn.h>
> >  #include <net/llc_pdu.h>
> > +#include <net/llc_sap.h>
> > +#include <net/sock.h>
> > +#include <net/tcp_states.h>
> >  
> >  #if 0
> >  #define dprintk(args...) printk(KERN_DEBUG args)
> > 
> > ---
> > base-commit: 14f9643dc90adea074a0ffb7a17d337eafc6a5cc
> > change-id: 20230808-llc_static-de4dddcc64b4
> > 
> > Best regards,
> > -- 
> > Nick Desaulniers <ndesaulniers@...gle.com>