]> git.ipfire.org Git - thirdparty/dbus.git/commitdiff
_dbus_spawn_program() now returns #NULL for unification in case of error
authorRalf Habacker <ralf.habacker@freenet.de>
Wed, 24 Nov 2021 08:47:49 +0000 (09:47 +0100)
committerRalf Habacker <ralf.habacker@freenet.de>
Mon, 29 Nov 2021 12:07:04 +0000 (13:07 +0100)
dbus/dbus-spawn-win.c
tools/dbus-run-session.c

index 07d03bace564a1aa25452ec200b72d03035fbaa4..f0773aa7fa8232e76cc83b1c6001cdb68c9674e1 100644 (file)
@@ -514,7 +514,7 @@ _dbus_spawn_program (const char *name,
   arg_string = build_commandline (argv);
 #endif
   if (!arg_string)
-    return INVALID_HANDLE_VALUE;
+    return NULL;
 
   env_string = build_env_string(envp);
 
@@ -545,7 +545,7 @@ _dbus_spawn_program (const char *name,
     free (env_string);
 
   if (!result)
-    return INVALID_HANDLE_VALUE;
+    return NULL;
 
   CloseHandle (pi.hThread);
   return pi.hProcess;
@@ -680,7 +680,7 @@ _dbus_spawn_async_with_babysitter (DBusBabysitter           **sitter_p,
     }
 
   PING();
-  if (handle == INVALID_HANDLE_VALUE)
+  if (handle == NULL)
     {
       sitter->child_handle = NULL;
       sitter->have_spawn_errno = TRUE;
index 290fcc409e5b83d394c178e475f381231d7a09ea..f481d7cefa3a45ea82928be98d545e01f5ed42e3 100644 (file)
@@ -442,7 +442,7 @@ run_session (const char *dbus_daemon,
   dbus_daemon_argv[4] = NULL;
 
   server_handle = _dbus_spawn_program (dbus_daemon, dbus_daemon_argv, NULL, TRUE);
-  if (!server_handle)
+  if (server_handle == NULL)
     {
       _dbus_win_set_error_from_last_error (&error, "Could not start dbus daemon");
       goto out;
@@ -530,7 +530,7 @@ run_session (const char *dbus_daemon,
     goto out;
 
   app_handle = _dbus_spawn_program (argv[prog_arg], argv + prog_arg, env, FALSE);
-  if (!app_handle)
+  if (app_handle == NULL)
     {
       _dbus_win_set_error_from_last_error (&error, "Unable to start child process");
       goto out;