[ovs-dev] [PATCH] ovsdb-server: Fix memory leak.
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.
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