Looks good.

--Justin


On Apr 26, 2013, at 11:37 AM, Ben Pfaff <[email protected]> wrote:

> What a dumb bug.
> 
> Signed-off-by: Ben Pfaff <[email protected]>
> ---
> lib/dpif-netdev.c |    2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/lib/dpif-netdev.c b/lib/dpif-netdev.c
> index e4a2f75..6838800 100644
> --- a/lib/dpif-netdev.c
> +++ b/lib/dpif-netdev.c
> @@ -412,7 +412,7 @@ do_add_port(struct dp_netdev *dp, const char *devname, 
> const char *type,
>     port->type = xstrdup(type);
> 
>     error = netdev_get_mtu(netdev, &mtu);
> -    if (!error) {
> +    if (!error && mtu > max_mtu) {
>         max_mtu = mtu;
>     }
> 
> -- 
> 1.7.2.5
> 
> _______________________________________________
> 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