From 7013a837f0e97eae2a5a15b1326d23f99f3f3540 Mon Sep 17 00:00:00 2001 From: aland Date: Mon, 5 Mar 2007 14:11:55 +0000 Subject: [PATCH] prev_eap_ds may be NULL. Don't dereference it. Bug found by Coverity --- src/modules/rlm_eap/libeap/eap_tls.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/modules/rlm_eap/libeap/eap_tls.c b/src/modules/rlm_eap/libeap/eap_tls.c index 4e36833..4450bb4 100644 --- a/src/modules/rlm_eap/libeap/eap_tls.c +++ b/src/modules/rlm_eap/libeap/eap_tls.c @@ -332,7 +332,8 @@ static eaptls_status_t eaptls_verify(EAP_HANDLER *handler) ((eap_ds->response->length == EAP_HEADER_LEN + 2) && ((eaptls_packet->flags & 0xc0) == 0x00))) { - if (prev_eap_ds->request->id == eap_ds->response->id) { + if (prev_eap_ds && + (prev_eap_ds->request->id == eap_ds->response->id)) { /* * Run the ACK handler directly from here. */ -- 2.1.4