Add a null check to avoid dereferencing a null pointer

Change-Id: I72f1dfc45f8765061fc46937e6e3a24f355b7ff1

diff --git a/packaging/murphy.spec b/packaging/murphy.spec
index 12e9d9201203b643d8ec571bdd6e7ac26e3d4e0f..11b78d002d6b630b5a66f4e32ed5934c72e6e42a 100644 (file)
--- a/packaging/murphy.spec
+++ b/packaging/murphy.spec
@@ -29,7 +29,7 @@
 Summary: Resource policy framework
 Name: murphy
 Version: 0.0.75
-Release: 23
+Release: 24
 License: BSD-3-Clause
 Group: System/Service
 URL: http://01.org/murphy/

diff --git a/src/core/console.c b/src/core/console.c
index a9591b1a1b7ab51b0e64f14517e84871f058cc71..720efbcaa005be6f040cbb20237129b6d5e7550f 100644 (file)
--- a/src/core/console.c
+++ b/src/core/console.c
@@ -700,14 +700,16 @@ static ssize_t input_evt(mrp_console_t *mc, void *buf, size_t size)
         clearerr(stdout);
         clearerr(stderr);
 
-        MRP_CONSOLE_BUSY(mc, {
-                if (cmd->flags & MRP_CONSOLE_RAWINPUT) {
-                    raw = raw_argument(buf, grp->name, cmd->name);
-                    cmd->raw(mc, grp->user_data, grp->name, cmd->name, raw);
-                }
-                else
-                    cmd->tok(mc, grp->user_data, argc, argv);
-            });
+        if (grp != NULL) {
+            MRP_CONSOLE_BUSY(mc, {
+                    if (cmd->flags & MRP_CONSOLE_RAWINPUT) {
+                        raw = raw_argument(buf, grp->name, cmd->name);
+                        cmd->raw(mc, grp->user_data, grp->name, cmd->name, raw);
+                    }
+                    else
+                        cmd->tok(mc, grp->user_data, argc, argv);
+                });
+        }
 
         /*
          * Although our watch for c->pout[RFD]/output_cb should take