Re: [ovs-dev] [PATCH v2] ofproto-dpif: Fix memory leak sending packets to controller.

2013-05-02 Thread Ben Pfaff
Thanks, applied to branch-1.4. At the same time, I noticed that I had forgotten to tag 1.4.6, so I did that too. On Wed, May 01, 2013 at 09:55:58AM -0700, Justin Pettit wrote: > The code's tricky, but I looked over it a couple of times, and it > looks reasonable to me. > > Thanks, > > --Justin

Re: [ovs-dev] [PATCH v2] ofproto-dpif: Fix memory leak sending packets to controller.

2013-05-01 Thread Justin Pettit
The code's tricky, but I looked over it a couple of times, and it looks reasonable to me. Thanks, --Justin On Apr 25, 2013, at 11:23 AM, Ben Pfaff wrote: > In the case where execute_controller_action() returned true to > handle_flow_miss(), indicating that the packet had been sent to the > c

Re: [ovs-dev] [PATCH v2] ofproto-dpif: Fix memory leak sending packets to controller.

2013-04-25 Thread Ben Pfaff
I forgot to note in this version that this applies to branch-1.4. On Thu, Apr 25, 2013 at 11:23:00AM -0700, Ben Pfaff wrote: > In the case where execute_controller_action() returned true to > handle_flow_miss(), indicating that the packet had been sent to the > controller, nothing ever freed the p

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

2011-11-30 Thread Pravin Shelar
Thanks, Pushed to master. On Wed, Nov 30, 2011 at 1:14 PM, Ben Pfaff wrote: > If this subset of the previous patch is unchanged (I think so at first > glance) then it's good, thank you, please commit. ___ dev mailing list dev@openvswitch.org http://ope

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

2011-11-30 Thread Ben Pfaff
If this subset of the previous patch is unchanged (I think so at first glance) then it's good, thank you, please commit. ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev