Spinning up a new server to convert tradspool to CNFS and notice the following
compile time warning when building INN 2.7.1. I do not recall seeing this when
building 2.7.0 (but it's been awhile). Running "make tests" results in all
tests successfully passing.

Build environment:

[news@gatekeeper ~/src/inn-2.7.1]$ uname -ar
FreeBSD gatekeeper 13.2-RELEASE FreeBSD 13.2-RELEASE
releng/13.2-n254617-525ecfdad597 GENERIC amd64

[news@gatekeeper ~/src/inn-2.7.1]$ gcc --version
gcc (FreeBSD Ports Collection) 12.2.0
Copyright (C) 2022 Free Software Foundation, Inc.
This is free software; see the source for copying conditions. There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

Configure command:

../configure --mandir=/usr/local/man --prefix=/usr/local/news
--with-spool-dir=/usr/local/news/spool --with-log-dir=/var/log/news
--with-tmp-dir=/usr/local/news/tmp --sysconfdir=/usr/local/news/etc
--with-run-dir=/usr/local/news/run --with-db-dir=/usr/local/news/db
--datarootdir=/usr/local/news/share --with-perl --with-blacklist
--with-sqlite3=/usr/local --with-bdb --with-bdb-lib=/usr/local/lib/db18
--with-bdb-include=/usr/local/include/db18 --without-krb5 --enable-keywords
--enable-largefiles --with-python --with-openssl=/usr

Warning exhibited:

.../libtool --mode=compile gcc -g -O2 -fPIE -D_FORTIFY_SOURCE=2
-fstack-protector-strong -I../include -I. -I/usr/local/include/db18
-I/usr/local/include -c -o cnfs/cnfs.o cnfs/cnfs.c
libtool: compile: gcc -g -O2 -D_FORTIFY_SOURCE=2 -fstack-protector-strong
-I../include -I. -I/usr/local/include/db18 -I/usr/local/include -c cnfs/cnfs.c
-fPIC -DPIC -o cnfs/.libs/cnfs.o
In function 'CNFS_setcurrent',
inlined from 'cnfs_init' at cnfs/cnfs.c:1211:17:
cnfs/cnfs.c:814:13: warning: 'strncpy' specified bound 16 equals destination
size [-Wstringop-truncation]
814 | strncpy(cycbuff->metaname, metacycbuff->name, CNFSNASIZ);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Hi Jesse,

> Spinning up a new server to convert tradspool to CNFS and notice the following
> compile time warning when building INN 2.7.1. I do not recall seeing this when
> building 2.7.0 (but it's been awhile).
>
> Warning exhibited:
>
> cnfs/cnfs.c:814:13: warning: 'strncpy' specified bound 16 equals destination
> size [-Wstringop-truncation]
> 814 | strncpy(cycbuff->metaname, metacycbuff->name, CNFSNASIZ);
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

This specific warning should have been ignored, and not triggered by
your gcc 12.2.0. At least, there is no reason why it was not seen in
INN 2.7.0, which similarly has it ignored. (Unless of course __GNUC__
is no longer set in this version from FreeBSD.)

#if __GNUC__ > 7
# pragma GCC diagnostic ignored "-Wstringop-truncation"
#endif
/* Don't use sprintf() or strlcat() directly...
* The terminating '\0' causes grief. */
strncpy(cycbuff->metaname, metacycbuff->name, CNFSNASIZ);

The code works fine so you should not worry about it.

There are some diagnostics ignored sometimes in the source code, but
that's OK. (A rewrite of such parts of code is welcome of course to get
rid of the warning. In this case, probably switching to use memcpy().)

--
Julien ÉLIE

« Il n'y a que le premier pas qui coûte. » (Mme du Deffand)