From 833d0d45e8bc625fae856ccfa895f8ed01fcd2cd Mon Sep 17 00:00:00 2001 From: Eduardo Abinader Date: Wed, 17 Aug 2016 19:29:39 +0200 Subject: [PATCH] radius: Sanity check for NULL pointer segfault When the RADIUS client has not yet been fully enabled, MIB command was segfaulting hostapd. Signed-off-by: Eduardo Abinader --- src/radius/radius_client.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/radius/radius_client.c b/src/radius/radius_client.c index a4edd5fa3..06c804d13 100644 --- a/src/radius/radius_client.c +++ b/src/radius/radius_client.c @@ -1636,11 +1636,16 @@ static int radius_client_dump_acct_server(char *buf, size_t buflen, int radius_client_get_mib(struct radius_client_data *radius, char *buf, size_t buflen) { - struct hostapd_radius_servers *conf = radius->conf; + struct hostapd_radius_servers *conf; int i; struct hostapd_radius_server *serv; int count = 0; + if (!radius) + return 0; + + conf = radius->conf; + if (conf->auth_servers) { for (i = 0; i < conf->num_auth_servers; i++) { serv = &conf->auth_servers[i];