7 Nov
2019
7 Nov
'19
12:34 p.m.
On Thu, Oct 17, 2019 at 21:59:48 -0400, Laine Stump wrote:
From: Laine Stump <laine@redhat.com>
Although until now, any use of the extra_args argument (a pointer to a struct containing extra attributes to add the the RTM_NEWLINK message) would always have the ifindex and mac set, so the code could assume it was safe to add both to the message if extra_args != NULL. There is now a use for setting a MAC address in the RTM_NEWLINK without setting the ifindex, so we should check each of these separately.
Signed-off-by: Laine Stump <laine@redhat.com> --- src/util/virnetlink.c | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-)
Reviewed-by: Jiri Denemark <jdenemar@redhat.com>