[29112] in CVS-changelog-for-Kerberos-V5

home help back first fref pref prev next nref lref last post

krb5 commit [krb5-1.14]: Make ksu work when unsetenv() returns NULL

daemon@ATHENA.MIT.EDU (Tom Yu)
Mon Oct 26 15:39:40 2015

Date: Mon, 26 Oct 2015 15:39:36 -0400
From: Tom Yu <tlyu@mit.edu>
Message-Id: <201510261939.t9QJdafs025299@drugstore.mit.edu>
To: cvs-krb5@mit.edu
Reply-To: krbdev@mit.edu
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Errors-To: cvs-krb5-bounces@mit.edu

https://github.com/krb5/krb5/commit/83ef046b405dfad26538eeccc117e1ab178bdd91
commit 83ef046b405dfad26538eeccc117e1ab178bdd91
Author: Greg Hudson <ghudson@mit.edu>
Date:   Sun Oct 4 19:54:35 2015 -0400

    Make ksu work when unsetenv() returns NULL
    
    Some older platforms (OS X 10.4, glibc 2.2.1) declare unsetenv() as
    returning void, as does ksu's compatibility definition of unsetenv().
    Don't use the return value in get_configured_defccname().
    
    (cherry picked from commit 7eee546db10e907666e02fdded4f512e8d0faf4c)
    
    ticket: 8267
    version_fixed: 1.14
    status: resolved

 src/clients/ksu/main.c |    7 +------
 1 files changed, 1 insertions(+), 6 deletions(-)

diff --git a/src/clients/ksu/main.c b/src/clients/ksu/main.c
index 453dae6..2f8d8e1 100644
--- a/src/clients/ksu/main.c
+++ b/src/clients/ksu/main.c
@@ -810,12 +810,7 @@ get_configured_defccname(krb5_context context, char **target_out)
 
     *target_out = NULL;
 
-    if (unsetenv(KRB5_ENV_CCNAME) != 0) {
-        retval = errno;
-        com_err(prog_name, retval, _("while clearing the value of %s"),
-                KRB5_ENV_CCNAME);
-        return retval;
-    }
+    unsetenv(KRB5_ENV_CCNAME);
 
     /* Make sure we don't have a cached value for a different uid. */
     retval = krb5_cc_set_default_name(context, NULL);
_______________________________________________
cvs-krb5 mailing list
cvs-krb5@mit.edu
https://mailman.mit.edu/mailman/listinfo/cvs-krb5

home help back first fref pref prev next nref lref last post