Alan DeKok wrote:
Michael Joosten <michael.joosten@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 --- 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; }