From: dtucker <dtucker>
Date: Thu, 2 Oct 2003 10:07:09 +0000 (+0000)
Subject:  - (dtucker) [session.c] Fix bus errors on some 64-bit Solaris configurations.
X-Git-Tag: V_3_8_P1~220
X-Git-Url: http://andersk.mit.edu/gitweb/openssh.git/commitdiff_plain/3a23ba0ea6e6fdc7a0a159c2d032ff6038ce84c5?hp=21c1aca3d4b4acf4a50f8170c46cb57fe2f023cc

 - (dtucker) [session.c] Fix bus errors on some 64-bit Solaris configurations.
   Based on patches by Matthias Koeppe and Thomas Baden.  ok djm@
---

diff --git a/ChangeLog b/ChangeLog
index 692e7a6b..4e1430e0 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -31,6 +31,8 @@
  - (dtucker) [configure.ac] Don't set DISABLE_SHADOW when configuring
    --with-pam.  ok djm@
  - (dtucker) [ssh-gss.h] Prototype change missed in sync.
+ - (dtucker) [session.c] Fix bus errors on some 64-bit Solaris configurations.
+   Based on patches by Matthias Koeppe and Thomas Baden.  ok djm@
 
 20030930
  - (bal) Fix issues in openbsd-compat/realpath.c
diff --git a/session.c b/session.c
index 647be401..ccdc4249 100644
--- a/session.c
+++ b/session.c
@@ -906,7 +906,7 @@ read_etc_default_login(char ***env, u_int *envsize, uid_t uid)
 {
 	char **tmpenv = NULL, *var;
 	u_int i, tmpenvsize = 0;
-	mode_t mask;
+	u_long mask;
 
 	/*
 	 * We don't want to copy the whole file to the child's environment,
@@ -927,7 +927,7 @@ read_etc_default_login(char ***env, u_int *envsize, uid_t uid)
 	
 	if ((var = child_get_env(tmpenv, "UMASK")) != NULL)
 		if (sscanf(var, "%5lo", &mask) == 1)
-			umask(mask);
+			umask((mode_t)mask);
 	
 	for (i = 0; tmpenv[i] != NULL; i++)
 		xfree(tmpenv[i]);