Gitiles
Code Review Sign In
review.blissroms.org / platform_packages_modules_DnsResolver / 153b5b856ca20430e33b8b96fff094916263e65e
commit153b5b856ca20430e33b8b96fff094916263e65e[log] [tgz]
authorMike Yu <yumike@google.com>Wed Mar 04 19:53:54 2020 +0800
committerMike Yu <yumike@google.com>Wed Mar 18 15:51:08 2020 +0800
tree08cca911dce557a0635f7ce13d858b03fff56ced
parente2162e545a9c6fa882f5adb884914e0318073c8a [diff]
Add tests to cover repeated setResolverConfiguration

Two tests are added to protect the resolver when it continually
receives the same setup requests.

When the resolver receives repeated and same setup:
[1] Do not clear the cache.
[2] Do not clear the stats (the stats for cleartext DNS servers
    got from GetResolverInfo()).
[3] Do not start a new re-evaluation process for the private DNS
    servers if they have been marked as in_progress.
[4] Need not to re-validate the private DNS servers if they have
    been validated.

Another test is added to protect the implementation of aosp/1108695.

Fix: 150678049
Test: resolv_integration_test passed
Change-Id: I7a866e7e305c0fb703ccb9546d1c70ce77e2d3c7
7 files changed
tree: 08cca911dce557a0635f7ce13d858b03fff56ced
  1. aidl_api/
  2. apex/
  3. binder/
  4. include/
  5. tests/
  6. .clang-format ⇨ ../../../build/soong/scripts/system-clang-format
  7. .editorconfig
  8. Android.bp
  9. Dns64Configuration.cpp
  10. Dns64Configuration.h
  11. DnsProxyListener.cpp
  12. DnsProxyListener.h
  13. DnsQueryLog.cpp
  14. DnsQueryLog.h
  15. DnsQueryLogTest.cpp
  16. DnsResolver.cpp
  17. DnsResolver.h
  18. DnsResolverService.cpp
  19. DnsResolverService.h
  20. DnsStats.cpp
  21. DnsStats.h
  22. DnsStatsTest.cpp
  23. DnsTlsDispatcher.cpp
  24. DnsTlsDispatcher.h
  25. DnsTlsQueryMap.cpp
  26. DnsTlsQueryMap.h
  27. DnsTlsServer.cpp
  28. DnsTlsServer.h
  29. DnsTlsSessionCache.cpp
  30. DnsTlsSessionCache.h
  31. DnsTlsSocket.cpp
  32. DnsTlsSocket.h
  33. DnsTlsSocketFactory.h
  34. DnsTlsTransport.cpp
  35. DnsTlsTransport.h
  36. getaddrinfo.cpp
  37. getaddrinfo.h
  38. gethnamaddr.cpp
  39. gethnamaddr.h
  40. hostent.h
  41. IDnsTlsSocket.h
  42. IDnsTlsSocketFactory.h
  43. IDnsTlsSocketObserver.h
  44. libnetd_resolv.map.txt
  45. LockedQueue.h
  46. NOTICE
  47. OWNERS
  48. params.h
  49. PREUPLOAD.cfg
  50. PrivateDnsConfiguration.cpp
  51. PrivateDnsConfiguration.h
  52. README-DoT.md
  53. README.md
  54. res_cache.cpp
  55. res_comp.cpp
  56. res_comp.h
  57. res_debug.cpp
  58. res_debug.h
  59. res_init.cpp
  60. res_init.h
  61. res_mkquery.cpp
  62. res_query.cpp
  63. res_send.cpp
  64. res_send.h
  65. res_stats.cpp
  66. resolv_cache.h
  67. resolv_cache_unit_test.cpp
  68. resolv_callback_unit_test.cpp
  69. resolv_private.h
  70. resolv_tls_unit_test.cpp
  71. resolv_unit_test.cpp
  72. ResolverController.cpp
  73. ResolverController.h
  74. ResolverEventReporter.cpp
  75. ResolverEventReporter.h
  76. ResolverStats.h
  77. sethostent.cpp
  78. stats.h
  79. stats.proto
  80. TEST_MAPPING
  81. util.cpp
  82. util.h
README.md

Logging

This code uses LOG(X) for logging. Log levels are VERBOSE,DEBUG,INFO,WARNING and ERROR. The default setting is WARNING and logs relate to WARNING and ERROR will be shown. If you want to enable the DEBUG level logs, using following command. adb shell service call dnsresolver 10 i32 1 VERBOSE 0 DEBUG 1 INFO 2 WARNING 3 ERROR 4 Verbose resolver logs could contain PII -- do NOT enable in production builds.