[ovs-dev] [coverity2 0/7] More Coverity Fixes
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.
More information about the dev