On Mon, May 13, 2013 at 8:44 AM, Kyle Mestery (kmestery) <[email protected]> wrote: > On May 13, 2013, at 10:33 AM, Jesse Gross <[email protected]> wrote: >> When sending a packet, a positive length indicates success and a >> negative length indicates failure. However, the check for success >> looked for non-zero values which catches both of these cases. This >> can result in incorrect stats and leak memory on failure. >> >> Introduced by commit be7cd27e44258bdb3c4e7dd8fd7389b5db56d55a (datapath: >> Unify vport error stats handling.). >> >> CC: Pravin B Shelar <[email protected]> >> Signed-off-by: Jesse Gross <[email protected]> > > > Nice catch. > > Acked-by: Kyle Mestery <[email protected]>
Thanks for the quick review, I applied this to master. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
