Re: [ovs-dev] [PATCH 6/6] netdev: Fix null pointer dereference reported by clang.

2017-07-11 Thread Ben Pfaff
On Mon, Jun 19, 2017 at 07:54:04PM +0100, Bhanuprakash Bodireddy wrote: > Clang reports that array access from 'dumps' variable result in null pointer > dereference. > > Signed-off-by: Bhanuprakash Bodireddy > --- > lib/netdev.c | 16 +--- > 1 file changed, 9 insertions(+), 7 deletio

Re: [ovs-dev] [PATCH 6/6] netdev: Fix null pointer dereference reported by clang.

2017-06-25 Thread Bodireddy, Bhanuprakash
Hi Mark, >> >>Clang reports that array access from 'dumps' variable result in null >>pointer dereference. >> >>Signed-off-by: Bhanuprakash Bodireddy >> > >Hi Bhanu, > >LGTM - I also compiled this with gcc, clang, and sparse without issue. >Checkpatch reports no obvious problems either. > >Acked-by:

Re: [ovs-dev] [PATCH 6/6] netdev: Fix null pointer dereference reported by clang.

2017-06-23 Thread Kavanagh, Mark B
>From: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev-boun...@openvswitch.org] >On Behalf Of >Bhanuprakash Bodireddy >Sent: Monday, June 19, 2017 7:54 PM >To: d...@openvswitch.org >Subject: [ovs-dev] [PATCH 6/6] netdev: Fix null pointer dereference reported >by clang.

[ovs-dev] [PATCH 6/6] netdev: Fix null pointer dereference reported by clang.

2017-06-19 Thread Bhanuprakash Bodireddy
Clang reports that array access from 'dumps' variable result in null pointer dereference. Signed-off-by: Bhanuprakash Bodireddy --- lib/netdev.c | 16 +--- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/lib/netdev.c b/lib/netdev.c index 001b7b3..336c141 100644 --- a/li