Win32 processes have two environments, a Unix-compatible one and an NT

environment.  NSS has no control over which of these environments will
be inherited by any child processes, so when NSS modifies or adds an
environment variable, it needs to do so to both environments.
This fixes bugzilla bug 66615.  Reviewed by wtc.

security/nss/lib/util/secport.c

@@ -38,7 +38,7 @@
  * 
  * NOTE - These are not public interfaces
  *
- * $Id: secport.c,v 1.5 2001/01/03 19:51:22 larryh%netscape.com Exp $
+ * $Id: secport.c,v 1.6 2001/01/26 03:24:05 nelsonb%netscape.com Exp $
  */
 
 #include "seccomon.h"
@@ -569,20 +569,21 @@ PORT_UCS2_ASCIIConversion(PRBool toUnicode, unsigned char *inBuf,
 int
 NSS_PutEnv(const char * envVarName, const char * envValue)
 {
+#if  defined(XP_MAC)
+    return SECFailure;
+#else
     SECStatus result = SECSuccess;
+    char *    encoded;
+    int       putEnvFailed;
 #ifdef _WIN32
     PRBool      setOK;
 
     setOK = SetEnvironmentVariable(envVarName, envValue);
     if (!setOK) {
         SET_ERROR_CODE
-        result = SECFailure;
+        return SECFailure;
     }
-#elif  defined(XP_MAC)
-	result = SECFailure;
-#else
-    char *          encoded;
-    int             putEnvFailed;
+#endif
 
     encoded = (char *)PORT_ZAlloc(strlen(envVarName) + 2 + strlen(envValue));
     strcpy(encoded, envVarName);
@@ -595,7 +596,7 @@ NSS_PutEnv(const char * envVarName, const char * envValue)
         result = SECFailure;
         PORT_Free(encoded);
     }
-#endif
     return result;
+#endif
 }