diff mbox series

[1/3] netconfig: Drop D-Bus path from debug message

Message ID 20220920140509.3304422-1-andrew.zaborowski@intel.com (mailing list archive)
State Accepted, archived
Headers show
Series [1/3] netconfig: Drop D-Bus path from debug message | expand

Checks

Context Check Description
tedd_an/pre-ci_am success Success
prestwoj/iwd-alpine-ci-fetch success Fetch PR
prestwoj/iwd-ci-fetch success Fetch PR
prestwoj/iwd-ci-gitlint success GitLint
prestwoj/iwd-ci-makedistcheck success Make Distcheck
prestwoj/iwd-ci-build success Build - Configure
prestwoj/iwd-alpine-ci-makedistcheck success Make Distcheck
prestwoj/iwd-alpine-ci-build success Build - Configure
prestwoj/iwd-ci-clang success clang PASS
prestwoj/iwd-ci-makecheckvalgrind success Make Check w/Valgrind
prestwoj/iwd-ci-makecheck success Make Check
prestwoj/iwd-ci-incremental_build success Incremental Build with patches
prestwoj/iwd-alpine-ci-makecheckvalgrind success Make Check w/Valgrind
prestwoj/iwd-alpine-ci-makecheck success Make Check
prestwoj/iwd-alpine-ci-incremental_build success Incremental Build with patches
prestwoj/iwd-ci-testrunner fail test-runner - FAIL:

Commit Message

Andrew Zaborowski Sept. 20, 2022, 2:05 p.m. UTC 
l_dbus_message_get_path(reply) is normally NULL in this callback so
don't bother printing it.
---
 src/netconfig-commit.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

Comments

Denis Kenzior Sept. 20, 2022, 3:07 p.m. UTC | #1 
Hi Andrew,

On 9/20/22 09:05, Andrew Zaborowski wrote:
> l_dbus_message_get_path(reply) is normally NULL in this callback so
> don't bother printing it.
> ---
>   src/netconfig-commit.c | 3 +--
>   1 file changed, 1 insertion(+), 2 deletions(-)
> 

All applied, thanks.

Regards,
-Denis
diff mbox series

Patch

diff --git a/src/netconfig-commit.c b/src/netconfig-commit.c
index 9894ace2..96436c1c 100644
--- a/src/netconfig-commit.c
+++ b/src/netconfig-commit.c
@@ -373,8 +373,7 @@  static void netconfig_agent_receive_reply(struct l_dbus_message *reply,
 	const char *error, *text;
 	bool success = true;
 
-	l_debug("agent reply from %s %s", l_dbus_message_get_sender(reply),
-		l_dbus_message_get_path(reply));
+	l_debug("agent reply from %s", l_dbus_message_get_sender(reply));
 
 	data->pending_id[INDEX_FOR_AF(cd->family)] = 0;