コミット: 3ae2578282187c775247cb2c3cc9f442d9c138e0 - external-busybox (git)

external/busybox

コミットメタ情報

• リビジョン: 3ae2578282187c775247cb2c3cc9f442d9c138e0 (tree)
• 日時: 2014-08-05 20:13:28
• 作者: Tanguy Pruvot <tanguy.pruvot@gmai...>
• コミッター: Tanguy Pruvot

ログメッセージ

android: clearenv should not set environ to NULL

Remove the old internal clearenv() function which is
no more required (exported in libc)

Change-Id: I357f6a3b057c366761b481c09b471d7c53d7a419

変更サマリ

• modified: android/android.c
• modified: libbb/xfuncs_printf.c

差分

--- a/android/android.c
+++ b/android/android.c
@@ -10,12 +10,22 @@
 #include <stdlib.h>
 #include "libbb.h"
 
-/* declared in stdlib.h */
-int clearenv()
+#ifndef BIONIC_ICS
+int clearenv(void)
 {
-	environ = NULL;
+	char **P = environ;
+
+	/* should never be NULL */
+	if (!environ)
+		environ = (char **)xzalloc(sizeof(char *));
+
+	if (P != NULL) {
+		for (; *P; ++P)
+			*P = NULL;
+	}
 	return 0;
 }
+#endif
 
 /* no /etc/shells anyway */
 char *getusershell() { return NULL; }

--- a/libbb/xfuncs_printf.c
+++ b/libbb/xfuncs_printf.c
@@ -322,6 +322,11 @@ char* FAST_FUNC xasprintf(const char *format, ...)
 
 void FAST_FUNC xsetenv(const char *key, const char *value)
 {
+#ifdef __BIONIC__
+	/* on login, can be NULL, and should not be for bionic */
+	if (environ == NULL)
+		bb_error_msg_and_die("environment is not initialized");
+#endif
 	if (setenv(key, value, 1))
 		bb_error_msg_and_die("%s", bb_msg_memory_exhausted);
 }