Changeset 4552

Timestamp:

07/23/08 10:36:18 (5 years ago)

Author:

zecke

Message:

app_restarter: Make sure that argv[0] is the app we want to execute...

Stupid oversight. argv[0] is supposed to the command name of the
application.

File:

• developers/zecke/app_restarter/app_restarter.c (modified) (3 diffs)

developers/zecke/app_restarter/app_restarter.c

@@ -197 +197 @@
  * On any kind of exit ask the user to restart
  */
-int fork_and_exec(char* file, int argc, char** argv)
+int fork_and_exec(int argc, char** argv)
 {
     pid_t pid = vfork();
 
     if (pid == 0) {
-        execvp(file, argv);
-        fprintf(stderr, "Failed to launch: %s\n", file);
+        execvp(argv[0], argv);
+        fprintf(stderr, "Failed to launch: %s\n", argv[0]);
         _exit(-1);
     } else {
         int status;
 
+        fprintf(stderr, "Waiting for pid: %d\n", pid);
+
         /* wait for the process to die */
         while (waitpid(pid, &status, 0) != pid);
@@ -233 +235 @@
 
     /* Create a list for execvp with a sentinel */
-    int commands = argc-3;
+    int commands = argc-2;
     char** command_arguments = (char**) malloc(sizeof(char*)*(commands+1));
     for (i = 0; i < commands; ++i)
@@ -240 +242 @@
 
     for (;;) {
-        int status = fork_and_exec(argv[2], commands, command_arguments);
+        int status = fork_and_exec(commands, command_arguments);
         handle_crash(status, argv[1]);
     }