From 4f9158119b51f6d6f828bbd7b6b76cff355a98fc Mon Sep 17 00:00:00 2001 From: Elliott Hughes Date: Mon, 5 Dec 2016 13:12:48 -0800 Subject: [PATCH] The "net.change" property is no longer needed. bionic was the only listener, and it was doing nothing useful when the property changed, so we've removed that listener. Bug: http://b/33308258 Test: DNS still works Change-Id: I15292e8e58e6b87cdad8f73e449ccaf334dff68b --- init/property_service.cpp | 15 +++------------ 1 file changed, 3 insertions(+), 12 deletions(-) diff --git a/init/property_service.cpp b/init/property_service.cpp index aed84386d..498a5a103 100644 --- a/init/property_service.cpp +++ b/init/property_service.cpp @@ -207,18 +207,9 @@ int property_set(const char* name, const char* value) { } } - // If name starts with "net." treat as a DNS property. - if (strncmp("net.", name, strlen("net.")) == 0) { - if (strcmp("net.change", name) == 0) { - return 0; - } - // The 'net.change' property is a special property used track when any - // 'net.*' property name is updated. It is _ONLY_ updated here. Its value - // contains the last updated 'net.*' property. - property_set("net.change", name); - } else if (persistent_properties_loaded && strncmp("persist.", name, strlen("persist.")) == 0) { - // Don't write properties to disk until after we have read all default properties - // to prevent them from being overwritten by default values. + // Don't write properties to disk until after we have read all default + // properties to prevent them from being overwritten by default values. + if (persistent_properties_loaded && strncmp("persist.", name, strlen("persist.")) == 0) { write_persistent_property(name, value); } property_changed(name, value);