On 20 February 2016 at 16:24, Jouni Malinen <j@xxxxx> wrote: > On Wed, Feb 17, 2016 at 01:14:14PM +0100, Janusz Dziedzic wrote: >> Setup ctrl_interface name. This should be used >> to check interface <-> udp port assigment. > >> diff --git a/wpa_supplicant/ctrl_iface_udp.c b/wpa_supplicant/ctrl_iface_udp.c >> @@ -363,6 +363,7 @@ struct ctrl_iface_priv * >> wpa_supplicant_ctrl_iface_init(struct wpa_supplicant *wpa_s) > >> + char port_str[40] = "udp:9877"; > > What is the point of setting the "udp:9877" value here? > >> + /* Setup correct ctrl_interface */ >> + os_snprintf(port_str, sizeof(port_str), "udp:%d", port); > > Wouldn't this unconditionally replace that "udp:9877" value? > >> + if (wpa_s->conf->ctrl_interface) >> + os_free(wpa_s->conf->ctrl_interface); > > os_free(NULL) is fine, so please no "if (ptr)" checks before os_free() > calls. > >> + wpa_s->conf->ctrl_interface = os_strdup(port_str); > > Should there be a check to verify that os_strdup() succeeds here? > Will fix this in next version. > -- > Jouni Malinen PGP id EFC895FA _______________________________________________ Hostap mailing list Hostap@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/hostap