[ruby-dev:26339] Re: ruby_setenv dumps core with mod_ruby/1.4.2

山本です。

>定義されていない環境変数を削除しようとするとSEGVします。

すみません、間違えました。今度は大丈夫だと思いますが・・・

Index: hash.c
===================================================================
RCS file: /src/ruby/hash.c,v
retrieving revision 1.128.2.11
diff -u -w -b -p -r1.128.2.11 hash.c
--- hash.c	28 Feb 2005 02:45:20 -0000	1.128.2.11
+++ hash.c	12 Jun 2005 12:29:36 -0000
@@ -1837,26 +1837,23 @@ ruby_setenv(name, value)
 	tmpenv[max] = 0;
 	environ = tmpenv;		/* tell exec where it is now */
     }
-    if (!value) {
-	if (environ != origenviron) {
+    if (environ[i]) {
 	    char **envp = origenviron;
 	    while (*envp && *envp != environ[i]) envp++;
 	    if (!*envp)
 		free(environ[i]);
-	}
+	if (!value) {
 	while (environ[i]) {
 	    environ[i] = environ[i+1];
 	    i++;
 	}
 	return;
     }
-    if (!environ[i]) {			/* does not exist yet */
+    }
+    else {			/* does not exist yet */
+	if (!value) return;
 	REALLOC_N(environ, char*, i+2);	/* just expand it a bit */
 	environ[i+1] = 0;	/* make sure it's null terminated */
-    }
-    else {
-	if (environ[i] != origenviron[i])
-	    free(environ[i]);
     }
     environ[i] = ALLOC_N(char, strlen(name) + strlen(value) + 2);
 #ifndef MSDOS

条件文が必要な理由も、過去ログからみつかりました。[ruby-dev:10887]
これじゃ削るわけにいかないですね・・・