Re: [Lost] [Patch] shell - Hänger in start_app

[Kevin Wolf <mail@xxxxxxxxxxxxx>]

Toni Kaufmann schrieb:
> Dieser Patch beseitigt einen Hänger in Shell, falls init ein Programm
> nicht starten kann.

Diese Lösung ist einfach, leicht verständlich und kaputt.

Daher folgender Gegenvorschlag, der keine Speicherlöcher und geöffneten 
Dateien hinterlassen sollte.
Index: src/modules/c/shell/commmands.c
===================================================================
--- src/modules/c/shell/commmands.c	(Revision 615)
+++ src/modules/c/shell/commmands.c	(Arbeitskopie)
@@ -150,9 +150,9 @@
             DEBUG_MSG("Starte das Programm");
             pid_t pid = init_execute(message);
 
-            // Wenn es gewuenscht wurde, wird jezt gewartet, bis der Prozess
+            // Wenn es gewuenscht wurde, wird jetzt gewartet, bis der Prozess
             // beendet wird.
-            if (wait == TRUE) {
+            if ((pid != 0) && wait) {
                 while (get_parent_pid(pid) != 0) {
                     yield();
                 }