Re: [ovs-dev] [PATCH] ofproto-dpif: Fix memory leak.

2011-11-30 Thread Pravin Shelar
On Wed, Nov 30, 2011 at 12:21 PM, Ben Pfaff wrote: > On Wed, Nov 30, 2011 at 12:16:08PM -0800, Pravin B Shelar wrote: >> Following patch fixes memory leak in case there is ODP_FIT_ERROR >> on flow key. > > The changes that match the summary look good to me.  Thank you, those > are good catches. >

Re: [ovs-dev] [PATCH] ofproto-dpif: Fix memory leak.

2011-11-30 Thread Ben Pfaff
On Wed, Nov 30, 2011 at 12:16:08PM -0800, Pravin B Shelar wrote: > Following patch fixes memory leak in case there is ODP_FIT_ERROR > on flow key. The changes that match the summary look good to me. Thank you, those are good catches. This one I'm less sure about: > @@ -2788,6 +2791,14 @@ update