1 # Fix a potential but unlikely to occur double free() in a couple places in ipropd_svc.c.

     2 # This has been reported to MIT who will be fixing this via pull request

     3 # https://github.com/krb5/krb5/pull/396 .

     4 # Patch source: in-house

     5 

     6 diff -ur krb5-1.13.3/src/kadmin/server/ipropd_svc.c krb5-1.13.3.memleak/src/kadmin/server/ipropd_svc.c

     7 --- krb5-1.13.3/src/kadmin/server/ipropd_svc.c

     8 +++ krb5-1.13.3.memleak/src/kadmin/server/ipropd_svc.c

     9 @@ -160,8 +160,6 @@

    10  	client_name = buf_to_string(&client_desc);

    11  	service_name = buf_to_string(&service_desc);

    12  	if (client_name == NULL || service_name == NULL) {

    13 -	    free(client_name);

    14 -	    free(service_name);

    15  	    krb5_klog_syslog(LOG_ERR,

    16  			     _("%s: out of memory recording principal names"),

    17  			     whoami);

    18 @@ -288,8 +286,6 @@

    19  	client_name = buf_to_string(&client_desc);

    20  	service_name = buf_to_string(&service_desc);

    21  	if (client_name == NULL || service_name == NULL) {

    22 -	    free(client_name);

    23 -	    free(service_name);

    24  	    DPRINT("%s: out of memory\n", whoami);

    25  	    krb5_klog_syslog(LOG_ERR,

    26  			     _("%s: out of memory recording principal names"),