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