From 65ab7eb1fa8d3a26c704cc4de95a668ccbaf43dc Mon Sep 17 00:00:00 2001 From: Jouni Malinen Date: Tue, 7 Feb 2017 17:46:29 +0200 Subject: [PATCH] GAS: Fix OSU Providers List response with invalid configuration If the OSU Providers configuration was invalid (included osu_server_uri but not osu_method_list), it was possible for the GAS response generation to hit a NULL dereference. Fix this by checking for the invalid configuration before trying to fill in the OSU methods. Signed-off-by: Jouni Malinen --- src/ap/gas_serv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/ap/gas_serv.c b/src/ap/gas_serv.c index 95f0a667e..96cd7030a 100644 --- a/src/ap/gas_serv.c +++ b/src/ap/gas_serv.c @@ -691,7 +691,7 @@ static void anqp_add_osu_provider(struct wpabuf *buf, /* OSU Method List */ count = wpabuf_put(buf, 1); - for (i = 0; p->method_list[i] >= 0; i++) + for (i = 0; p->method_list && p->method_list[i] >= 0; i++) wpabuf_put_u8(buf, p->method_list[i]); *count = i;