Avoid an invalid read by moving debug message after
sanity check.
---
 src/service.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/service.c b/src/service.c
index f5c0a23..d5f95ed 100644
--- a/src/service.c
+++ b/src/service.c
@@ -4358,13 +4358,13 @@ static void service_schedule_added(struct 
connman_service *service)
 
 static void service_schedule_removed(struct connman_service *service)
 {
-       DBG("service %p %s", service, service->path);
-
        if (!service || !service->path) {
                DBG("service %p or path is NULL", service);
                return;
        }
 
+       DBG("service %p %s", service, service->path);
+
        g_hash_table_remove(services_notify->add, service->path);
        g_hash_table_replace(services_notify->remove, g_strdup(service->path),
                        NULL);
-- 
1.9.1

_______________________________________________
connman mailing list
connman@connman.net
https://lists.connman.net/mailman/listinfo/connman

Reply via email to