[PATCH iproute2 2/3] iplink: Process "alias" parameter correctly
flat view
STALE3226d
From: Serhey Popovych <hidden>
Date: 2017-12-18 18:55:03
Subsystem:
the rest · Maintainer:
Linus Torvalds
Do not stop parameters processing after "alias" parameter: it might not be a last one. Seems copy pasted from "type" parameter code. Check it's length does not exceed IFALIASZ - 1. Better we warn than get RTNL error. Signed-off-by: Serhey Popovych <redacted> --- ip/iplink.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/ip/iplink.c b/ip/iplink.c
index 4f9c169..4c96711 100644
--- a/ip/iplink.c
+++ b/ip/iplink.c@@ -770,11 +770,12 @@ int iplink_parse(int argc, char **argv, struct iplink_req *req, argc--; argv++; break; } else if (matches(*argv, "alias") == 0) { + len = strlen(*argv); + if (len >= IFALIASZ) + invarg("alias too long\n", *argv); NEXT_ARG(); addattr_l(&req->n, sizeof(*req), IFLA_IFALIAS, - *argv, strlen(*argv)); - argc--; argv++; - break; + *argv, len); } else if (strcmp(*argv, "group") == 0) { NEXT_ARG(); if (*group != -1)
--
1.7.10.4