]> git.sur5r.net Git - i3/i3/commit
Replace strncpy call with memcpy when result is not NUL-terminated 3282/head
authorOrestis Floros <orestisf1993@gmail.com>
Tue, 8 May 2018 14:05:46 +0000 (17:05 +0300)
committerOrestis Floros <orestisf1993@gmail.com>
Tue, 8 May 2018 14:05:46 +0000 (17:05 +0300)
commitf4981f97bcd1c28fcc64f002ac89e5011870582f
treee8d3c0aa81f8177ad3b389c641333f7b0ce8ea20
parent1cdb1ab721deb9e2cead81d224f9f52390c73340
Replace strncpy call with memcpy when result is not NUL-terminated

This fixes a new warning from GCC 8.1, -Wstringop-truncation:
https://gcc.gnu.org/gcc-8/changes.html
https://gcc.gnu.org/onlinedocs/gcc-8.1.0/gcc/Warning-Options.html#index-Wstringop-truncation

Replacing with memcpy is what gcc suggests:
> As another example, the following call to strncpy results in copying
> to d just the characters preceding the terminating NUL, without
> appending the NUL to the end. Assuming the result of strncpy is
> necessarily a NUL-terminated string is a common mistake, and so the
> call is diagnosed. To avoid the warning when the result is not
> expected to be NUL-terminated, call memcpy instead.
>    void copy (char *d, const char *s)
>    {
>      strncpy (d, s, strlen (s));
>    }
i3bar/src/ipc.c
i3bar/src/xcb.c