OpenAFS Master Repository branch, openafs-stable-1_8_x, updated. openafs-stable-1_8_5-27-g64d0352

Gerrit Code Review gerrit@openafs.org
Sun, 26 Jan 2020 07:52:58 -0500


The following commit has been merged in the openafs-stable-1_8_x branch:
commit 64d0352df1406d9601d6d772555c4c39efc05d7e
Author: Andrew Deason <adeason@sinenomine.net>
Date:   Fri Feb 1 16:31:50 2019 -0600

    Avoid calling krb5_free_context(NULL)
    
    Several places in the code currently call krb5_free_context(ctx) in a
    cleanup code path, where 'ctx' may or may not be NULL. This is not
    guaranteed to be okay, so check for NULL to make sure we don't cause
    issues in these code paths.
    
    While we are here cleaning up krb5_free_context() calls, also fix a
    few call sites in afscp_util.c that were not calling krb5_free_context
    in all error paths.
    
    Reviewed-on: https://gerrit.openafs.org/13461
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
    (cherry picked from commit 26b1dc036719a588a5cadecb14053bd4079c1f48)
    
    Change-Id: I3b0d22f51f4fe85897116b7f96d096570258eed2
    Reviewed-on: https://gerrit.openafs.org/13902
    Reviewed-by: Andrew Deason <adeason@sinenomine.net>
    Reviewed-by: Michael Meffie <mmeffie@sinenomine.net>
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Stephan Wiesand <stephan.wiesand@desy.de>

 src/aklog/aklog.c         |    7 ++++---
 src/libafscp/afscp_util.c |    9 +++++----
 src/rxkad/ticket5.c       |    4 +++-
 3 files changed, 12 insertions(+), 8 deletions(-)

-- 
OpenAFS Master Repository