Looks Good.

On Thu, Mar 31, 2011 at 2:16 PM, Ben Pfaff <[email protected]> wrote:
> Otherwise the ofproto's attempt to flush flows from the dpif will fail with
> an error, causing a spurious log message.
> ---
>  vswitchd/bridge.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/vswitchd/bridge.c b/vswitchd/bridge.c
> index bf89778..72091a2 100644
> --- a/vswitchd/bridge.c
> +++ b/vswitchd/bridge.c
> @@ -1690,13 +1690,13 @@ bridge_destroy(struct bridge *br)
>             port_destroy(port);
>         }
>         list_remove(&br->node);
> +        ofproto_destroy(br->ofproto);
>         error = dpif_delete(br->dpif);
>         if (error && error != ENOENT) {
>             VLOG_ERR("failed to delete %s: %s",
>                      dpif_name(br->dpif), strerror(error));
>         }
>         dpif_close(br->dpif);
> -        ofproto_destroy(br->ofproto);
>         mac_learning_destroy(br->ml);
>         hmap_destroy(&br->ifaces);
>         hmap_destroy(&br->ports);
> --
> 1.7.1
>
> _______________________________________________
> dev mailing list
> [email protected]
> http://openvswitch.org/mailman/listinfo/dev
>
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to