[PATCH 1/2] service: Make log output of 'online_check_is_enabled_check' consistent.
Grant Erickson <[email protected]>
| Newsgroups | dev.linux.lists.connman |
|---|---|
| Message-ID | <[email protected]> |
This makes the info-level log output of
'online_check_is_enabled_check' consistent with other messages using
the format of:
... [Ii]nterface <interface name> [ <service type> ] ...
to achieve:
Online check disabled; interface wlan0 [ wifi ] remains in ready
state.
---
src/service.c | 12 ++++++++++--
1 file changed, 10 insertions(+), 2 deletions(-)
diff --git a/src/service.c b/src/service.c
index 99b322a91684..d3bd1fe83b37 100644
--- a/src/service.c
+++ b/src/service.c
@@ -2195,9 +2195,17 @@ static void cancel_online_check(struct connman_service *service,
static bool online_check_is_enabled_check(
const struct connman_service *service)
{
+ g_autofree char *interface = NULL;
+
if (!__connman_service_is_online_check_enabled()) {
- connman_info("Online check disabled. "
- "Default service remains in READY state.");
+ interface = connman_service_get_interface(service);
+
+ connman_info("Online check disabled; "
+ "interface %s [ %s ] remains in %s state.",
+ interface,
+ __connman_service_type2string(service->type),
+ state2string(CONNMAN_SERVICE_STATE_READY));
+
return false;
}
--
2.42.0