Gitiles
Code Review Sign In
gerrit-public.fairphone.software / platform / system / tools / aidl / 05a532aee0465fe0c170f1968d644c67d6fbeac4
commit05a532aee0465fe0c170f1968d644c67d6fbeac4[log] [tgz]
authorJooyung Han <jooyung@google.com>Wed Aug 18 10:41:40 2021 +0900
committerJooyung Han <jooyung@google.com>Wed Aug 18 10:53:28 2021 +0900
treef50b5233a35f77305ae09c03e801c2269d749f8b
parent01720ed508044261ff672dac8466f65d023912cd [diff]
Fix --apimapping with unstructured parcelables

This fix consists of several related smaller fixes:
- --apimapping accepts positions arguments as inputs.
- --apimapping is a top-level command in HELP MSG.
- FOUND_PARCELABLE is handled as error consistently when -b is set.
- typo in aidl_parser_fuzzer.dict

Previously, FOUND_PARCELABLE error was returned from
load_and_validate_aidl and handled by callers. Treating it as error or
success was not consistent in various places.

Now, FOUND_PARCELABLE is returned only if -b is turned on and it is
always treated as an error by callers.

This fixes the crash that happens when --apimapping is called with
unstructured parcelables.

Bug: 196686928
Test: aidl_unittests
Test: m
Change-Id: I95cc9cc556825dc78f975d1f092b6420ec5399f8
5 files changed
tree: f50b5233a35f77305ae09c03e801c2269d749f8b
  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