[PATCH v2] P2P: fix memory leak when dbus provides bonjour params while adding a UPnP service

Jouni Malinen j at w1.fi
Sat Jul 20 10:37:22 PDT 2024


On Thu, Jul 18, 2024 at 06:23:49PM +0200, Davide Caratti wrote:
> Using D-Bus, it is possible to add a valid UPnP service where 'query' and
> 'response' are specified. In this case, memory for 'query' and 'response'
> is allocated but not used nor freed. Valgrind complains as follows:

> Fix this ensuring that query and resp are freed both in the error and
> non-error path of wpas_dbus_handler_p2p_add_service(). Also, add a
> test in test_dbus.py to verify the correct behavior.

Thanks, applied.
 
-- 
Jouni Malinen                                            PGP id EFC895FA



More information about the Hostap mailing list