aboutsummaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2022-04-08Merge cherrypicks of [17395477] into security-aosp-qt-release.android-security-10.0.0_r75android-security-10.0.0_r74android-security-10.0.0_r73android-security-10.0.0_r72android-security-10.0.0_r71android-security-10.0.0_r70android-security-10.0.0_r69android-security-10.0.0_r68android-security-10.0.0_r67android10-security-releaseAndroid Build Coastguard Worker
Change-Id: I6a65efc1be792cea73686a461cfc0b0c517d0447
2022-04-08Add automatic default value for primitive type fieldsJooyung Han
Primitive type fields are auto-initialized with '\0' when not specified. Bug: 206718630 Bug: 206718630 Test: aidl --lang cpp -o out -h out \ tests/android/aidl/tests/StructuredParcelable.aidl Test: aidl --lang ndk -o out -h out \ tests/android/aidl/tests/StructuredParcelable.aidl Merged-In: I15cfe2a90c0c9adca1692d9a9c997901b90bbbb6 Change-Id: I15cfe2a90c0c9adca1692d9a9c997901b90bbbb6 (cherry picked from commit 94d3c0c503ad14f23ad1dce4bfd8020e031657a4) Merged-In: I15cfe2a90c0c9adca1692d9a9c997901b90bbbb6
2019-06-13Snap for 5656475 from f840ab08799b36d1f95726fc2b24b3a4612b380c to qt-releaseandroid-vts-10.0_r9android-vts-10.0_r8android-vts-10.0_r7android-vts-10.0_r6android-vts-10.0_r5android-vts-10.0_r4android-vts-10.0_r3android-vts-10.0_r2android-vts-10.0_r16android-vts-10.0_r15android-vts-10.0_r14android-vts-10.0_r13android-vts-10.0_r12android-vts-10.0_r11android-vts-10.0_r10android-vts-10.0_r1android-security-10.0.0_r66android-security-10.0.0_r65android-security-10.0.0_r64android-security-10.0.0_r63android-security-10.0.0_r62android-security-10.0.0_r61android-security-10.0.0_r60android-security-10.0.0_r59android-security-10.0.0_r58android-security-10.0.0_r57android-security-10.0.0_r56android-security-10.0.0_r55android-security-10.0.0_r54android-security-10.0.0_r53android-security-10.0.0_r52android-security-10.0.0_r51android-security-10.0.0_r50android-security-10.0.0_r49android-security-10.0.0_r48android-cts-10.0_r9android-cts-10.0_r8android-cts-10.0_r7android-cts-10.0_r6android-cts-10.0_r5android-cts-10.0_r4android-cts-10.0_r3android-cts-10.0_r2android-cts-10.0_r16android-cts-10.0_r15android-cts-10.0_r14android-cts-10.0_r13android-cts-10.0_r12android-cts-10.0_r11android-cts-10.0_r10android-cts-10.0_r1android-10.0.0_r6android-10.0.0_r5android-10.0.0_r47android-10.0.0_r46android-10.0.0_r4android-10.0.0_r3android-10.0.0_r2android-10.0.0_r17android-10.0.0_r11android-10.0.0_r10android-10.0.0_r1android10-tests-releaseandroid10-s3-releaseandroid10-s2-releaseandroid10-s1-releaseandroid10-releaseandroid-build-team Robot
Change-Id: I84147691e9b2282791640ba3b2f29d5f3f393e73
2019-06-12fix: incorrect output pathJiyong Park
This change fixes a bug that when -o option is used without explicitly specifying the output path, the output path is determined by concatenating the directory specified by the -o option and the input file path and the replacing the suffix .aidl to .java. This especially is problematic in Windows, because the output path then contains drive letter (e.g. "C:") in the middle of the path if the input path is an absolute path. For example, aidl -o C:\output C:\input\IFoo.aidl resulted the incorrect output path of C:\output\C:\input\IFoo.java. It should be C:\input\IFoo.java This change fixes the problem by not concatenating the output dir with the input file path. Bug: 134966834 Test: aidl_unittests Merged-In: I417623089f7409032483a2f6c79e745c0fc340d3 (cherry picked from commit 56f73d76e064a0bba1a103e5c37bc3d5d57860f9) Change-Id: I417623089f7409032483a2f6c79e745c0fc340d3
2019-06-08Snap for 5645017 from e2faf9c0d8db8c6af1dbafd95447d1c8e8cef085 to qt-releaseandroid-build-team Robot
Change-Id: Id471f7d6f9baaacf96d57b39e834cf5cd6457342
2019-06-07AIDL API check ignores blank lines and commentsJiyong Park
The API signature file now contains warning preamble as comments. The comments shouldn't fail 'diff'. Bug: 134452637 Test: m out/soong/.intermediates/frameworks/base/services/net/ipmemorystore-aidl-interfaces-api/checkapi_dump.timestamp with ag/7952481 applied in qt-dev. Merged-In: I5547b768c216ee67e7d2d3b82752a94fea27004c (cherry picked from commit 8558309645adf15978321528b71a60e45731a250) Change-Id: I5547b768c216ee67e7d2d3b82752a94fea27004c
2019-06-06Make it difficult to force update AIDL API signature filesJiyong Park
We have found that people tend to ignore build errors and force update the frozen AIDL API signature files. To better prevent it from happening, this change - Makes the error message more visible and explicit. The message also explains why force updating the API signature files are not allowed. - Put similar warning messages to the API signature files as comment when the new signature API files are created. Bug: 134452637 Test: add a random method to IBaz.aidl and run `m test-root-package` "ERROR: AIDL API change detected on a released platform" is shown. Test: `m test-root-package-freeze-api` build/api/3/test_package/IBaz.aidl contains the warning message (cherry picked from commit f1bc25ed27703e69ff87ecc1c59e204eac2297cf) Merged-In: I860620daa61770a68990456fafd9e5950fde9405 Change-Id: I860620daa61770a68990456fafd9e5950fde9405
2019-05-21Snap for 5587110 from e58e2aec064a5c4f7e6c776597fdbacf79f3281c to qt-releaseandroid-build-team Robot
Change-Id: Id2fff82bfa216625303dc9d733c2cfc0a61de0a5
2019-05-20Fix getInterfaceVersion() inconsistency bug between backendsJeongik Cha
Java: - Server writeNoException(), and then writeInt(version) - Client As-is: just readInt() -> read exception code, not version To-be: readException, and then readInt() -> consume exception code first Cpp: - Transaction code As-is: getInterface's ID To-be: FIRST_CALL_TRANSACTION + the ID - Server & Client As-is: No exception read/write before version code To-be: Add exception value before version code Bug: 133118233 Test: m Test: atest CtsNdkBinderTestCases Test: ./runtests.sh Change-Id: I168a0b8ff7e1871afd95a0ab76b544bfe131a9ed Merged-In: I168a0b8ff7e1871afd95a0ab76b544bfe131a9ed
2019-05-11Snap for 5558509 from 1844e2f3dc98ec3dc1fa46d4db594865891163e4 to qt-releaseandroid-build-team Robot
Change-Id: Ic4048f739b26e34aa6d55444e239d95e4e5ea5dd
2019-05-09Merge changes from topic "b/126501406" into qt-devLuke Huang
* changes: AIDL NDK logger bugfix Log supporting for binder ndk
2019-05-07AIDL NDK logger bugfixJeongik Cha
1. When the error occurs in server side, it exits before log, so make logger called before exit. 2. service_specific_code had wrong data, so change it to valid code. Bug: 126501406 Test: m Change-Id: Ibbb072a067ab2c16f3fdefcc777c4900fb8492ae Merged-In: Ibbb072a067ab2c16f3fdefcc777c4900fb8492ae
2019-05-07Log supporting for binder ndkJeongik Cha
Move log code Generating code aidl_to_cpp -> aidl_to_common Add dependency of libjsoncpp(_ndk) -ndk: libjsoncpp_ndk -ndk_platform: libjsoncpp Bug: 126501406 Test: m Test: ./runtests.sh Change-Id: Ife69f146ce313564082d070e5e047ec8e1111093 Merged-In: Ife69f146ce313564082d070e5e047ec8e1111093
2019-05-04Snap for 5533229 from e308abd33006e46fc978ff7881bf43222afbca95 to qt-releaseandroid-build-team Robot
Change-Id: Ibfc86f5532ce67eac97a0aa51199a87d46d1ca0c
2019-05-03Update build test APIs.Steven Moreland
APIs must match ToT so that we know they don't change and we can rely on them. However, this is for a test, so it doesn't matter in practice. This moves forward the release process, but we should look at avoiding needing test APIs frozen. Bug: 131924172 Test: force checkEquality to be called in aidl_interface.go, repro build error reported by release team, update APIs to fix error Change-Id: Iabec6e853655b5f535b35f6e16b53f782f609b24
2019-04-25Snap for 5503058 from d7b758db94de2a6449f5315ec223a9e8b73cdea9 to qt-releaseandroid-build-team Robot
Change-Id: I8608bc6fcdcd58863e817fc2d864ac852f8ba077
2019-04-24Merge "Don't allow modification to the AIDL interfaces when SDK is ↵TreeHugger Robot
finalized" into qt-dev
2019-04-24Ignore ParcelFileDescriptor in framework.aidlJeongik Cha
ParcelFileDescriptor is treated as a built-in type, but it's also in the framework.aidl. So aidl ignores its declaration in framework.aidl to prevent duplication Bug: 130899491 Test: m Test: check if "attempt to redefine built in class android.os.ParcelFileDescriptor" doesn't show Change-Id: Iffcd6f6a5348abb59a2df481883ca48493f2ed66 Merged-In: Iae33b1543f81fdc68ee20588f705f03204c14502
2019-04-23Don't allow modification to the AIDL interfaces when SDK is finalizedJiyong Park
When the SDK is finalized, modification (including backward compatible changes) to the AIDL interfaces is prohibited. Modification can only be made in the non-finalized branch, e.g. master, by introducing a new version. Bug: 130011602 Test: manual Set PLATFORM_VERSION_CODENAME to REL and `m netd_aidl_interface-cpp` Fails with following error. Check equality of out/soong/.intermediates/system/netd/server/netd_aidl_interface-api/dump and system/netd/server/binder/api/1 FAILED: out/soong/.intermediates/system/netd/server/netd_aidl_interface-api/checkapi_dump.timestamp diff -r system/netd/server/binder/api/1 out/soong/.intermediates/system/netd/server/netd_aidl_interface-api/dump && touch out/soong/.intermediates/system/netd/server/netd_aidl_interface-api/checkapi_dump.timestamp diff -r system/netd/server/binder/api/1/android/net/INetd.aidl out/soong/.intermediates/system/netd/server/netd_aidl_interface-api/dump/android/net/INetd.aidl 92a93,96 > void firewallAddUidInterfaceRules(in @utf8InCpp String ifName, in int[] uids); > void firewallRemoveUidInterfaceRules(in int[] uids); > void trafficSwapActiveStatsMap(); Merged-In: I6229e6e6d19153564939e9fb9ea19a1177c2279c Change-Id: I6229e6e6d19153564939e9fb9ea19a1177c2279c (cherry picked from commit 7c3c11c36a6b0b39c565b009a50fdfb0f7fe6889)
2019-04-15Snap for 5470312 from 13824f236b906d5cc1d718002c173e4356f0dbe2 to qt-releaseandroid-build-team Robot
Change-Id: I39af33f3b957951013b00555feb7f0bfbe7a4c44
2019-04-15Merge "fix: can't assign method IDs when built with --version" into qt-devTreeHugger Robot
2019-04-12Merge "Hide getDefaultTransactionName and getTransactionName" into qt-devTreeHugger Robot
2019-04-11Hide getDefaultTransactionName and getTransactionNameJiyong Park
The two methods are now with @hide as they are not intended for public use. Bug: 124210145 Test: m Test: aidl_unittests Change-Id: I8665f11e8912159d37f0d54e643df25cdacf9e16
2019-04-11fix: can't assign method IDs when built with --versionJiyong Park
Fix a bug that a meta transaction incorrectly turns hasUnassignedIds on. Also add a few more unit tests for the ID assignments. Bug: 130328570 Test: aidl_unittests Merged-In: I375ec8fdf9f633bc71f7d5d03f564789f563b30e Change-Id: I375ec8fdf9f633bc71f7d5d03f564789f563b30e (cherry picked from commit a1bdb7364a2d2e9c417686b4b84cba443e2d1bca)
2019-04-10Snap for 5452883 from 0dc10177706b9dd818e2ef12a01f17eeb663cf17 to qt-releaseandroid-build-team Robot
Change-Id: I4a6cce6a425f5898538394af0b2c6fa70cb3485d
2019-04-10Implement getInterfaceVersion for C++Jiyong Park
This change fixes a bug that getInterfaceVerison() is not implemented in the stub class when --lang=cpp and --version is set. Bug: 130195984 Test: m Test: aidl_unittests Merged-In: I2b107db8c47ef287ec2be736b6beb84768cb04e6 Change-Id: I2b107db8c47ef287ec2be736b6beb84768cb04e6 (cherry picked from commit 22d4cfc8fbef14080ce082e9e387ed6da2fa1321)
2019-04-09Snap for 5447940 from 545f44dc4c6ec299b4745e74d6589ab9e7e2eec4 to qt-releaseandroid-build-team Robot
Change-Id: Iee0a1d1399e507c9f0107148eafc9ab11377d258
2019-04-08Merge "Change parameters in log data to array of pairs" into qt-devTreeHugger Robot
2019-04-08Change parameters in log data to array of pairsJeongik Cha
As-is: Object type {a:1, b:2, c:3} To-be: Array type [{name: a, value:1}, {name:b, value:2}, {name:c, value:3}] Object type doesn't keep insertion order. So change it to array type Bug: 128833454 Test: m -j Change-Id: Ic7c36560de03a4f2bbefef3c4a0a790ce4b90869 Merged-In: Ic7c36560de03a4f2bbefef3c4a0a790ce4b90869 (cherry picked from commit 2a2581ea3a58c92a429db85485e74c692dd99de4)
2019-04-08Simplify paths for the generated source filesJiyong Park
Previously, the generated source files of an aidl_interface were found under: out/soong/.intermediate/<path_to_module>/<module>-cpp-gen-<number>/gen , where <number> is assigned arbitrarily. This caused a problem when the paths are added to the source path of an IDE. The paths had to be updated whenever the assigned numbers are changed. Now, the paths get simpler: out/soong/.intermediate/<path_to_module>/<module>-cpp-source/gen This is done by building multiple source files using a single genrule module. Bug: 130011113 Test: m Test: system/tools/aidl/runtests.sh Merged-In: I308f3c3ced5b2f7ca3a95ea3fd8ab49d8dd9105b Change-Id: I308f3c3ced5b2f7ca3a95ea3fd8ab49d8dd9105b (cherry picked from commit b8dceed62c9383b50b4a1f2525c49d78d152b3cf)
2019-04-06Snap for 5443223 from 081b90a81b3b6d7ccd740fd87faf16b9259f4dab to qt-releaseandroid-build-team Robot
Change-Id: I938d7ad0dbfea95c02ed7e67a448ae0fb2d2a552
2019-04-05Revert "Revert "Don't force prepend I to header file names""Jiyong Park
This reverts commit cc14a584a301533653639d9f6847111ca8a79c68. Bug: 129549755 Test: m Test: m libvirtualtouchpadclient Test: ./runtests.sh Merged-In: Id0103c1f05654c87a3eac449c985c805ae963d0e Change-Id: Id0103c1f05654c87a3eac449c985c805ae963d0e (cherry picked from commit 5b7e532899a56e88e5fbf53ffd6686d941be7da9)
2019-04-04Snap for 5434517 from 9396d8e69044c140e2a06a2d9ea369099285fe52 to qt-releaseandroid-build-team Robot
Change-Id: I31bd84db4f3674a0efeecdddcbe04f60d57062a9
2019-04-03Revert "Don't force prepend I to header file names" am: cc14a584a3 am: ↵Neil Fuller
2133f7fdd0 am: 4aa7867b72 Change-Id: I969ee261037d1baa9e2e9227b10bc428b9b200ad
2019-04-03Revert "Don't force prepend I to header file names" am: cc14a584a3Neil Fuller
am: 2133f7fdd0 Change-Id: I3638bfa98080d8f66b0726c06efde9b511e2c301
2019-04-03Revert "Don't force prepend I to header file names"Neil Fuller
am: cc14a584a3 Change-Id: I7d023dd58facc31a55154086ad1036fd64568f30
2019-04-03Revert "Don't force prepend I to header file names"Neil Fuller
This reverts commit 7df2c34b6e40828fdf362c665911173e188fe1c4. Reason for revert: Possible cause of break in stage-aosp-master Change-Id: I89f2c1c5b645c2434eaac149c32b8220555225e4
2019-04-03Don't force prepend I to header file names am: 7df2c34b6e am: acc4190f0aJiyong Park
am: 1b0ba5eb7a Change-Id: Ic51874427d4e0603179db35d273cd47d22412bcc
2019-04-03Don't force prepend I to header file names am: 7df2c34b6eJiyong Park
am: acc4190f0a Change-Id: I9e928e1bcb89606d35f05227f9f904f9ef670bb4
2019-04-03Don't force prepend I to header file namesJiyong Park
am: 7df2c34b6e Change-Id: Iaec077af6a9e114a7e6881f2294e279b31e220ab
2019-04-02Don't force prepend I to header file namesJiyong Park
This is to be in sync with the rule in aidl_interface.go. Bug: 129549755 Test: m Change-Id: I5bd5bec380991dd6340d569706198242f7ad3d79
2019-04-02Snap for 5426495 from ffa0ec2691d5008f9dd2bc15feb4748e633f0133 to qt-releaseandroid-build-team Robot
Change-Id: Ic66df508c6a6a7720954cd2eece4e9d679ced128
2019-04-01Modernise code to use override specifier am: de138911c8 am: ef9d233b48Yi Kong
am: abc414c0dd Change-Id: Id9dbf76ec06fb095b90f1745b2f420ca6d0b166c
2019-04-01Modernise code to use override specifier am: de138911c8Yi Kong
am: ef9d233b48 Change-Id: Ifdb51337efe2535486bc3cf6a42374dd5696caec
2019-04-01Modernise code to use override specifierYi Kong
am: de138911c8 Change-Id: I9167c72fd1bc646a1d266dd32b6470fe101074e9
2019-03-30Snap for 5422062 from 9bf8404f7b8c908be846eda1e0c8e6886ea0bfee to qt-releaseandroid-build-team Robot
Change-Id: I54debe9bd02ac69ed6899a98e14cf2f3704d2385
2019-03-30Modernise code to use override specifierYi Kong
Generated by clang-tidy. Test: m checkbuild Change-Id: I9edd67d241453ebca9a3e84c72e2b122e72cd5df
2019-03-29Rename LoggableInterface to ILoggableInterface am: 6e4353264f am: 09e3c582edSteven Moreland
am: 8123a1d001 Change-Id: I8499967f157de347b5b8696554d65f06353ac2c8
2019-03-29Rename LoggableInterface to ILoggableInterface am: 6e4353264fSteven Moreland
am: 09e3c582ed Change-Id: I35efbcba5298acd2d850584bfea2ec5600866bad
2019-03-29Rename LoggableInterface to ILoggableInterfaceSteven Moreland
am: 6e4353264f Change-Id: I85b04ccb763f0bbc776ee3e5293102984da00854