This could introduce a need to null-check values where this function
is used (src/dhcp.c, src/network.c, src/provider.c). However, src/dhcp.c
requires that URL can be cleared.
---
 src/service.c |    3 ---
 1 files changed, 0 insertions(+), 3 deletions(-)

diff --git a/src/service.c b/src/service.c
index d442c16..05925b7 100644
--- a/src/service.c
+++ b/src/service.c
@@ -2026,9 +2026,6 @@ int __connman_service_timeserver_remove(struct 
connman_service *service,
 void __connman_service_set_pac(struct connman_service *service,
                                        const char *pac)
 {
-       if (pac == NULL)
-               return;
-
        g_free(service->pac);
        service->pac = g_strdup(pac);
 }
-- 
1.7.0.4


_______________________________________________
connman mailing list
connman@connman.net
http://lists.connman.net/listinfo/connman

Reply via email to