I just tried this patch out and it seems to work so I'm going to go ahead and merge.
Ethan On Thu, Apr 14, 2011 at 11:11 AM, Ben Pfaff <[email protected]> wrote: > On Thu, Apr 14, 2011 at 11:05:39AM -0700, Ethan Jackson wrote: >> > What motivated this change? >> >> Just defensive coding. I noticed it when looking over the patch >> again. If n_stb_slaves is 0, e.g. there are no enabled slaves. It's >> better to choose something (e.g. the active slave) instead of trigger >> a divide by zero. > > I hadn't noticed the division by zero. Good spotting, thanks. > _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
