commit bab5014cbf038e123519622248b59be83fd8af8b
author Ken Chen <cken@google.com> Tue Mar 19 17:41:28 2019 +0800
committer Ken Chen <cken@google.com> Thu Mar 21 17:12:48 2019 +0800
tree b981397f2bdd63c133f250b420be037cccae90d5
parent af80d830e9ded5bbdb34778d34531917646ec608

Disable sensitive logs

- PII logs can only appear in VERBOSE level
- logSeverityStrToEnum() no more support VERBOSE level input, only
accept DEBUG, INFO, WARNING, and ERROR.
- developer can set DBG flag from code to have a debug build, the DEBUG
level is automatically promote to VERBOSE.
- uniform log format to [FILE NAME]: [FUNC NAME]: [MSG]
- move from ALOG to LOG on DnsProxyListener
- adjust severity for some logs
- correct print format on uint8_t type

Bug: 128736560
Test: builds, boots
Test: atest resolv_integration_test
Change-Id: I0ff03824901168165bbe1f5abae9ff3e74db63d6

resolv/res_init.cpp

diff --git a/resolv/res_init.cpp b/resolv/res_init.cpp
index 7f11ce6..ceb3023 100644
--- a/resolv/res_init.cpp
+++ b/resolv/res_init.cpp
@@ -187,7 +187,7 @@
             dots--;
         }
         *pp = NULL;
-        LOG(DEBUG) << "res_init(): dnsrch list:";
+        LOG(DEBUG) << __func__ << ": dnsrch list:";
         for (pp = statp->dnsrch; *pp; pp++) LOG(DEBUG) << "\t" << *pp;
     }
 
@@ -204,7 +204,7 @@
     int i;
     res_state_ext* ext = statp->_u._ext.ext;
 
-    LOG(DEBUG) << ";; res_setoptions(\"" << options << "\", \"" << source << "\")...";
+    LOG(DEBUG) << "res_setoptions(\"" << options << "\", \"" << source << "\")...";
 
     while (*cp) {
         /* skip leading and inner runs of spaces */
@@ -216,14 +216,14 @@
                 statp->ndots = i;
             else
                 statp->ndots = RES_MAXNDOTS;
-            LOG(DEBUG) << ";;\tndots=" << statp->ndots;
+            LOG(DEBUG) << "\tndots=" << statp->ndots;
 
         } else if (!strncmp(cp, "debug", sizeof("debug") - 1)) {
             if (!(statp->options & RES_DEBUG)) {
-                LOG(DEBUG) << ";; res_setoptions(\"" << options << "\", \"" << source << "\")..";
+                LOG(DEBUG) << "res_setoptions(\"" << options << "\", \"" << source << "\")..";
                 statp->options |= RES_DEBUG;
             }
-            LOG(DEBUG) << ";;\tdebug";
+            LOG(DEBUG) << "\tdebug";
 
         } else if (!strncmp(cp, "no_tld_query", sizeof("no_tld_query") - 1) ||
                    !strncmp(cp, "no-tld-query", sizeof("no-tld-query") - 1)) {