diff options
| author |   Scott Murray <scott.murray@konsulko.com> | 2022-12-28 09:44:30 -0500 |
|---|---|---|
| committer | Scott Murray <scott.murray@konsulko.com> | 2022-12-28 14:51:30 +0000 |
| commit | 86ecfa16aeb5c4942990f0181dab7f60b4f45a83 (patch) | |
| tree | 00d8a945ac84b400b1ca31adc750ccc8ea92ae3d | |
| parent | d2377cec23dca4b8caab2208767bc23de32ae807 (diff) | |
Fix error handlingricefish_18.0.3ricefish_18.0.2ricefish_18.0.1ricefish_18.0.0ricefish/18.0.3ricefish/18.0.2ricefish/18.0.1ricefish/18.0.0quillback_17.1.4quillback_17.1.3quillback_17.1.2quillback_17.1.1quillback_17.1.0quillback_17.0.2quillback_17.0.1quillback_17.0.0quillback/17.1.4quillback/17.1.3quillback/17.1.2quillback/17.1.1quillback/17.1.0quillback/17.0.2quillback/17.0.1quillback/17.0.0devtool-patcheddevtool-base18.0.318.0.218.0.118.0.017.1.417.1.317.1.217.1.117.1.017.0.217.0.117.0.0
There is a DEBUG logging statement that attempts to pass NULL to
g_variant_print in the error state when connman is not present,
tweak things to avoid that.
Bug-AGL: SPEC-4623
Signed-off-by: Scott Murray <scott.murray@konsulko.com>
Change-Id: I0eeccf332456103dd45ad5e88836cb0484b47e7d
| -rw-r--r-- | src/connman-call.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/connman-call.c b/src/connman-call.c index 0479646..8f87bbb 100644 --- a/src/connman-call.c +++ b/src/connman-call.c @@ -244,10 +244,10 @@ GVariant *connman_get_properties(struct connman_state *ns, else g_set_error(error, CONNMAN_ERROR, CONNMAN_ERROR_ILLEGAL_ARGUMENT, "No %s", access_type); + } else { + DEBUG("properties: %s", g_variant_print(reply, TRUE)); } - DEBUG("properties: %s", g_variant_print(reply, TRUE)); - return reply; } |