Last call for 1.1.3?

Michael Joosten michael.joosten at c-lab.de
Tue Aug 22 18:29:00 CEST 2006


Alan DeKok wrote:

>Michael Joosten <michael.joosten at c-lab.de> wrote:
>  
>
>>Done. Bug 385
>>    
>>
>...
>  
>
>>Also attached. Made against 1.1.1, but I don't that matters.
>>    
>>
>
>  Did you try compiling it?  There are a bunch of typos, e.g.
>ca_path, not CA_PATH.
>
>  I've edited it, and committed the fixes.
>  
>
Sorry, looked just at the labels, not the struct field names.  Just made 
my own correction, if you want to compare:

Michael

-------------- next part --------------
--- radiusd/src/modules/rlm_eap/types/rlm_eap_tls/rlm_eap_tls.c-	2006-04-28 20:21:49.000000000 +0200
+++ radiusd/src/modules/rlm_eap/types/rlm_eap_tls/rlm_eap_tls.c	2006-08-22 18:20:08.000000000 +0200
@@ -449,15 +449,25 @@
 	if (conf) {
 		if (conf->dh_file) free(conf->dh_file);
 		conf->dh_file = NULL;
+		if (conf->ca_path) free(conf->ca_path);
+		conf->ca_path = NULL;
 		if (conf->certificate_file) free(conf->certificate_file);
 		conf->certificate_file = NULL;
 		if (conf->private_key_file) free(conf->private_key_file);
 		conf->private_key_file = NULL;
 		if (conf->private_key_password) free(conf->private_key_password);
 		conf->private_key_password = NULL;
+		if (conf->ca_file) free(conf->ca_file);
+		conf->ca_file = NULL;
 		if (conf->random_file) free(conf->random_file);
 		conf->random_file = NULL;
 
+		if (conf->check_cert_cn) free(conf->check_cert_cn);
+		conf->check_cert_cn = NULL;
+		if (conf->cipher_list) free(conf->cipher_list);
+		conf->cipher_list = NULL;
+		if (conf->check_cert_issuer) free(conf->check_cert_issuer);
+		conf->check_cert_issuer = NULL;
 		free(inst->conf);
 		inst->conf = NULL;
 	}


More information about the Freeradius-Devel mailing list