modinfo: skip parm entries without a name

Specially crafted/corrupt modules may have a parm/parmtype entry which
lacks the "name:" prefix. Throw a warning and skip such entries.

Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
Link: https://github.com/kmod-project/kmod/pull/451
Signed-off-by: Lucas De Marchi <demarchi@kernel.org>

diff --git a/tools/modinfo.c b/tools/modinfo.c
index 55e97da3c7b6e209d9480baf8e1d8b380f88a729..21688ed538a8c108d88dfc6106f3f2c16d6c4a76 100644 (file)
--- a/tools/modinfo.c
+++ b/tools/modinfo.c
@@ -88,6 +88,11 @@ static int process_parm(const char *key, const char *value, struct param **param
                return 0;
        }
 
+       if (colon == value) {
+               ERR("Missing param name in value \"%s\"\n", value);
+               return 0;
+       }
+
        name = value;
        namelen = colon - value;
        if (streq(key, "parm")) {