This is the mail archive of the
java-patches@sourceware.cygnus.com
mailing list for the Java project.
Patch: don't use sprintf
- To: Java Patch List <java-patches at sourceware dot cygnus dot com>
- Subject: Patch: don't use sprintf
- From: Tom Tromey <tromey at cygnus dot com>
- Date: 17 Dec 1999 09:07:59 -0700
- Reply-To: tromey at cygnus dot com
I'm checking in the appended patch, which removes a use of sprintf and
fixes a couple formatting problems.
1999-12-17 Tom Tromey <tromey@cygnus.com>
* java/lang/natPosixProcess.cc (startProcess): Don't use sprintf.
Tom
Index: java/lang/natPosixProcess.cc
===================================================================
RCS file: /cvs/java/libgcj/libjava/java/lang/natPosixProcess.cc,v
retrieving revision 1.3
diff -u -r1.3 natPosixProcess.cc
--- natPosixProcess.cc 1999/09/10 22:03:08 1.3
+++ natPosixProcess.cc 1999/12/17 16:01:49
@@ -168,12 +168,13 @@
if (envp)
{
// preserve PATH unless specified explicitly
- char *path_val = getenv("PATH");
+ char *path_val = getenv ("PATH");
environ = env;
- if (getenv("PATH") == NULL)
+ if (getenv ("PATH") == NULL)
{
- char *path_env = (char *) _Jv_Malloc (strlen(path_val) + 5 + 1);
- sprintf (path_env, "PATH=%s", path_val);
+ char *path_env = (char *) _Jv_Malloc (strlen (path_val) + 5 + 1);
+ strcpy (path_env, "PATH=");
+ strcat (path_env, path_val);
putenv (path_env);
}
}