From f7635c9ffe738f1b9376d04d8c4acec2fc3caf8b Mon Sep 17 00:00:00 2001 From: Thomas Haller Date: Tue, 10 Aug 2021 17:26:07 +0200 Subject: [PATCH] platform/netlink: use switch for type check in validate_nla() --- src/libnm-platform/nm-netlink.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/src/libnm-platform/nm-netlink.c b/src/libnm-platform/nm-netlink.c index d836e4c9ff..c7dfb3664a 100644 --- a/src/libnm-platform/nm-netlink.c +++ b/src/libnm-platform/nm-netlink.c @@ -622,14 +622,17 @@ validate_nla(const struct nlattr *nla, int maxtype, const struct nla_policy *pol if (pt->maxlen && len > pt->maxlen) return -NME_UNSPEC; - if (pt->type == NLA_STRING) { - const char *data; + switch (pt->type) { + case NLA_STRING: + { + const char *data = nla_data(nla); nm_assert(minlen > 0); - data = nla_data(nla); if (data[len - 1u] != '\0') return -NME_UNSPEC; + break; + } } return 0;