Ubuntu 24.04 rte_pcapng.c warning

Ubuntu 24.04 rte_pcapng.c warning

[Cody Cheng]

[-- Attachment #1: Type: text/plain, Size: 366 bytes --]

Hi,

We have added the Ubuntu 24.04 environment to the Community Lab! Due to a
warning in rte_pcapng.c, we have temporarily disabled Werror. Once this
warning has been fixed, we will re-enable Werror. I have attached the logs
for the warning error below.

 log.txt
<https://drive.google.com/file/d/1qnBcQWg9AHG8RksjGjv8QaqTqw6yC3Gc/view?usp=drive_web>

Thanks,
Cody

[-- Attachment #2: Type: text/html, Size: 1109 bytes --]

Re: Ubuntu 24.04 rte_pcapng.c warning

[Cody Cheng]

[-- Attachment #1: Type: text/plain, Size: 4312 bytes --]

Here is the warning:

FAILED: lib/76b5a35@@rte_pcapng@sta/pcapng_rte_pcapng.c.o
ccache cc -Ilib/76b5a35@@rte_pcapng@sta -Ilib -I../lib -Ilib/pcapng
-I../lib/pcapng -I. -I../ -Iconfig -I../config -Ilib/eal/include
-I../lib/eal/include -Ilib/eal/linux/include
-I../lib/eal/linux/include -Ilib/eal/x86/include
-I../lib/eal/x86/include -Ilib/eal/common -I../lib/eal/common
-Ilib/eal -I../lib/eal -Ilib/kvargs -I../lib/kvargs -Ilib/log
-I../lib/log -Ilib/telemetry/../metrics -I../lib/telemetry/../metrics
-Ilib/telemetry -I../lib/telemetry -Ilib/ethdev -I../lib/ethdev
-Ilib/net -I../lib/net -Ilib/mbuf -I../lib/mbuf -Ilib/mempool
-I../lib/mempool -Ilib/ring -I../lib/ring -Ilib/meter -I../lib/meter
-fdiagnostics-color=always -pipe -D_FILE_OFFSET_BITS=64 -Wall
-Winvalid-pch -Wextra -Werror -std=c11 -O3 -include rte_config.h
-Wcast-qual -Wdeprecated -Wformat -Wformat-nonliteral
-Wformat-security -Wmissing-declarations -Wmissing-prototypes
-Wnested-externs -Wold-style-definition -Wpointer-arith -Wsign-compare
-Wstrict-prototypes -Wundef -Wwrite-strings
-Wno-address-of-packed-member -Wno-packed-not-aligned
-Wno-missing-field-initializers -Wno-zero-length-bounds -D_GNU_SOURCE
-fPIC -march=native -mrtm -DALLOW_EXPERIMENTAL_API
-DALLOW_INTERNAL_API -Wno-format-truncation
-DRTE_LOG_DEFAULT_LOGTYPE=lib.pcapng -MD -MQ
'lib/76b5a35@@rte_pcapng@sta/pcapng_rte_pcapng.c.o' -MF
'lib/76b5a35@@rte_pcapng@sta/pcapng_rte_pcapng.c.o.d' -o
'lib/76b5a35@@rte_pcapng@sta/pcapng_rte_pcapng.c.o' -c
../lib/pcapng/rte_pcapng.c
In file included from /usr/include/string.h:548,
                 from ../lib/pcapng/rte_pcapng.c:9:
In function ‘memcpy’,
    inlined from ‘pcapng_add_option’ at ../lib/pcapng/rte_pcapng.c:131:2,
    inlined from ‘pcapng_section_block’ at ../lib/pcapng/rte_pcapng.c:192:8:
/usr/include/x86_64-linux-gnu/bits/string_fortified.h:29:10: error:
argument 2 null where non-null expected [-Werror=nonnull]
   29 |   return __builtin___memcpy_chk (__dest, __src, __len,
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   30 |                                  __glibc_objsize0 (__dest));
      |                                  ~~~~~~~~~~~~~~~~~~~~~~~~~~
/usr/include/x86_64-linux-gnu/bits/string_fortified.h:29:10: note: in
a call to built-in function ‘__builtin___memcpy_chk’
In function ‘memcpy’,
    inlined from ‘pcapng_add_option’ at ../lib/pcapng/rte_pcapng.c:131:2,
    inlined from ‘rte_pcapng_add_interface’ at ../lib/pcapng/rte_pcapng.c:308:8:
/usr/include/x86_64-linux-gnu/bits/string_fortified.h:29:10: error:
argument 2 null where non-null expected [-Werror=nonnull]
   29 |   return __builtin___memcpy_chk (__dest, __src, __len,
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   30 |                                  __glibc_objsize0 (__dest));
      |                                  ~~~~~~~~~~~~~~~~~~~~~~~~~~
/usr/include/x86_64-linux-gnu/bits/string_fortified.h:29:10: note: in
a call to built-in function ‘__builtin___memcpy_chk’
In function ‘memcpy’,
    inlined from ‘pcapng_add_option’ at ../lib/pcapng/rte_pcapng.c:131:2,
    inlined from ‘rte_pcapng_write_stats’ at ../lib/pcapng/rte_pcapng.c:371:9:
/usr/include/x86_64-linux-gnu/bits/string_fortified.h:29:10: error:
argument 2 null where non-null expected [-Werror=nonnull]
   29 |   return __builtin___memcpy_chk (__dest, __src, __len,
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   30 |                                  __glibc_objsize0 (__dest));
      |                                  ~~~~~~~~~~~~~~~~~~~~~~~~~~
/usr/include/x86_64-linux-gnu/bits/string_fortified.h:29:10: note: in
a call to built-in function ‘__builtin___memcpy_chk’
cc1: all warnings being treated as errors


On Tue, May 28, 2024 at 12:32 PM Cody Cheng <ccheng@iol.unh.edu> wrote:

> Hi,
>
> We have added the Ubuntu 24.04 environment to the Community Lab! Due to a
> warning in rte_pcapng.c, we have temporarily disabled Werror. Once this
> warning has been fixed, we will re-enable Werror. I have attached the logs
> for the warning error below.
>
>  log.txt
> <https://drive.google.com/file/d/1qnBcQWg9AHG8RksjGjv8QaqTqw6yC3Gc/view?usp=drive_web>
>
> Thanks,
> Cody
>

[-- Attachment #2: Type: text/html, Size: 5531 bytes --]

Re: Ubuntu 24.04 rte_pcapng.c warning

[Thomas Monjalon]

Hello,

There was a patch pending for this issue.
It has been merged quickly because it looks urgent,
but I would have preferred a test of the patch from you.
Next time ;)

Thanks


28/05/2024 20:28, Cody Cheng:
> Here is the warning:
> 
> FAILED: lib/76b5a35@@rte_pcapng@sta/pcapng_rte_pcapng.c.o
> ccache cc -Ilib/76b5a35@@rte_pcapng@sta -Ilib -I../lib -Ilib/pcapng
> -I../lib/pcapng -I. -I../ -Iconfig -I../config -Ilib/eal/include
> -I../lib/eal/include -Ilib/eal/linux/include
> -I../lib/eal/linux/include -Ilib/eal/x86/include
> -I../lib/eal/x86/include -Ilib/eal/common -I../lib/eal/common
> -Ilib/eal -I../lib/eal -Ilib/kvargs -I../lib/kvargs -Ilib/log
> -I../lib/log -Ilib/telemetry/../metrics -I../lib/telemetry/../metrics
> -Ilib/telemetry -I../lib/telemetry -Ilib/ethdev -I../lib/ethdev
> -Ilib/net -I../lib/net -Ilib/mbuf -I../lib/mbuf -Ilib/mempool
> -I../lib/mempool -Ilib/ring -I../lib/ring -Ilib/meter -I../lib/meter
> -fdiagnostics-color=always -pipe -D_FILE_OFFSET_BITS=64 -Wall
> -Winvalid-pch -Wextra -Werror -std=c11 -O3 -include rte_config.h
> -Wcast-qual -Wdeprecated -Wformat -Wformat-nonliteral
> -Wformat-security -Wmissing-declarations -Wmissing-prototypes
> -Wnested-externs -Wold-style-definition -Wpointer-arith -Wsign-compare
> -Wstrict-prototypes -Wundef -Wwrite-strings
> -Wno-address-of-packed-member -Wno-packed-not-aligned
> -Wno-missing-field-initializers -Wno-zero-length-bounds -D_GNU_SOURCE
> -fPIC -march=native -mrtm -DALLOW_EXPERIMENTAL_API
> -DALLOW_INTERNAL_API -Wno-format-truncation
> -DRTE_LOG_DEFAULT_LOGTYPE=lib.pcapng -MD -MQ
> 'lib/76b5a35@@rte_pcapng@sta/pcapng_rte_pcapng.c.o' -MF
> 'lib/76b5a35@@rte_pcapng@sta/pcapng_rte_pcapng.c.o.d' -o
> 'lib/76b5a35@@rte_pcapng@sta/pcapng_rte_pcapng.c.o' -c
> ../lib/pcapng/rte_pcapng.c
> In file included from /usr/include/string.h:548,
>                  from ../lib/pcapng/rte_pcapng.c:9:
> In function ‘memcpy’,
>     inlined from ‘pcapng_add_option’ at ../lib/pcapng/rte_pcapng.c:131:2,
>     inlined from ‘pcapng_section_block’ at ../lib/pcapng/rte_pcapng.c:192:8:
> /usr/include/x86_64-linux-gnu/bits/string_fortified.h:29:10: error:
> argument 2 null where non-null expected [-Werror=nonnull]
>    29 |   return __builtin___memcpy_chk (__dest, __src, __len,
>       |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>    30 |                                  __glibc_objsize0 (__dest));
>       |                                  ~~~~~~~~~~~~~~~~~~~~~~~~~~
> /usr/include/x86_64-linux-gnu/bits/string_fortified.h:29:10: note: in
> a call to built-in function ‘__builtin___memcpy_chk’
> In function ‘memcpy’,
>     inlined from ‘pcapng_add_option’ at ../lib/pcapng/rte_pcapng.c:131:2,
>     inlined from ‘rte_pcapng_add_interface’ at ../lib/pcapng/rte_pcapng.c:308:8:
> /usr/include/x86_64-linux-gnu/bits/string_fortified.h:29:10: error:
> argument 2 null where non-null expected [-Werror=nonnull]
>    29 |   return __builtin___memcpy_chk (__dest, __src, __len,
>       |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>    30 |                                  __glibc_objsize0 (__dest));
>       |                                  ~~~~~~~~~~~~~~~~~~~~~~~~~~
> /usr/include/x86_64-linux-gnu/bits/string_fortified.h:29:10: note: in
> a call to built-in function ‘__builtin___memcpy_chk’
> In function ‘memcpy’,
>     inlined from ‘pcapng_add_option’ at ../lib/pcapng/rte_pcapng.c:131:2,
>     inlined from ‘rte_pcapng_write_stats’ at ../lib/pcapng/rte_pcapng.c:371:9:
> /usr/include/x86_64-linux-gnu/bits/string_fortified.h:29:10: error:
> argument 2 null where non-null expected [-Werror=nonnull]
>    29 |   return __builtin___memcpy_chk (__dest, __src, __len,
>       |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>    30 |                                  __glibc_objsize0 (__dest));
>       |                                  ~~~~~~~~~~~~~~~~~~~~~~~~~~
> /usr/include/x86_64-linux-gnu/bits/string_fortified.h:29:10: note: in
> a call to built-in function ‘__builtin___memcpy_chk’
> cc1: all warnings being treated as errors
> 
> 
> On Tue, May 28, 2024 at 12:32 PM Cody Cheng <ccheng@iol.unh.edu> wrote:
> 
> > Hi,
> >
> > We have added the Ubuntu 24.04 environment to the Community Lab! Due to a
> > warning in rte_pcapng.c, we have temporarily disabled Werror. Once this
> > warning has been fixed, we will re-enable Werror. I have attached the logs
> > for the warning error below.
> >
> >  log.txt
> > <https://drive.google.com/file/d/1qnBcQWg9AHG8RksjGjv8QaqTqw6yC3Gc/view?usp=drive_web>
> >
> > Thanks,
> > Cody
> >
> 

Re: Ubuntu 24.04 rte_pcapng.c warning

[Cody Cheng]

[-- Attachment #1: Type: text/plain, Size: 5091 bytes --]

Got it!

I've re-enabled Werror now that the patch has been merged.

Thanks,
Cody

On Tue, May 28, 2024 at 5:41 PM Thomas Monjalon <thomas@monjalon.net> wrote:

> Hello,
>
> There was a patch pending for this issue.
> It has been merged quickly because it looks urgent,
> but I would have preferred a test of the patch from you.
> Next time ;)
>
> Thanks
>
>
> 28/05/2024 20:28, Cody Cheng:
> > Here is the warning:
> >
> > FAILED: lib/76b5a35@@rte_pcapng@sta/pcapng_rte_pcapng.c.o
> > ccache cc -Ilib/76b5a35@@rte_pcapng@sta -Ilib -I../lib -Ilib/pcapng
> > -I../lib/pcapng -I. -I../ -Iconfig -I../config -Ilib/eal/include
> > -I../lib/eal/include -Ilib/eal/linux/include
> > -I../lib/eal/linux/include -Ilib/eal/x86/include
> > -I../lib/eal/x86/include -Ilib/eal/common -I../lib/eal/common
> > -Ilib/eal -I../lib/eal -Ilib/kvargs -I../lib/kvargs -Ilib/log
> > -I../lib/log -Ilib/telemetry/../metrics -I../lib/telemetry/../metrics
> > -Ilib/telemetry -I../lib/telemetry -Ilib/ethdev -I../lib/ethdev
> > -Ilib/net -I../lib/net -Ilib/mbuf -I../lib/mbuf -Ilib/mempool
> > -I../lib/mempool -Ilib/ring -I../lib/ring -Ilib/meter -I../lib/meter
> > -fdiagnostics-color=always -pipe -D_FILE_OFFSET_BITS=64 -Wall
> > -Winvalid-pch -Wextra -Werror -std=c11 -O3 -include rte_config.h
> > -Wcast-qual -Wdeprecated -Wformat -Wformat-nonliteral
> > -Wformat-security -Wmissing-declarations -Wmissing-prototypes
> > -Wnested-externs -Wold-style-definition -Wpointer-arith -Wsign-compare
> > -Wstrict-prototypes -Wundef -Wwrite-strings
> > -Wno-address-of-packed-member -Wno-packed-not-aligned
> > -Wno-missing-field-initializers -Wno-zero-length-bounds -D_GNU_SOURCE
> > -fPIC -march=native -mrtm -DALLOW_EXPERIMENTAL_API
> > -DALLOW_INTERNAL_API -Wno-format-truncation
> > -DRTE_LOG_DEFAULT_LOGTYPE=lib.pcapng -MD -MQ
> > 'lib/76b5a35@@rte_pcapng@sta/pcapng_rte_pcapng.c.o' -MF
> > 'lib/76b5a35@@rte_pcapng@sta/pcapng_rte_pcapng.c.o.d' -o
> > 'lib/76b5a35@@rte_pcapng@sta/pcapng_rte_pcapng.c.o' -c
> > ../lib/pcapng/rte_pcapng.c
> > In file included from /usr/include/string.h:548,
> >                  from ../lib/pcapng/rte_pcapng.c:9:
> > In function ‘memcpy’,
> >     inlined from ‘pcapng_add_option’ at ../lib/pcapng/rte_pcapng.c:131:2,
> >     inlined from ‘pcapng_section_block’ at
> ../lib/pcapng/rte_pcapng.c:192:8:
> > /usr/include/x86_64-linux-gnu/bits/string_fortified.h:29:10: error:
> > argument 2 null where non-null expected [-Werror=nonnull]
> >    29 |   return __builtin___memcpy_chk (__dest, __src, __len,
> >       |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> >    30 |                                  __glibc_objsize0 (__dest));
> >       |                                  ~~~~~~~~~~~~~~~~~~~~~~~~~~
> > /usr/include/x86_64-linux-gnu/bits/string_fortified.h:29:10: note: in
> > a call to built-in function ‘__builtin___memcpy_chk’
> > In function ‘memcpy’,
> >     inlined from ‘pcapng_add_option’ at ../lib/pcapng/rte_pcapng.c:131:2,
> >     inlined from ‘rte_pcapng_add_interface’ at
> ../lib/pcapng/rte_pcapng.c:308:8:
> > /usr/include/x86_64-linux-gnu/bits/string_fortified.h:29:10: error:
> > argument 2 null where non-null expected [-Werror=nonnull]
> >    29 |   return __builtin___memcpy_chk (__dest, __src, __len,
> >       |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> >    30 |                                  __glibc_objsize0 (__dest));
> >       |                                  ~~~~~~~~~~~~~~~~~~~~~~~~~~
> > /usr/include/x86_64-linux-gnu/bits/string_fortified.h:29:10: note: in
> > a call to built-in function ‘__builtin___memcpy_chk’
> > In function ‘memcpy’,
> >     inlined from ‘pcapng_add_option’ at ../lib/pcapng/rte_pcapng.c:131:2,
> >     inlined from ‘rte_pcapng_write_stats’ at
> ../lib/pcapng/rte_pcapng.c:371:9:
> > /usr/include/x86_64-linux-gnu/bits/string_fortified.h:29:10: error:
> > argument 2 null where non-null expected [-Werror=nonnull]
> >    29 |   return __builtin___memcpy_chk (__dest, __src, __len,
> >       |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> >    30 |                                  __glibc_objsize0 (__dest));
> >       |                                  ~~~~~~~~~~~~~~~~~~~~~~~~~~
> > /usr/include/x86_64-linux-gnu/bits/string_fortified.h:29:10: note: in
> > a call to built-in function ‘__builtin___memcpy_chk’
> > cc1: all warnings being treated as errors
> >
> >
> > On Tue, May 28, 2024 at 12:32 PM Cody Cheng <ccheng@iol.unh.edu> wrote:
> >
> > > Hi,
> > >
> > > We have added the Ubuntu 24.04 environment to the Community Lab! Due
> to a
> > > warning in rte_pcapng.c, we have temporarily disabled Werror. Once this
> > > warning has been fixed, we will re-enable Werror. I have attached the
> logs
> > > for the warning error below.
> > >
> > >  log.txt
> > > <
> https://drive.google.com/file/d/1qnBcQWg9AHG8RksjGjv8QaqTqw6yC3Gc/view?usp=drive_web
> >
> > >
> > > Thanks,
> > > Cody
> > >
> >
>
>
>
>
>
>

[-- Attachment #2: Type: text/html, Size: 6294 bytes --]