On Wed, Jun 11, 2014 at 1:52 PM, Thomas Graf <tg...@suug.ch> wrote:
> On 06/11/14 at 11:11am, Jesse Gross wrote:
>> On Wed, Jun 11, 2014 at 3:04 AM, Thomas Graf <tg...@suug.ch> wrote:
>> > 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
>                                                             ^^^^^^^
>
>> Sorry, is there something you're trying to point out here?
>
> Sorry for not being more verbose. Just the minor typo "trucate" above.

Thanks, I see it now.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to