Gitiles
Code Review Sign In
review.blissroms.org / platform_packages_modules_adb / 3770507114f64200c5177e0b86a413a79eac4417
commit3770507114f64200c5177e0b86a413a79eac4417[log] [tgz]
authorJosh Gao <jmgao@google.com>Tue Mar 05 13:01:25 2019 -0800
committerJosh Gao <jmgao@google.com>Wed Mar 06 13:03:30 2019 -0800
tree621fc2618c8786152db3b4bf044796e5df0b3a09
parent84c409d1f90ebf04f8d33abee6d190cb83445557 [diff]
adbd: cleanups in remount/verity.

1. Don't clear O_CLOEXEC on an fd we're duping to stdin/out/err: dup2
   does that for us if oldfd != newfd, and we know the fd can't be
   stdin/out/err.
2. Use unique_fd.
3. Remove gotos by breaking the remount logic into its own function.

Test: mma
Change-Id: If9929e7f20de445322fd498e1379a2d4e572cb52
2 files changed
tree: 621fc2618c8786152db3b4bf044796e5df0b3a09
  1. client/
  2. daemon/
  3. fastdeploy/
  4. sysdeps/
  5. .clang-format ⇨ ../.clang-format-4
  6. adb.bash
  7. adb.cpp
  8. adb.h
  9. adb_auth.h
  10. adb_integration_test_adb.xml
  11. adb_integration_test_device.xml
  12. adb_io.cpp
  13. adb_io.h
  14. adb_io_test.cpp
  15. adb_listeners.cpp
  16. adb_listeners.h
  17. adb_listeners_test.cpp
  18. adb_mdns.h
  19. adb_trace.cpp
  20. adb_trace.h
  21. adb_unique_fd.cpp
  22. adb_unique_fd.h
  23. adb_utils.cpp
  24. adb_utils.h
  25. adb_utils_test.cpp
  26. Android.bp
  27. benchmark_device.py
  28. bugreport_test.cpp
  29. fdevent.cpp
  30. fdevent.h
  31. fdevent_test.cpp
  32. fdevent_test.h
  33. file_sync_protocol.h
  34. MODULE_LICENSE_APACHE2
  35. NOTICE
  36. OVERVIEW.TXT
  37. OWNERS
  38. protocol.txt
  39. security_log_tags.h
  40. services.cpp
  41. services.h
  42. SERVICES.TXT
  43. shell_protocol.h
  44. shell_service_protocol.cpp
  45. shell_service_protocol_test.cpp
  46. socket.h
  47. socket_spec.cpp
  48. socket_spec.h
  49. socket_spec_test.cpp
  50. socket_test.cpp
  51. sockets.cpp
  52. sockets.dia
  53. SYNC.TXT
  54. sysdeps.h
  55. sysdeps_test.cpp
  56. sysdeps_unix.cpp
  57. sysdeps_win32.cpp
  58. sysdeps_win32_test.cpp
  59. test_adb.py
  60. test_device.py
  61. trace.sh
  62. transport.cpp
  63. transport.h
  64. transport_benchmark.cpp
  65. transport_fd.cpp
  66. transport_local.cpp
  67. transport_test.cpp
  68. transport_usb.cpp
  69. types.h
  70. types_test.cpp
  71. usb.h