From: Phil Sutter
> Sent: 17 August 2017 18:09
> To: Stephen Hemminger
> Cc: [email protected]
> Subject: [iproute PATCH v2 1/7] ipntable: Make sure filter.name is
> NULL-terminated
>
> Signed-off-by: Phil Sutter <[email protected]>
> ---
> ip/ipntable.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/ip/ipntable.c b/ip/ipntable.c
> index 879626ee4f491..7be1f04d33d90 100644
> --- a/ip/ipntable.c
> +++ b/ip/ipntable.c
> @@ -633,7 +633,8 @@ static int ipntable_show(int argc, char **argv)
> } else if (strcmp(*argv, "name") == 0) {
> NEXT_ARG();
>
> - strncpy(filter.name, *argv, sizeof(filter.name));
> + strncpy(filter.name, *argv, sizeof(filter.name) - 1);
> + filter.name[sizeof(filter.name) - 1] = '\0';
Why not check for overflow instead?
if (filter.name[sizeof(filter.name) - 1])
usage("filer name too long");
David
