Re: [ovs-dev] [coverity3 01/13] bridge: Ethernet address is 6 bytes, not 4 or 8.

2011-02-23 Thread Ben Pfaff
Thanks, I pushed this one. On Wed, Feb 23, 2011 at 01:34:36PM -0800, Ethan Jackson wrote: > Ah oops, that's my bug. Thanks for fixing it, looks good. Go ahead and push. > > On Wed, Feb 23, 2011 at 1:24 PM, Ben Pfaff wrote: > > 'ea' here is a function parameter declared as an array, so "sizeof e

Re: [ovs-dev] [coverity3 01/13] bridge: Ethernet address is 6 bytes, not 4 or 8.

2011-02-23 Thread Ethan Jackson
Ah oops, that's my bug. Thanks for fixing it, looks good. Go ahead and push. On Wed, Feb 23, 2011 at 1:24 PM, Ben Pfaff wrote: > 'ea' here is a function parameter declared as an array, so "sizeof ea" is > sizeof(uint8_t *), which is either 4 or 8. > > Coverity #10689, 10735. > --- >  vswitchd/br

[ovs-dev] [coverity3 01/13] bridge: Ethernet address is 6 bytes, not 4 or 8.

2011-02-23 Thread Ben Pfaff
'ea' here is a function parameter declared as an array, so "sizeof ea" is sizeof(uint8_t *), which is either 4 or 8. Coverity #10689, 10735. --- vswitchd/bridge.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/vswitchd/bridge.c b/vswitchd/bridge.c index c2c4579..a448e05