diff options
author | Jouni Malinen <jouni.malinen@atheros.com> | 2011-04-14 02:45:14 +0300 |
---|---|---|
committer | Jouni Malinen <j@w1.fi> | 2011-04-14 02:45:14 +0300 |
commit | 066608f3ff5f0042636b02737d15035129a65829 (patch) | |
tree | 74d29d043c839421d4b8b2dfa9d2183d18aa5ecd /src/eap_peer | |
parent | b211f3eb715f208ab3dd2ca6e4296f445e9d3725 (diff) | |
download | external_wpa_supplicant_8_ti-066608f3ff5f0042636b02737d15035129a65829.zip external_wpa_supplicant_8_ti-066608f3ff5f0042636b02737d15035129a65829.tar.gz external_wpa_supplicant_8_ti-066608f3ff5f0042636b02737d15035129a65829.tar.bz2 |
Avoid theoretical NULL pointer dereference during TLS reassemble
This function does not get called with in_data == NULL in practice, but
it seems to be at least partly prepared for that case, so better make it
consistent by handling the NULL value throughout the function.
Diffstat (limited to 'src/eap_peer')
-rw-r--r-- | src/eap_peer/eap_tls_common.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/eap_peer/eap_tls_common.c b/src/eap_peer/eap_tls_common.c index 7bd50f6..d1567e9 100644 --- a/src/eap_peer/eap_tls_common.c +++ b/src/eap_peer/eap_tls_common.c @@ -361,7 +361,8 @@ static int eap_peer_tls_reassemble_fragment(struct eap_ssl_data *data, eap_peer_tls_reset_input(data); return -1; } - wpabuf_put_buf(data->tls_in, in_data); + if (in_data) + wpabuf_put_buf(data->tls_in, in_data); data->tls_in_left -= in_len; if (data->tls_in_left > 0) { |