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

Ben Pfaff 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 mailing list