From 2e4327816587ee7d2e3238268cacbdf40faa4b0e Mon Sep 17 00:00:00 2001 From: Christian Eggers Date: Fri, 23 Sep 2022 16:55:55 +0200 Subject: [PATCH] advertising: parse_secondary: fix mask value It looks like a wrong mask value is used here as only the bits mentioned in adv_secondary::flag can be set again within the loop. Replace magic number by preprocessor macro. --- src/advertising.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/advertising.c b/src/advertising.c index 15ef44def..42ac62760 100644 --- a/src/advertising.c +++ b/src/advertising.c @@ -1051,7 +1051,7 @@ static bool parse_secondary(DBusMessageIter *iter, return false; /* Reset secondary channels before parsing */ - client->flags &= 0xfe00; + client->flags &= ~MGMT_ADV_FLAG_SEC_MASK; dbus_message_iter_get_basic(iter, &str); -- 2.47.3