On Tue, Apr 12, 2016 at 12:25:24PM -0400, Russell Bryant wrote: > On Mon, Apr 11, 2016 at 9:48 PM, Ben Pfaff <[email protected]> wrote: > > > The new helpers get_local_datapath() and get_patched_datapath() make code > > a little shorter and easier to read. They also avoid a pitfall that was > > present in at least a few of the instances: CONTAINER_OF is not safe on a > > null pointer, because it does a raw pointer subtraction and will change > > NULL to something else. This wasn't actually a problem in these particular > > cases because the value it was subtracting was zero (although arguably it > > is still undefined behavior because the compiler is allowed to assume that > > a pointer on which arithmetic is performed is nonnull). > > > > Signed-off-by: Ben Pfaff <[email protected]> > > > > Acked-by: Russell Bryant <[email protected]>
Thanks for the reviews. I applied both of these to master. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
