Gitiles
Code Review Sign In
gerrit-public.fairphone.software / platform / system / tools / aidl / f2551d82b776f2e524af13db35b7a546e1190c9c
commitf2551d82b776f2e524af13db35b7a546e1190c9c[log] [tgz]
authorKevin Jeon <kevinjeon@google.com>Tue Jul 27 16:06:07 2021 +0000
committerKevin Jeon <kevinjeon@google.com>Wed Aug 04 01:06:44 2021 +0000
tree5cd118c771e9569cbddfd7b9b21cb3dc1a1a75e3
parent1f3d8f76e9294908711ca72da1f114f4148b3d81 [diff]
Hoist enforceInterface and escape onTransact

This change adds as check for user-defined transactions in order to call
enforceInterface before checking for different case in onTransact. It
also adds break statements to onTransact.

'system/tools/aidl/tests/golden_test.sh update' was used to update the
reference outputs.

Test: - builds and boots
      - atest aidl_unittests CtsNdkBinderTestCases -m -c
      - cd out/target/product/redfin/symbols/system/framework/arm64;
        nm --size-sort -Sr boot-framework.oat | grep onTransact
      - cd out/target/product/redfin/system/framework;
        ./apkanalyzer dex packages framework.jar | grep 'onTransact' |
        sort -r -g -k 5,5 | awk '{$1=$2=$3=$4=""; print $0}')
Bug: 194712114
Change-Id: Ib13b0f694bd3bf151677abe98cdd3368c873beaf
8 files changed
tree: 5cd118c771e9569cbddfd7b9b21cb3dc1a1a75e3
  1. aidl_api/
  2. build/
  3. metadata/
  4. tests/
  5. trace/
  6. .clang-format
  7. aidl-format.sh
  8. aidl.cpp
  9. aidl.h
  10. aidl_checkapi.cpp
  11. aidl_checkapi.h
  12. aidl_const_expressions.cpp
  13. aidl_dumpapi.cpp
  14. aidl_dumpapi.h
  15. aidl_language.cpp
  16. aidl_language.h
  17. aidl_language_l.ll
  18. aidl_language_y.yy
  19. aidl_to_cpp.cpp
  20. aidl_to_cpp.h
  21. aidl_to_cpp_common.cpp
  22. aidl_to_cpp_common.h
  23. aidl_to_java.cpp
  24. aidl_to_java.h
  25. aidl_to_ndk.cpp
  26. aidl_to_ndk.h
  27. aidl_to_rust.cpp
  28. aidl_to_rust.h
  29. aidl_typenames.cpp
  30. aidl_typenames.h
  31. aidl_unittest.cpp
  32. Android.bp
  33. ast_cpp.cpp
  34. ast_cpp.h
  35. ast_cpp_unittest.cpp
  36. ast_java.cpp
  37. ast_java.h
  38. ast_java_unittest.cpp
  39. code_writer.cpp
  40. code_writer.h
  41. code_writer_unittest.cpp
  42. comments.cpp
  43. comments.h
  44. diagnostics.cpp
  45. diagnostics.h
  46. diagnostics.inc
  47. diagnostics_unittest.cpp
  48. generate_aidl_mappings.cpp
  49. generate_aidl_mappings.h
  50. generate_cpp.cpp
  51. generate_cpp.h
  52. generate_cpp_unittest.cpp
  53. generate_java.cpp
  54. generate_java.h
  55. generate_java_binder.cpp
  56. generate_ndk.cpp
  57. generate_ndk.h
  58. generate_rust.cpp
  59. generate_rust.h
  60. hiddenapi-greylist
  61. import_resolver.cpp
  62. import_resolver.h
  63. io_delegate.cpp
  64. io_delegate.h
  65. io_delegate_unittest.cpp
  66. location.cpp
  67. location.h
  68. logging.cpp
  69. logging.h
  70. main.cpp
  71. NOTICE
  72. options.cpp
  73. options.h
  74. options_unittest.cpp
  75. os.h
  76. OWNERS
  77. parser.cpp
  78. parser.h
  79. preprocess.cpp
  80. preprocess.h
  81. PREUPLOAD.cfg
  82. README.md
  83. run_integration_tests.sh
  84. TEST_MAPPING
README.md

Documentation for this project is currently maintained here:

https://source.android.com/devices/architecture/aidl/overview