Gitiles
Code Review Sign In
review.blissroms.org / platform_packages_modules_adb / acbe1e225d136a675e698ab5f6cf1e4d971a1588
commitacbe1e225d136a675e698ab5f6cf1e4d971a1588[log] [tgz]
authorGreg Kaiser <gkaiser@google.com>Mon Mar 18 07:00:00 2019 -0700
committerGreg Kaiser <gkaiser@google.com>Mon Mar 18 07:00:00 2019 -0700
treeaf6b005b9dec8c8946496dcea5c4599196b96483
parentd2fbd5cd896ffaf7b6778e4f6f700ee504bf90f4 [diff]
adb: Fix return value in error case

When we introduced __adb_check_server_version() as a function
returning a 'bool', we missed missed converting once instance
of "return -1;" to "return false;".  Thus, we're returning 'true'
in this case as our non-zero value gets implicitly converted to
'true'.

We fix this case as well with this CL.

Test: TreeHugger
Change-Id: I90b01567d927f36d2d963561b676b3bfabfcee49
1 file changed
tree: af6b005b9dec8c8946496dcea5c4599196b96483
  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