mirror of
https://git.proxmox.com/git/mirror_iproute2
synced 2025-08-18 07:12:53 +00:00
lib utils: Use helpers to get AF bit/byte len
Added funcs to get AF_XXX len in bit/bytes and replace places where switch(AF_XXX) is used for this. Signed-off-by: Vadim Kochan <vadim4j@gmail.com>
This commit is contained in:
parent
2e7e805d0a
commit
f3a2ddc124
@ -101,6 +101,9 @@ extern int get_s8(__s8 *val, const char *arg, int base);
|
||||
extern char* hexstring_n2a(const __u8 *str, int len, char *buf, int blen);
|
||||
extern __u8* hexstring_a2n(const char *str, __u8 *buf, int blen);
|
||||
|
||||
extern int af_bit_len(int af);
|
||||
extern int af_byte_len(int af);
|
||||
|
||||
extern const char *format_host(int af, int len, const void *addr,
|
||||
char *buf, int buflen);
|
||||
extern const char *rt_addr_n2a(int af, const void *addr,
|
||||
|
22
ip/iproute.c
22
ip/iproute.c
@ -268,20 +268,6 @@ static int filter_nlmsg(struct nlmsghdr *n, struct rtattr **tb, int host_len)
|
||||
return 1;
|
||||
}
|
||||
|
||||
static int calc_host_len(const struct rtmsg *r)
|
||||
{
|
||||
if (r->rtm_family == AF_INET6)
|
||||
return 128;
|
||||
else if (r->rtm_family == AF_INET)
|
||||
return 32;
|
||||
else if (r->rtm_family == AF_DECnet)
|
||||
return 16;
|
||||
else if (r->rtm_family == AF_IPX)
|
||||
return 80;
|
||||
else
|
||||
return -1;
|
||||
}
|
||||
|
||||
static void print_rtax_features(FILE *fp, unsigned int features)
|
||||
{
|
||||
unsigned int of = features;
|
||||
@ -302,7 +288,7 @@ int print_route(const struct sockaddr_nl *who, struct nlmsghdr *n, void *arg)
|
||||
int len = n->nlmsg_len;
|
||||
struct rtattr * tb[RTA_MAX+1];
|
||||
char abuf[256];
|
||||
int host_len = -1;
|
||||
int host_len;
|
||||
__u32 table;
|
||||
SPRINT_BUF(b1);
|
||||
static int hz;
|
||||
@ -320,7 +306,7 @@ int print_route(const struct sockaddr_nl *who, struct nlmsghdr *n, void *arg)
|
||||
return -1;
|
||||
}
|
||||
|
||||
host_len = calc_host_len(r);
|
||||
host_len = af_bit_len(r->rtm_family);
|
||||
|
||||
parse_rtattr(tb, RTA_MAX, RTM_RTA(r), len);
|
||||
table = rtm_get_table(r, tb);
|
||||
@ -1134,9 +1120,9 @@ static int save_route(const struct sockaddr_nl *who, struct nlmsghdr *n,
|
||||
int len = n->nlmsg_len;
|
||||
struct rtmsg *r = NLMSG_DATA(n);
|
||||
struct rtattr *tb[RTA_MAX+1];
|
||||
int host_len = -1;
|
||||
int host_len;
|
||||
|
||||
host_len = calc_host_len(r);
|
||||
host_len = af_bit_len(r->rtm_family);
|
||||
len -= NLMSG_LENGTH(sizeof(*r));
|
||||
parse_rtattr(tb, RTA_MAX, RTM_RTA(r), len);
|
||||
|
||||
|
@ -66,14 +66,7 @@ int print_rule(const struct sockaddr_nl *who, struct nlmsghdr *n, void *arg)
|
||||
|
||||
parse_rtattr(tb, FRA_MAX, RTM_RTA(r), len);
|
||||
|
||||
if (r->rtm_family == AF_INET)
|
||||
host_len = 32;
|
||||
else if (r->rtm_family == AF_INET6)
|
||||
host_len = 128;
|
||||
else if (r->rtm_family == AF_DECnet)
|
||||
host_len = 16;
|
||||
else if (r->rtm_family == AF_IPX)
|
||||
host_len = 80;
|
||||
host_len = af_bit_len(r->rtm_family);
|
||||
|
||||
if (n->nlmsg_type == RTM_DELRULE)
|
||||
fprintf(fp, "Deleted ");
|
||||
|
58
lib/utils.c
58
lib/utils.c
@ -431,6 +431,27 @@ int get_addr_1(inet_prefix *addr, const char *name, int family)
|
||||
return 0;
|
||||
}
|
||||
|
||||
int af_bit_len(int af)
|
||||
{
|
||||
switch (af) {
|
||||
case AF_INET6:
|
||||
return 128;
|
||||
case AF_INET:
|
||||
return 32;
|
||||
case AF_DECnet:
|
||||
return 16;
|
||||
case AF_IPX:
|
||||
return 80;
|
||||
}
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
||||
int af_byte_len(int af)
|
||||
{
|
||||
return af_bit_len(af) / 8;
|
||||
}
|
||||
|
||||
int get_prefix_1(inet_prefix *dst, char *arg, int family)
|
||||
{
|
||||
int err;
|
||||
@ -456,17 +477,8 @@ int get_prefix_1(inet_prefix *dst, char *arg, int family)
|
||||
|
||||
err = get_addr_1(dst, arg, family);
|
||||
if (err == 0) {
|
||||
switch(dst->family) {
|
||||
case AF_INET6:
|
||||
dst->bitlen = 128;
|
||||
break;
|
||||
case AF_DECnet:
|
||||
dst->bitlen = 16;
|
||||
break;
|
||||
default:
|
||||
case AF_INET:
|
||||
dst->bitlen = 32;
|
||||
}
|
||||
dst->bitlen = af_bit_len(family);
|
||||
|
||||
if (slash) {
|
||||
if (get_netmask(&plen, slash+1, 0)
|
||||
|| plen > dst->bitlen) {
|
||||
@ -697,7 +709,6 @@ static const char *resolve_address(const void *addr, int len, int af)
|
||||
}
|
||||
#endif
|
||||
|
||||
|
||||
const char *format_host(int af, int len, const void *addr,
|
||||
char *buf, int buflen)
|
||||
{
|
||||
@ -705,27 +716,8 @@ const char *format_host(int af, int len, const void *addr,
|
||||
if (resolve_hosts) {
|
||||
const char *n;
|
||||
|
||||
if (len <= 0) {
|
||||
switch (af) {
|
||||
case AF_INET:
|
||||
len = 4;
|
||||
break;
|
||||
case AF_INET6:
|
||||
len = 16;
|
||||
break;
|
||||
case AF_IPX:
|
||||
len = 10;
|
||||
break;
|
||||
#ifdef AF_DECnet
|
||||
/* I see no reasons why gethostbyname
|
||||
may not work for DECnet */
|
||||
case AF_DECnet:
|
||||
len = 2;
|
||||
break;
|
||||
#endif
|
||||
default: ;
|
||||
}
|
||||
}
|
||||
len = len <= 0 ? af_byte_len(af) : len;
|
||||
|
||||
if (len > 0 &&
|
||||
(n = resolve_address(addr, len, af)) != NULL)
|
||||
return n;
|
||||
|
Loading…
Reference in New Issue
Block a user