Re: [ovs-dev] [PATCH] socket-util-unix: Avoid buffer read overrun in get_unix_name_len().

2016-09-15 Thread Ben Pfaff
On Thu, Sep 15, 2016 at 10:27:09AM -0700, Joe Stringer wrote: > On 14 September 2016 at 20:39, Ben Pfaff wrote: > > If the socket length does not include any of the bytes of the path, then > > the code should not read even the first byte of the path. > > > > Found by valgrind. > > > > CC: Thadeu L

Re: [ovs-dev] [PATCH] socket-util-unix: Avoid buffer read overrun in get_unix_name_len().

2016-09-15 Thread Joe Stringer
On 14 September 2016 at 20:39, Ben Pfaff wrote: > If the socket length does not include any of the bytes of the path, then > the code should not read even the first byte of the path. > > Found by valgrind. > > CC: Thadeu Lima de Souza Cascardo > Reported-by: Joe Stringer > Signed-off-by: Ben Pfa

Re: [ovs-dev] [PATCH] socket-util-unix: Avoid buffer read overrun in get_unix_name_len().

2016-09-15 Thread Thadeu Lima de Souza Cascardo
On Wed, Sep 14, 2016 at 08:39:03PM -0700, Ben Pfaff wrote: > If the socket length does not include any of the bytes of the path, then > the code should not read even the first byte of the path. > > Found by valgrind. > > CC: Thadeu Lima de Souza Cascardo > Reported-by: Joe Stringer > Signed-off

[ovs-dev] [PATCH] socket-util-unix: Avoid buffer read overrun in get_unix_name_len().

2016-09-14 Thread Ben Pfaff
If the socket length does not include any of the bytes of the path, then the code should not read even the first byte of the path. Found by valgrind. CC: Thadeu Lima de Souza Cascardo Reported-by: Joe Stringer Signed-off-by: Ben Pfaff --- lib/socket-util-unix.c | 4 ++-- 1 file changed, 2 ins