[ovs-dev] [PATCH v2] netdev: check for NULL fields in netdev_get_addrs

Aaron Conole aconole at redhat.com
Tue Jul 18 15:32:44 UTC 2017


Daniel Alvarez Sanchez <dalvarez at redhat.com> writes:

> When the interfaces list is retrieved through getiffaddrs(), there
> might be elements with iface_name set to NULL.
>
> This patch checks ifa_name to be not NULL before comparing it to the
> actual device name in the loop that calculates how many interfaces
> exist with that same name.
>
> Also, this patch checks that ifa_netmask is not NULL for coherence
> with the existing code so that it doesn't allocate more memory
> than needed if this field is NULL.
>
> Note, that these checks are already being done later in the function
> so it should be done in both places.
>
> Signed-off-by: Daniel Alvarez <dalvarez at redhat.com>
> ---

Acked-by: Aaron Conole <aconole at redhat.com>


More information about the dev mailing list