Re: [PATCH 4/4] lwip: Fix a buffer overflow and clear compiler warnings

2018-08-14 Thread Samuel Thibault
Joan Lledó, le mar. 14 août 2018 18:17:28 +0200, a ecrit: > Missatge de Samuel Thibault del dia dt., 14 > d’ag. 2018 a les 9:34: > > > - strncpy (in->dev_name, arg, DEV_NAME_LEN); > > > + strncpy (in->dev_name, arg, sizeof(in->dev_name)-1); > > > > Mmm, but if arg is longer than the give

Re: [PATCH 4/4] lwip: Fix a buffer overflow and clear compiler warnings

2018-08-14 Thread Joan Lledó
Missatge de Samuel Thibault del dia dt., 14 d’ag. 2018 a les 9:34: > > - strncpy (in->dev_name, arg, DEV_NAME_LEN); > > + strncpy (in->dev_name, arg, sizeof(in->dev_name)-1); > > Mmm, but if arg is longer than the given size and doesn't contain a \0, > in->dev_name will not contain one e

Re: [PATCH 4/4] lwip: Fix a buffer overflow and clear compiler warnings

2018-08-14 Thread Samuel Thibault
Joan Lledó, le sam. 11 août 2018 18:17:10 +0200, a ecrit: > GCC 8 new warning -Wstringop-truncation detected some buffer overflows. > > * lwip/options.c (parse_opt): Fix the buffer overflow. > * lwip/pfinet-ops.c (dev_ifconf): Likewise. > --- > lwip/options.c| 2 +- > lwip/pfinet-ops.c | 2 +-