network/routing-policy-rule: use int32_t for suppress_prefixlen

The kernel parses FRA_SUPPRESS_PREFIXLEN as uint32_t, but internally
handled as signed integer and negative values as unset. Let's explicitly
specify the size of the variable.

No functional change, just refactoring.
This commit is contained in:
Yu Watanabe
2024-08-20 02:21:21 +09:00
parent 0a3a3a2a1e
commit 933448defe
3 changed files with 4 additions and 4 deletions
+2 -2
View File
@@ -714,7 +714,7 @@ int parse_ip_port_range(const char *s, uint16_t *low, uint16_t *high, bool allow
return 0;
}
int parse_ip_prefix_length(const char *s, int *ret) {
int parse_ip_prefix_length(const char *s, int32_t *ret) {
unsigned l;
int r;
@@ -725,7 +725,7 @@ int parse_ip_prefix_length(const char *s, int *ret) {
if (l > 128)
return -ERANGE;
*ret = (int) l;
*ret = (int32_t) l;
return 0;
}
+1 -1
View File
@@ -141,7 +141,7 @@ int parse_nice(const char *p, int *ret);
int parse_ip_port(const char *s, uint16_t *ret);
int parse_ip_port_range(const char *s, uint16_t *low, uint16_t *high, bool allow_zero);
int parse_ip_prefix_length(const char *s, int *ret);
int parse_ip_prefix_length(const char *s, int32_t *ret);
int parse_oom_score_adjust(const char *s, int *ret);
+1 -1
View File
@@ -40,7 +40,7 @@ typedef struct RoutingPolicyRule {
/* FRA_FLOW */
/* FRA_TUN_ID */
int32_t suppress_ifgroup; /* FRA_SUPPRESS_IFGROUP */
int suppress_prefixlen; /* FRA_SUPPRESS_PREFIXLEN */
int32_t suppress_prefixlen; /* FRA_SUPPRESS_PREFIXLEN */
uint32_t table; /* FRA_TABLE, also used in struct fib_rule_hdr */
uint32_t fwmask; /* FRA_FWMASK */
char *oif; /* FRA_OIFNAME */