[PATCH] nl80211: delay event processing during cmd handling

Benjamin Berg benjamin at sipsolutions.net
Fri Oct 10 10:21:52 PDT 2025


Hi,

On Fri, 2025-10-10 at 10:17 -0700, Ben Greear wrote:
> On 8/7/25 04:25, Benjamin Berg wrote:
> > From: Benjamin Berg <benjamin.berg at intel.com>
> I believe this TEST_FAIL_TAG("queued") part is causing compile
> problems when debugging is not enabled,
> because it will become "0" and compiler doesn't like a bare zero.

Hmm, makes sense. Easiest solution might be to explicitly cast the
result to (void), I expect that should be fine for the compiler then.

Benjamin

> 
>  From os.h:
> 
> #if defined(WPA_TRACE_BFD) && defined(CONFIG_TESTING_OPTIONS)
> #define TEST_FAIL() testing_test_fail(NULL, false)
> #define TEST_FAIL_TAG(tag) testing_test_fail(tag, false)
> int testing_test_fail(const char *tag, bool is_alloc);
> int testing_set_fail_pattern(bool is_alloc, char *patterns);
> int testing_get_fail_pattern(bool is_alloc, char *buf, size_t
> buflen);
> #else
> #define TEST_FAIL() 0
> #define TEST_FAIL_TAG(tag) 0
> 
> 
> > +
> > +queue_event:
> > +	TEST_FAIL_TAG("queued");
> > +
> > +	event = os_malloc(sizeof(*event));
> > +	WPA_ASSERT(event);
> 
> Thanks,
> Ben



More information about the Hostap mailing list