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]> _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
