On Mon, Jan 09, 2023 at 08:39:44PM +0100, Hans-Bernhard Bröker wrote:
> > Whatever the original intention was behind strncat, this seems like
> > a reasonable way to use strncat based on its behaviour and documentation.
>
> It is not.
>
> The size argument exists to enable the caller to make sure
Am 09.01.2023 um 19:58 schrieb Gavin Smith:
As I said in another message, it appears that everywhere strncat is
used, the length argument is used for the number of bytes to copy in
the source buffer, not for the size remaining in the destination buffer.
And that is at least superficially wrong
On Mon, Jan 09, 2023 at 07:35:42PM +0100, Hans-Bernhard Bröker wrote:
> It's not. It warns about a practice that has often been used to avoid
> another warning, while not actually achieving any improvement.
>
> In a nutshell, strcat() itself has long since been frowned upon as a highly
> likely s
Am 09.01.2023 um 18:44 schrieb Gavin Smith:
There have long been compiler warnings about the use of strncat in
install-info.c but there isn't any indication that the code is actually
incorrect. This compiler warning seems to be completely bogus.
It's not. It warns about a practice that has
On Mon, Jan 09, 2023 at 05:44:05PM +, Gavin Smith wrote:
> There have long been compiler warnings about the use of strncat in
> install-info.c but there isn't any indication that the code is actually
> incorrect. This compiler warning seems to be completely bogus. The
> real way the code coul
On Sat, Jan 07, 2023 at 04:53:38PM +0330, Faezeh Bidjarchian wrote:
> Hi
>
> I install the latest version of texinfo (texinfo-7.0.1) and found this
> warning:
>
> gcc -DHAVE_CONFIG_H -I. -I.. -I.. -I../gnulib/lib -I../gnulib/lib
> -DLOCALEDIR=\"/usr/local/share/locale\" -g -O2 -MT install-info
Hi
I install the latest version of texinfo (texinfo-7.0.1) and found this
warning:
gcc -DHAVE_CONFIG_H -I. -I.. -I.. -I../gnulib/lib -I../gnulib/lib
-DLOCALEDIR=\"/usr/local/share/locale\" -g -O2 -MT install-info.o -MD -MP
-MF .deps/install-info.Tpo -c -o install-info.o install-info.c
In file