[kaffe] CVS kaffe (jim): Fixed exec bug.

Kaffe CVS cvs-commits at kaffe.org
Thu Apr 15 16:24:03 PDT 2004


PatchSet 4645 
Date: 2004/04/15 22:56:07
Author: jim
Branch: HEAD
Tag: (none) 
Log:
Fixed exec bug.

While trying run cgi-scripts from Tomcat, I discovered
this bug in our exec code.

Members: 
	ChangeLog:1.2221->1.2222 
	libraries/clib/native/UNIXProcess.c:1.28->1.29 

Index: kaffe/ChangeLog
diff -u kaffe/ChangeLog:1.2221 kaffe/ChangeLog:1.2222
--- kaffe/ChangeLog:1.2221	Thu Apr 15 20:58:08 2004
+++ kaffe/ChangeLog	Thu Apr 15 22:56:07 2004
@@ -1,5 +1,11 @@
 2004-04-15  Jim Pick <jim at kaffe.org>
 
+	* libraries/clib/native/UNIXProcess.c:
+	Fix bug where the exec code would search the path for
+	a binary, but then didn't use the found file.
+
+2004-04-15  Jim Pick <jim at kaffe.org>
+
 	* configure.ac:
 	Removed stray "HEADER" in file that broke
 	--with-staticlib.
Index: kaffe/libraries/clib/native/UNIXProcess.c
diff -u kaffe/libraries/clib/native/UNIXProcess.c:1.28 kaffe/libraries/clib/native/UNIXProcess.c:1.29
--- kaffe/libraries/clib/native/UNIXProcess.c:1.28	Mon Apr 12 11:40:07 2004
+++ kaffe/libraries/clib/native/UNIXProcess.c	Thu Apr 15 22:56:10 2004
@@ -261,8 +261,13 @@
 	    {
 	      rc = 0;
 	      nptr = NULL;
+	      KFREE (argv[0]);
+	      argv[0] = file;
+	    }
+	  else
+	    {
+	      KFREE (file);
 	    }
-	  KFREE (file);
 	}
     }
   else if (access (argv[0], X_OK) != -1)




More information about the kaffe mailing list