On Thu, Jul 30, 2015 at 10:03 AM, Justin Pettit <jpet...@nicira.com> wrote: > >> 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.
Thanks. I fixed this in the earlier patch that introduced it (Alex pointed it out, I think). > Acked-by: Justin Pettit <jpet...@nicira.com> Thanks! -- "I don't normally do acked-by's. I think it's my way of avoiding getting blamed when it all blows up." Andrew Morton _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
