include_dir: skip processing on error (CID 54744)

We already skipped processing for DBUS_ERROR_FILE_NOT_FOUND;
but if the error was something else, we would pass the NULL
pointer dir to _dbus_directory_get_next_file(), which dereferences it.
Reported by Coverity: CID 54744: Dereference after null check (FORWARD_NULL)

Bug: https://bugs.freedesktop.org/show_bug.cgi?id=90021
[smcv: re-worded commit message]
Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>

diff --git a/bus/config-parser.c b/bus/config-parser.c
index 1540fe435fc21b034204736321a4b8ee3ff2d8b2..3f59730b1a5390d20bf5a810aa3432350c68d0ff 100644 (file)
--- a/bus/config-parser.c
+++ b/bus/config-parser.c
@@ -2271,6 +2271,8 @@ include_dir (BusConfigParser   *parser,
           dbus_error_free (error);
           goto success;
         }
+      else
+        goto failed;
     }
 
   dbus_error_init (&tmp_error);