From 217dc6d4cc7c82523f48a9fe7669cd70c4c0eb09 Mon Sep 17 00:00:00 2001 From: Denis Kenzior Date: Tue, 5 Nov 2019 10:42:04 -0600 Subject: [PATCH] unit: Fixup test-wsc - Add missing break statement - Add missing va_end - Fix logic inversion introduced by 2d995b17c1dff Fixes: 2d95b17c1dff ("unit: Update event handler in WSC, eapol tests") Reported-By: Will Dietz --- unit/test-wsc.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/unit/test-wsc.c b/unit/test-wsc.c index 0e845dc2..01420d47 100644 --- a/unit/test-wsc.c +++ b/unit/test-wsc.c @@ -1948,7 +1948,7 @@ static void verify_handshake_event(struct handshake_state *hs, { const struct wsc_credential *cred; - assert(va_arg(args, unsigned int) != + assert(va_arg(args, unsigned int) == EAP_WSC_EVENT_CREDENTIAL_OBTAINED); cred = va_arg(args, const struct wsc_credential *); @@ -1956,10 +1956,13 @@ static void verify_handshake_event(struct handshake_state *hs, sizeof(struct wsc_credential))); data->cur_cred += 1; + break; } default: break; } + + va_end(args); } static int verify_8021x(uint32_t ifindex,