| Message ID | 20130412231633.03A4D13FDF2@ushik.mtv.corp.google.com |
|---|---|
| State | Rejected |
| Headers | show |
On Fri, Apr 12, 2013 at 04:14:13PM -0700, Dmitry Shmidt wrote: > diff --git a/src/rsn_supp/preauth.c b/src/rsn_supp/preauth.c > @@ -416,7 +416,7 @@ void pmksa_candidate_add(struct wpa_sm *sm, const u8 *bssid, > int rsn_preauth_scan_results(struct wpa_sm *sm) > { > - if (sm->ssid_len == 0) > + if (sm == NULL || sm->ssid_len == 0) > return -1; As pointed out in a separate email thread, this is not needed in hostap.git and not the proper way of addressing the issue behind the proposed change here. As such, I'm dropping this.
diff --git a/src/rsn_supp/preauth.c b/src/rsn_supp/preauth.c index ab61867..f5138ed 100644 --- a/src/rsn_supp/preauth.c +++ b/src/rsn_supp/preauth.c @@ -416,7 +416,7 @@ void pmksa_candidate_add(struct wpa_sm *sm, const u8 *bssid, */ int rsn_preauth_scan_results(struct wpa_sm *sm) { - if (sm->ssid_len == 0) + if (sm == NULL || sm->ssid_len == 0) return -1; /*
Signed-off-by: Dmitry Shmidt <dimitrysh@google.com> --- src/rsn_supp/preauth.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)