> On Jul 28, 2015, at 8:44 AM, Ben Pfaff <b...@nicira.com> wrote: > > @@ -309,7 +295,7 @@ main(int argc, char *argv[]) > ctx.ovnsb_idl_txn = idl_loop_run(&ovnsb_idl_loop); > ctx.ovs_idl_txn = idl_loop_run(&ovs_idl_loop); > > - const struct ovsrec_bridge *br_int = get_bridge(&ctx, br_int_name); > + const struct ovsrec_bridge *br_int = get_br_int(ctx.ovs_idl);
This behavior already existed, but "br_int" may be null, and that could cause a problem when encaps_cleanup() is called a bit further down. Otherwise: Acked-by: Justin Pettit <jpet...@nicira.com> --Justin _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev