From 63a463cfbdeb83586031cff2332c5374e5b54777 Mon Sep 17 00:00:00 2001
From: millert <>
Date: Thu, 4 Jun 2009 20:39:13 +0000
Subject: [PATCH] Don't assume that we can overwrite strings in the
 environment. Someone may have passed a read-only string to putenv() (I'm
 looking at you cron!).

---
 src/lib/libc/stdlib/setenv.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/lib/libc/stdlib/setenv.c b/src/lib/libc/stdlib/setenv.c
index 242830d7..2e882cdb 100644
--- a/src/lib/libc/stdlib/setenv.c
+++ b/src/lib/libc/stdlib/setenv.c
@@ -1,4 +1,4 @@
-/*	$OpenBSD: setenv.c,v 1.10 2009/06/03 15:52:16 millert Exp $ */
+/*	$OpenBSD: setenv.c,v 1.11 2009/06/04 20:39:13 millert Exp $ */
 /*
  * Copyright (c) 1987 Regents of the University of California.
  * All rights reserved.
@@ -101,11 +101,13 @@ setenv(const char *name, const char *value, int rewrite)
 	if ((C = __findenv(name, (int)(np - name), &offset)) != NULL) {
 		if (!rewrite)
 			return (0);
+#if 0 /* XXX - existing entry may not be writable */
 		if (strlen(C) >= l_value) {	/* old larger; copy over */
 			while ((*C++ = *value++))
 				;
 			return (0);
 		}
+#endif
 	} else {					/* create new slot */
 		size_t cnt;
 		char **P;