WPS: Fix num_probereq_cb clearing on DISABLE to avoid segfault
authorChen, Yi <yichen@qti.qualcomm.com>
Wed, 2 Sep 2015 15:55:01 +0000 (21:25 +0530)
committerJouni Malinen <j@w1.fi>
Mon, 7 Sep 2015 17:51:58 +0000 (20:51 +0300)
commit24fd20438f00a6f1bdeb6f23358ef60c2696b488
treea0ffd51d30dd6c6d807b37f094f3354fabc3e471
parentae3eacf7717f0553c9434b920a799a1d61886b22
WPS: Fix num_probereq_cb clearing on DISABLE to avoid segfault

Reset hapd->num_probereq_cb to 0 on an interface deinit to avoid
unexpected behavior if the same interface is enabled again without fully
freeing the data structures. hostapd_register_probereq_cb() increments
hapd->num_probereq_cb by one and leaves all old values unchanged. In
this deinit+init case, that would result in the first entry in the list
having an uninitialized pointer and the next Probe Request frame
processing would likely cause the process to terminate on segmentation
fault.

This issue could be hit when hostapd was used with WPS enabled (non-zero
wps_state configuration parameter) and control interface command DISABLE
and ENABLE were used.

Signed-off-by: Jouni Malinen <jouni@qca.qualcomm.com>
src/ap/hostapd.c