diff options
| author | Jouni Malinen <j@w1.fi> | 2015-05-02 19:26:06 +0300 |
|---|---|---|
| committer | Dan Pasanen <dan.pasanen@gmail.com> | 2015-05-11 09:02:08 -0500 |
| commit | 341a8b3a15959e123e920b0b549faa3786903107 (patch) | |
| tree | f0eb0516fb59691e6765bdd2b5082bf932229a49 | |
| parent | f24da54fee7e3cb0ba28fbb69d56055d310afb53 (diff) | |
| download | android_external_wpa_supplicant_8-341a8b3a15959e123e920b0b549faa3786903107.tar.gz android_external_wpa_supplicant_8-341a8b3a15959e123e920b0b549faa3786903107.tar.bz2 android_external_wpa_supplicant_8-341a8b3a15959e123e920b0b549faa3786903107.zip | |
EAP-pwd server: Fix Total-Length parsing for fragment reassembly
The remaining number of bytes in the message could be smaller than the
Total-Length field size, so the length needs to be explicitly checked
prior to reading the field and decrementing the len variable. This could
have resulted in the remaining length becoming negative and interpreted
as a huge positive integer.
In addition, check that there is no already started fragment in progress
before allocating a new buffer for reassembling fragments. This avoid a
potential memory leak when processing invalid message.
Change-Id: I1501af3aefa3a9784704cbd8ec625f66b4bb5df5
Signed-off-by: Jouni Malinen <j@w1.fi>
| -rw-r--r-- | src/eap_server/eap_server_pwd.c | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/src/eap_server/eap_server_pwd.c b/src/eap_server/eap_server_pwd.c index cbce683c..9bb7e01d 100644 --- a/src/eap_server/eap_server_pwd.c +++ b/src/eap_server/eap_server_pwd.c @@ -920,11 +920,21 @@ static void eap_pwd_process(struct eap_sm *sm, void *priv, * the first fragment has a total length */ if (EAP_PWD_GET_LENGTH_BIT(lm_exch)) { + if (len < 2) { + wpa_printf(MSG_DEBUG, + "EAP-pwd: Frame too short to contain Total-Length field"); + return; + } tot_len = WPA_GET_BE16(pos); wpa_printf(MSG_DEBUG, "EAP-pwd: Incoming fragments, total " "length = %d", tot_len); if (tot_len > 15000) return; + if (data->inbuf) { + wpa_printf(MSG_DEBUG, + "EAP-pwd: Unexpected new fragment start when previous fragment is still in use"); + return; + } data->inbuf = wpabuf_alloc(tot_len); if (data->inbuf == NULL) { wpa_printf(MSG_INFO, "EAP-pwd: Out of memory to " |