From f168fb2e16b3124786864159ac12dd660fa8dea9 Mon Sep 17 00:00:00 2001 From: Denis Kenzior Date: Tue, 4 Feb 2020 11:05:10 -0600 Subject: [PATCH] manager: Fix valgrind complaint ==192== Conditional jump or move depends on uninitialised value(s) ==192== at 0x4531D3: l_queue_find (queue.c:346) ==192== by 0x42F1F8: manager_config_notify (manager.c:667) ==192== by 0x45A895: process_multicast (genl.c:970) ==192== by 0x45A895: received_data (genl.c:1037) ==192== by 0x4577B2: io_callback (io.c:126) ==192== by 0x456B0D: l_main_iterate (main.c:473) ==192== by 0x456BCB: l_main_run (main.c:520) ==192== by 0x456DDA: l_main_run_with_signal (main.c:642) ==192== by 0x4034B0: main (main.c:497) --- src/manager.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/manager.c b/src/manager.c index 0a610754..648259e8 100644 --- a/src/manager.c +++ b/src/manager.c @@ -650,7 +650,8 @@ static void manager_config_notify(struct l_genl_msg *msg, void *user_data) * in that case retry the setup now that an interface, likely * the initial default one, has been added. */ - state = manager_find_pending(manager_parse_wiphy_id(msg)); + wiphy_id = manager_parse_wiphy_id(msg); + state = manager_find_pending(wiphy_id); if (state && state->retry) { state->retry = false;