From f7df8fda1a880a0922e08302d5666adf2be4ddfb Mon Sep 17 00:00:00 2001 From: Thomas Haller Date: Thu, 21 Feb 2019 08:57:41 +0100 Subject: [PATCH] platform/netlink: assert for valid policy for string attribute in validate_nla The policy for strings must indicate a minlen of at least 1. Everything else is a bug, because the policy contains invalid data -- and is determined at compile-time. --- src/platform/nm-netlink.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/platform/nm-netlink.c b/src/platform/nm-netlink.c index d367df304d..2d4f5ea549 100644 --- a/src/platform/nm-netlink.c +++ b/src/platform/nm-netlink.c @@ -586,7 +586,11 @@ validate_nla (const struct nlattr *nla, int maxtype, return -NME_UNSPEC; if (pt->type == NLA_STRING) { - const char *data = nla_data (nla); + const char *data; + + nm_assert (minlen > 0); + + data = nla_data (nla); if (data[nla_len (nla) - 1] != '\0') return -NME_UNSPEC; }