[ovs-dev] [coverity2 0/7] More Coverity Fixes

Justin Pettit jpettit at nicira.com
Wed Feb 23 19:11:31 UTC 2011


On Feb 23, 2011, at 9:30 AM, Ben Pfaff wrote:

> On Tue, Feb 22, 2011 at 10:18 PM, Justin Pettit <jpettit at nicira.com> wrote:
>> Here are some additional fixes for issues that Coverity flagged.  With
>> this series, the remaining issues Coverity marks as "High Impact" that
>> appear to me to be real are addressed.*  This also addresses most of the
>> issues that Coverity marks as "Medium Impact" that looked real to me,
>> but there are a few remaining ones that I will take another crack at
>> later (unless someone beats me to it).
> 
> Thanks a lot for doing this.
> 
> I have one suggestion for any possible future series: order it in rough
> descending order of importance, with serious bugs at the beginning and
> e.g. memory leaks in test programs at the end.  (I try to do this when
> I compose patch series, too.)


Sounds good.  I'll do that in the future.

Thanks for the reviews.  I pushed.

--Justin






More information about the dev mailing list