summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMatt Porter <mporter@konsulko.com>2018-07-24 12:21:36 -0400
committerMatt Porter <mporter@konsulko.com>2018-07-24 14:08:31 -0400
commit4da53f18e97043b387cb46cc29797ba6494fbb0d (patch)
tree2e743f660755abd1bd75e79aacd9ab3f5bd058be
parent91f8941a5d3571ee107635a945cdd00dcb267b6f (diff)
Fix crash in parsing of technology added and removed signalsflounder_5.99.2flounder/5.99.25.99.2
The object path in the TechnologyAdded and TechnologyRemoved signals is provided as a tuple. The binding currently segfaults when receiving this signal since it tries to parse it with an invalid variant format string. Update the format string parameter to reflect that it should parse the object path as a tuple. Bug-AGL: SPEC-1585 Change-Id: I36a588ec43475a24996b9a58c1f326a35d260396 Signed-off-by: Matt Porter <mporter@konsulko.com>
-rw-r--r--binding/network-api.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/binding/network-api.c b/binding/network-api.c
index f2c3c7a..90b132e 100644
--- a/binding/network-api.c
+++ b/binding/network-api.c
@@ -278,7 +278,7 @@ static void network_manager_signal_callback(
if (!g_strcmp0(signal_name, "TechnologyAdded")) {
- g_variant_get(parameters, "o(a{sv})", &path, &array);
+ g_variant_get(parameters, "(oa{sv})", &path, &array);
basename = connman_strip_path(path);
g_assert(basename); /* guaranteed by dbus */
@@ -310,7 +310,7 @@ static void network_manager_signal_callback(
} else if (!g_strcmp0(signal_name, "TechnologyRemoved")) {
- g_variant_get(parameters, "o", &path);
+ g_variant_get(parameters, "(o)", &path);
basename = connman_strip_path(path);
g_assert(basename); /* guaranteed by dbus */