util: Add helper to return name of address family
We have a few places where we want to include the name of the internet protocol version (IPv4 or IPv6) in a message, which we handle with an open-coded ?: expression. This seems like something that might be more widely useful, so make a trivial helper to return the correct string based on the address family. Signed-off-by: David Gibson <david@gibson.dropbear.id.au> Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
This commit is contained in:

committed by
Stefano Brivio

parent
f4e38b5cd2
commit
67a6258918
@@ -309,7 +309,7 @@ unsigned int nl_get_ext_if(int s, sa_family_t af)
|
||||
if (defifi) {
|
||||
if (ndef > 1)
|
||||
info("Multiple default %s routes, picked first",
|
||||
af == AF_INET ? "IPv4" : "IPv6");
|
||||
af_name(af));
|
||||
return defifi;
|
||||
}
|
||||
|
||||
@@ -318,11 +318,11 @@ unsigned int nl_get_ext_if(int s, sa_family_t af)
|
||||
return anyifi;
|
||||
|
||||
info("Multiple interfaces with %s routes, use -i to select one",
|
||||
af == AF_INET ? "IPv4" : "IPv6");
|
||||
af_name(af));
|
||||
}
|
||||
|
||||
if (!nany)
|
||||
info("No interfaces with %s routes", af == AF_INET ? "IPv4" : "IPv6");
|
||||
info("No interfaces with %s routes", af_name(af));
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
Reference in New Issue
Block a user