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

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

2011-11-30 Thread Pravin B Shelar
Following patch fixes memory leak in case there is ODP_FIT_ERROR on flow key. --- ofproto/ofproto-dpif.c | 11 +++ 1 files changed, 11 insertions(+), 0 deletions(-) diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c index 14b5447..aabe064 100644 --- a/ofproto/ofproto-dpif.c ++