Re: [PATCH v4] net: don't call strlen on non-terminated string in dev_set_alias()

2017-06-06 Thread David Miller
From: Florian Westphal Date: Tue, 6 Jun 2017 23:57:35 +0200 > David Miller wrote: >> From: Alexander Potapenko >> Date: Tue, 6 Jun 2017 15:56:54 +0200 >> >> > KMSAN reported a use of uninitialized memory in dev_set_alias(), >> > which was caused by calling strlcpy() (which in turn called strl

Re: [PATCH v4] net: don't call strlen on non-terminated string in dev_set_alias()

2017-06-06 Thread Florian Westphal
David Miller wrote: > From: Alexander Potapenko > Date: Tue, 6 Jun 2017 15:56:54 +0200 > > > KMSAN reported a use of uninitialized memory in dev_set_alias(), > > which was caused by calling strlcpy() (which in turn called strlen()) > > on the user-supplied non-terminated string. > > > > Signed

Re: [PATCH v4] net: don't call strlen on non-terminated string in dev_set_alias()

2017-06-06 Thread Alexander Potapenko
On Tue, Jun 6, 2017 at 10:36 PM, David Miller wrote: > From: Alexander Potapenko > Date: Tue, 6 Jun 2017 15:56:54 +0200 > >> KMSAN reported a use of uninitialized memory in dev_set_alias(), >> which was caused by calling strlcpy() (which in turn called strlen()) >> on the user-supplied non-termi

Re: [PATCH v4] net: don't call strlen on non-terminated string in dev_set_alias()

2017-06-06 Thread David Miller
From: Alexander Potapenko Date: Tue, 6 Jun 2017 15:56:54 +0200 > KMSAN reported a use of uninitialized memory in dev_set_alias(), > which was caused by calling strlcpy() (which in turn called strlen()) > on the user-supplied non-terminated string. > > Signed-off-by: Alexander Potapenko We sho

[PATCH v4] net: don't call strlen on non-terminated string in dev_set_alias()

2017-06-06 Thread Alexander Potapenko
KMSAN reported a use of uninitialized memory in dev_set_alias(), which was caused by calling strlcpy() (which in turn called strlen()) on the user-supplied non-terminated string. Signed-off-by: Alexander Potapenko --- v4: dropped the comment at all, as suggested by Yury Norov v3: removed the mult