[ovs-dev] [PATCH] ovsdb-server: Fix memory leak.

Ethan Jackson ethan at nicira.com
Thu Feb 17 21:18:45 UTC 2011


ok, I added peter to the commit message.  I'll go ahead and merge.

Ethan

On Thu, Feb 17, 2011 at 1:15 PM, Ben Pfaff <blp at nicira.com> wrote:
> On Thu, Feb 17, 2011 at 01:07:49PM -0800, Ethan Jackson wrote:
>> > I don't think that the change to main() is required, because I think
>> > that the 'data' member inside this shash is always NULL.
>>
>> I would tend to think we should leave it as free_data in main because
>> it's that way everywhere else so it sets a precedent.  Seems like
>> defensive coding.  I'm fine with switching it back though if you think
>> it matters.
>
> I don't care.
>
> Eventually I'm going to implement a "sset" that's just a set of
> strings for situations like this.
>




More information about the dev mailing list