[ovs-dev] [PATCH] ovsdb-server: Fix memory leak.
blp at nicira.com
Thu Feb 17 21:15:01 UTC 2011
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