From bfdc2a3172567dc7a77920557cbdb64d55d471bc Mon Sep 17 00:00:00 2001 From: Jouni Malinen Date: Tue, 7 Jan 2014 15:58:01 +0200 Subject: [PATCH] bsd: Fix NULL pointer dereference on error path The error path in bsd_init() on struct bsd_driver_data allocation was jumping to location where drv is dereferenced. That will crash and it is easier to just return from the function since no cleanup steps are needed in this case. Signed-hostap: Jouni Malinen --- src/drivers/driver_bsd.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/drivers/driver_bsd.c b/src/drivers/driver_bsd.c index 097e7d670..71c9ce917 100644 --- a/src/drivers/driver_bsd.c +++ b/src/drivers/driver_bsd.c @@ -857,8 +857,8 @@ bsd_init(struct hostapd_data *hapd, struct wpa_init_params *params) drv = os_zalloc(sizeof(struct bsd_driver_data)); if (drv == NULL) { - printf("Could not allocate memory for bsd driver data\n"); - goto bad; + wpa_printf(MSG_ERROR, "Could not allocate memory for bsd driver data"); + return NULL; } drv->event_buf_len = rtbuf_len();