Browse Source

FILS: Clean up HLP resize check

The "!wpabuf_resize(...) == 0" condition does not make any sense. It
happens to work, but this is really supposed to simple check with
wpabuf_resize() returns non-zero and "wpabuf_resize(...)" is the
cleanest way of doing so.

Signed-off-by: Jouni Malinen <jouni@qca.qualcomm.com>
Jouni Malinen 8 years ago
parent
commit
0d6dc6830c
1 changed files with 2 additions and 2 deletions
  1. 2 2
      src/ap/fils_hlp.c

+ 2 - 2
src/ap/fils_hlp.c

@@ -281,8 +281,8 @@ static void fils_dhcp_handler(int sd, void *eloop_ctx, void *sock_ctx)
 	} else {
 		wpabuf_put_data(resp, pos, end - pos);
 	}
-	if (!wpabuf_resize(&sta->fils_hlp_resp, wpabuf_len(resp) +
-			   2 * wpabuf_len(resp) / 255 + 100) == 0) {
+	if (wpabuf_resize(&sta->fils_hlp_resp, wpabuf_len(resp) +
+			  2 * wpabuf_len(resp) / 255 + 100)) {
 		wpabuf_free(resp);
 		return;
 	}