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

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

krb5 commit [krb5-1.13]: Fix iprop server stub error management

daemon@ATHENA.MIT.EDU (Tom Yu)
Fri Jan 15 17:14:52 2016

Date: Fri, 15 Jan 2016 17:10:28 -0500
From: Tom Yu <tlyu@mit.edu>
Message-Id: <201601152210.u0FMASMv005965@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/f513d6705809f30ae907059b5a367ac2718a96f6
commit f513d6705809f30ae907059b5a367ac2718a96f6
Author: Greg Hudson <ghudson@mit.edu>
Date:   Thu Jan 14 17:51:53 2016 -0500

    Fix iprop server stub error management
    
    The ipropd stubs free client_name and server_name in the cleanup
    handler, so should not free them in out-of-memory conditions.
    Reported by Will Fiveash.
    
    (cherry picked from commit d998b088adb875bc0a4e13c184075f91fb23336b)
    
    ticket: 8347
    version_fixed: 1.13.4
    tags: -pullup
    status: resolved

 src/kadmin/server/ipropd_svc.c |    4 ----
 1 files changed, 0 insertions(+), 4 deletions(-)

diff --git a/src/kadmin/server/ipropd_svc.c b/src/kadmin/server/ipropd_svc.c
index 06ecb68..802ffc2 100644
--- a/src/kadmin/server/ipropd_svc.c
+++ b/src/kadmin/server/ipropd_svc.c
@@ -160,8 +160,6 @@ iprop_get_updates_1_svc(kdb_last_t *arg, struct svc_req *rqstp)
 	client_name = buf_to_string(&client_desc);
 	service_name = buf_to_string(&service_desc);
 	if (client_name == NULL || service_name == NULL) {
-	    free(client_name);
-	    free(service_name);
 	    krb5_klog_syslog(LOG_ERR,
 			     _("%s: out of memory recording principal names"),
 			     whoami);
@@ -288,8 +286,6 @@ ipropx_resync(uint32_t vers, struct svc_req *rqstp)
 	client_name = buf_to_string(&client_desc);
 	service_name = buf_to_string(&service_desc);
 	if (client_name == NULL || service_name == NULL) {
-	    free(client_name);
-	    free(service_name);
 	    DPRINT("%s: out of memory\n", whoami);
 	    krb5_klog_syslog(LOG_ERR,
 			     _("%s: out of memory recording principal names"),
_______________________________________________
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