aboutsummaryrefslogtreecommitdiff
path: root/toys/pending
diff options
context:
space:
mode:
authorEthan Sommer <e5ten.arch@gmail.com>2020-04-14 20:55:30 -0400
committerRob Landley <rob@landley.net>2020-04-15 00:53:02 -0500
commitc4d5d44e0ff311847f262bf80456fed725a5f581 (patch)
tree1a77880ea3861fafbe4bb6fa92e0b2b96f062303 /toys/pending
parent5291d7c50b6921b8e216bdc22ab63ba33e020be2 (diff)
downloadtoybox-c4d5d44e0ff311847f262bf80456fed725a5f581.tar.gz
ip: appease clang warnings
fix false positive -Wformat-security by passing string literal directly to error_exit, instead of assigning it to a variable and passing that fix -Wparentheses-equality by removing extra parentheses
Diffstat (limited to 'toys/pending')
-rw-r--r--toys/pending/ip.c20
1 files changed, 7 insertions, 13 deletions
diff --git a/toys/pending/ip.c b/toys/pending/ip.c
index cc86c19f..621d1eb7 100644
--- a/toys/pending/ip.c
+++ b/toys/pending/ip.c
@@ -1447,7 +1447,7 @@ static int ipaddr_print( struct linkdata *link, int flag_l)
}
for (; NLMSG_OK(addr_ptr, len); addr_ptr = NLMSG_NEXT(addr_ptr, len)) {
- if ((addr_ptr->nlmsg_type == RTM_NEWADDR))
+ if (addr_ptr->nlmsg_type == RTM_NEWADDR)
print_addrinfo(addr_ptr, flag_l);
if ((addr_ptr->nlmsg_type == NLMSG_DONE) ||
(addr_ptr->nlmsg_type == NLMSG_ERROR) ||
@@ -1499,15 +1499,13 @@ struct {
static void show_iproute_help(void)
{
- char *errmsg = "\n\n" \
+ error_exit("\n\n" \
"iproute { list | flush } SELECTOR\n" \
"iproute get ADDRESS [from ADDRESS iif STRING]\n" \
" [oif STRING]\n" \
"iproute { add | del | change | append | replace | test } ROUTE\n" \
" SELECTOR := [root PREFIX] [match PREFIX] [proto RTPROTO]\n" \
- " ROUTE := [TYPE] PREFIX [proto RTPROTO] [metric METRIC]";
-
- error_exit(errmsg);
+ " ROUTE := [TYPE] PREFIX [proto RTPROTO] [metric METRIC]");
}
static void print_rta_metrics(char* out, const struct rtattr *mxattr)
@@ -2174,12 +2172,10 @@ static int iproute(char **argv)
// ===========================================================================
static void show_iprule_help(void)
{
- char *errmsg = "usage: ip rule [ list | add | del ] SELECTOR ACTION\n"
+ error_exit("usage: ip rule [ list | add | del ] SELECTOR ACTION\n"
"SELECTOR := [ from PREFIX ] [ to PREFIX ] [pref NUMBER] [ tos TOS ]\n"
" [ fwmark FWMARK] [ dev/iif STRING ] [type TYPE]\n"
- "ACTION := [ table TABLE_ID ] [ realms [SRCREALM/]DSTREALM ]";
-
- error_exit(errmsg);
+ "ACTION := [ table TABLE_ID ] [ realms [SRCREALM/]DSTREALM ]");
}
static int ruleupdate(char **argv)
@@ -2423,12 +2419,10 @@ static int iprule(char **argv)
//============================================================================
static void show_iptunnel_help(void)
{
- char *errmsg = "usage: iptunnel { add | change | del | show } [NAME]\n"
+ error_exit("usage: iptunnel { add | change | del | show } [NAME]\n"
" [mode { ipip | gre | sit }] [remote ADDR] [local ADDR]\n"
" [[i|o]seq] [[i|o]key KEY] [[i|o]csum] [ttl TTL]\n"
- " [tos TOS] [[no]pmtudisc] [dev PHYS_DEV]";
-
- error_exit(errmsg);
+ " [tos TOS] [[no]pmtudisc] [dev PHYS_DEV]");
}
static int tnl_ioctl(char *dev, int rtype, struct ip_tunnel_parm *ptnl)