Commit b21104c9 authored Dec 18, 2021 by Ondrej Zajicek (work)

Nest: Do not ignore secondary flag changes in ifa updates

Compare all IA_* flags that are set by sysdep iface code.

The old code ignores IA_SECONDARY flag when comparing whether iface
address updates from kernel changed anything. This is usually not an
issue as kernel removes all secondary addresses due to removal of the
primary one, but it breaks when sysctl 'promote_secondaries' is enabled
and kernel promotes secondary addresses to primary ones.

Thanks to 'Alexander' for the bugreport.

parent f772afc5

nest/iface.c

+1 −1

Original line number	Original line	Diff line number	Diff line
	@@ -591,7 +591,7 @@ ifa_update(struct ifa *a)
	if (ipa_equal(b->brd, a->brd) &&		if (ipa_equal(b->brd, a->brd) &&
	ipa_equal(b->opposite, a->opposite) &&		ipa_equal(b->opposite, a->opposite) &&
	b->scope == a->scope &&		b->scope == a->scope &&
	!((b->flags ^ a->flags) & IA_PEER))		!((b->flags ^ a->flags) & (IA_SECONDARY \| IA_PEER \| IA_HOST)))
	{		{
	b->flags \|= IA_UPDATED;		b->flags \|= IA_UPDATED;
	return b;		return b;

Admin message