Gitiles
Code Review Sign In
review.blissroms.org / platform_packages_modules_DnsResolver / 1d68adbba1fb3af845a1f58846938d671d3e6889
commit1d68adbba1fb3af845a1f58846938d671d3e6889[log] [tgz]
authorZhang Wei-e7976c <e7976c@motorola.com>Fri Aug 23 23:13:22 2019 -0500
committerchenbruce <chenbruce@google.com>Thu Oct 31 18:16:12 2019 +0800
treeceaefb1ae0bfd4a0164baa1752241ea5ccaaa3e3
parent4873cd61ca4b2abba3a504c6752157406baecb8a [diff]
Add private DNS validation thread tracker

The behavior is that the different dns changed in 10ms every 3-5s,
and this dns vibrate will be set by DnsResolver. In ResolverController,
will start one thread for validate each dns server. If dns server removed,
and the trace of dns server will be erased too, but dns validate thread
not destory immediately.
So for this scene will start new thread for dns vibrate in 10ms,
and many thread will cause OOM.

Test:
We verified this solution in that special user environment and merged it
at end of Feb 2019.
We didn't get any regression issue report so far (Sep 2019)

Bug: 139782337

Change-Id: I88431d77f575ccffa5af7c596a915f223e515027
2 files changed
tree: ceaefb1ae0bfd4a0164baa1752241ea5ccaaa3e3
  1. aidl/
  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. DnsResolver.cpp
  14. DnsResolver.h
  15. dnsresolver_binder_test.cpp
  16. DnsResolverService.cpp
  17. DnsResolverService.h
  18. DnsStats.cpp
  19. DnsStats.h
  20. DnsStatsTest.cpp
  21. DnsTlsDispatcher.cpp
  22. DnsTlsDispatcher.h
  23. DnsTlsQueryMap.cpp
  24. DnsTlsQueryMap.h
  25. DnsTlsServer.cpp
  26. DnsTlsServer.h
  27. DnsTlsSessionCache.cpp
  28. DnsTlsSessionCache.h
  29. DnsTlsSocket.cpp
  30. DnsTlsSocket.h
  31. DnsTlsSocketFactory.h
  32. DnsTlsTransport.cpp
  33. DnsTlsTransport.h
  34. getaddrinfo.cpp
  35. getaddrinfo.h
  36. gethnamaddr.cpp
  37. gethnamaddr.h
  38. hostent.h
  39. IDnsTlsSocket.h
  40. IDnsTlsSocketFactory.h
  41. IDnsTlsSocketObserver.h
  42. libnetd_resolv.map.txt
  43. LockedQueue.h
  44. NOTICE
  45. OWNERS
  46. PREUPLOAD.cfg
  47. PrivateDnsConfiguration.cpp
  48. PrivateDnsConfiguration.h
  49. README-DoT.md
  50. README.md
  51. res_cache.cpp
  52. res_comp.cpp
  53. res_debug.cpp
  54. res_debug.h
  55. res_init.cpp
  56. res_init.h
  57. res_mkquery.cpp
  58. res_query.cpp
  59. res_send.cpp
  60. res_send.h
  61. res_stats.cpp
  62. resolv_cache.h
  63. resolv_cache_unit_test.cpp
  64. resolv_integration_test.cpp
  65. resolv_private.h
  66. resolv_tls_unit_test.cpp
  67. resolv_unit_test.cpp
  68. ResolverController.cpp
  69. ResolverController.h
  70. ResolverEventReporter.cpp
  71. ResolverEventReporter.h
  72. ResolverStats.h
  73. sethostent.cpp
  74. stats.proto
  75. TEST_MAPPING
  76. util.cpp
  77. 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.