On 06/10/14 at 04:47pm, Jesse Gross wrote:
> In some cases, the names of tunnel protocols are longer than the
> maximum allowed after concatenating port numbers. This truncates
> names to the correct size rather than assert-failing on the assumption
> that collisions between names of protocols are rather rate.
> 
> Signed-off-by: Jesse Gross <je...@nicira.com>
> ---
>  lib/netdev-vport.c | 10 ++++------
>  1 file changed, 4 insertions(+), 6 deletions(-)
> 
> diff --git a/lib/netdev-vport.c b/lib/netdev-vport.c
> index c214bf7..4dca091 100644
> --- a/lib/netdev-vport.c
> +++ b/lib/netdev-vport.c
> @@ -151,14 +151,12 @@ netdev_vport_get_dpif_port(const struct netdev *netdev,
>          const char *type = netdev_get_type(netdev);
>  
>          /*
> -         * Note: IFNAMSIZ is 16 bytes long. The maximum length of a VXLAN
> -         * or LISP port name below is 15 or 14 bytes respectively. Still,
> -         * assert here on the size of strlen(type) in case that changes
> -         * in the future.
> +         * Note: IFNAMSIZ is 16 bytes long. If the length of the type plus
> +         * the rest of the name is greater than this, trucate the type to
                                                         '''''''

> +         * fit. The probability of collisions seems quite low.
>           */
>          BUILD_ASSERT(NETDEV_VPORT_NAME_BUFSIZE >= IFNAMSIZ);
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to