Fix possible null pointer dereference.
authorJohn Darrington <john@darrington.wattle.id.au>
Thu, 11 Aug 2016 19:52:52 +0000 (21:52 +0200)
committerJohn Darrington <john@darrington.wattle.id.au>
Thu, 11 Aug 2016 19:53:45 +0000 (21:53 +0200)
Found by cppcheck

src/language/lexer/variable-parser.c

index a45d384293ba9b6251433a60abd4d89480dbc9fa..390b98dae36860019f90638f02c45bbfb9e443d2 100644 (file)
@@ -201,9 +201,9 @@ add_variable (struct variable ***v, size_t *nv, size_t *mv,
                "All variables in this variable list must have the "
                "same width.  %s will be omitted from the list."),
          var_get_name ((*v)[0]), add_name, add_name);
-  else if ((pv_opts & PV_NO_DUPLICATE) && included[idx])
+  else if ((pv_opts & PV_NO_DUPLICATE) && included && included[idx])
     msg (SE, _("Variable %s appears twice in variable list."), add_name);
-  else if ((pv_opts & PV_DUPLICATE) || !included[idx])
+  else if ((pv_opts & PV_DUPLICATE) || !included || !included[idx])
     {
       if (*nv >= *mv)
         {