On Wed, Mar 11, 2020 at 1:05 PM Ananyev, Konstantin <[email protected]> wrote: > > From: Kevin Traynor <[email protected]> > > Sent: Wednesday, March 11, 2020 11:33 AM > > To: [email protected] > > Cc: Kevin Traynor <[email protected]>; [email protected]; Ananyev, > > Konstantin <[email protected]>; Nicolau, Radu > > <[email protected]>; Akhil Goyal <[email protected]> > > Subject: [PATCH v2 2/2] examples/ipsec-gw: fix gcc 10 maybe-uninitialized > > warning > > > > gcc 10.0.1 reports: > > > > ../examples/ipsec-secgw/ipsec_process.c: In function ‘ipsec_process’: > > ../examples/ipsec-secgw/ipsec_process.c:132:34: > > error: ‘grp.m’ may be used uninitialized in this function > > [-Werror=maybe-uninitialized] > > 132 | grp[n].cnt = pkts + i - grp[n].m; > > | ~~~~~~^~ > > > > This is a correct warning for the initial execution of the statement. > > However, it is the design of the loop that grp[0].cnt will later be > > written with the correct value using an initialized grp[0].m before it > > is used. > > > > In order to remove the warning, initialize grp[0].m for the initial and > > unused calculation of grp[0].cnt. > > > > Fixes: 3e5f4625dc17 ("examples/ipsec-secgw: make data-path to use IPsec > > library") > > Cc: [email protected] > > > > Suggested-by: Konstantin Ananyev <[email protected]> > > Signed-off-by: Kevin Traynor <[email protected]> > Acked-by: Konstantin Ananyev <[email protected]>
Applied, thanks. -- David Marchand
