summaryrefslogtreecommitdiff
path: root/libs
AgeCommit message (Collapse)Author
2025-09-18[SP 2025-09-01] RPC Binder: clearer errors for wrong transact type Steven Moreland
Bug: 423127919 # this bug error is totally different than this case. If we hit this case, we would have gotten a different harder error. However, I found this while looking at this bug. Bug: 416734088 Test: binderRpcTest Flag: EXEMPT bug fix Change-Id: I3e0db452ebeed1a520ffc0181ed3577eb38bea9a Merged-In: I3e0db452ebeed1a520ffc0181ed3577eb38bea9a (cherry picked from commit 51ed14eebf5a6b7e0d1d6f5b9c81b01a666b270a)
2025-09-18[SP 2025-09-01] RPC Binder: shutdown on SPAN error. Steven Moreland
This error return is not recoverable. Bug: 416734088 Test: binderRpcTest Change-Id: If0b8a8f36f797dcf927bfc2b5ae51e37e915f2f6 Merged-In: If0b8a8f36f797dcf927bfc2b5ae51e37e915f2f6 (cherry picked from commit dddbc115f88d99379ad9e118e551924c84f16e3a)
2025-09-18[SP 2025-09-01] RPC Binder: shutdown on ENOMEM Steven Moreland
We were expecting crashes in these case, but explicitly shut down. More work needs to be done to ignore transactions in out of memory conditions. Bug: 404210068 Bug: 414720799 Bug: 416734088 Test: binderRpcTest Change-Id: Iaf9a34b4031fb7b9807c962bcc67de8cd9102088 Merged-In: Iaf9a34b4031fb7b9807c962bcc67de8cd9102088 (cherry picked from commit cba4e3642a8a58d54481ed4c14f179bcc7f9ae70)
2025-09-18[SP 2025-09-01] Protect objects in Parcel::appendFrom Devin Moore
* only aquire objects within the range supplied to appendFrom * don't append over existing objects * unset the mObjectsSorted flag a couple more cases * keep mObjectPositions sorted Flag: EXEMPT bug fix Ignore-AOSP-First: security fix Test: binder_parcel_fuzzer Bug: 402319736 Change-Id: I63715fdd81781aaf04f5fc0cb8bdb74c09d5d807 (cherry picked from commit 28e7af08b92e7b97f46d8ecd88ebd3f27a065e08)
2025-04-18Revert "Delay initialization of a ConsumerBase instance to construction of a ↵ Anton Ivanov
sp/wp." This reverts commit 4efd0d936e2a2bfd9a46432270d7960062265c7b. Reason for revert: Caused b/409264080 Bug: 393217449 (cherry picked from https://googleplex-android-review.googlesource.com/q/commit:39266357b892a5b1f1d0a232d9706e512615b08d) Merged-In: Ia41cc9c08ce48d67f7b06b50adf6c573e4325482 Change-Id: Ia41cc9c08ce48d67f7b06b50adf6c573e4325482
2025-04-07Revert "Ensure Choreographer is held as sp<> in SurfaceControl." Anton Ivanov
This reverts commit 9045666047d55134c2e4f0108a88b5799653003e. Reason for revert: Supsected cause for b/405669080 Test: presubmit Flag: EXEMPT refactor Bug: 405669080 (cherry picked from https://googleplex-android-review.googlesource.com/q/commit:ade9b0ca6046beae913335000cdd398ac4381bb9) Merged-In: I96c45abfe8446c924e9ea31f822e040134e87db9 Change-Id: I96c45abfe8446c924e9ea31f822e040134e87db9
2025-03-24Merge "Add border API to surface control" into main Cairn Overturf
2025-03-24Merge "Delete deprecate deprecate_uiautomation_input_injection flag" into main Antonio Kantek
2025-03-21Merge "Revert^2 "Use TransactionState in SurfaceFlinger."" into main Treehugger Robot
2025-03-21Delete deprecate deprecate_uiautomation_input_injection flag Antonio Kantek
This flag was not used during the deprecation of the UiAutomation test APIs in changelist https://googleplex-android-review.googlesource.com/q/I205f26ce9d33f5f304093ef18177d818fc3422c5. Test: m Flag: EXEMPT deleting unused feature flag Bug: 277261245 Change-Id: Ib6f075ba86151256dbfa9d12a67093c9af2a2ec0
2025-03-21Merge "Ensure libgui classes that are held by sp<> are only constructed with ↵ Treehugger Robot
sp<>::make." into main
2025-03-21Merge "Add a unit test for receiving 'finish' message after channel close" ↵ Treehugger Robot
into main
2025-03-21Add a unit test for receiving 'finish' message after channel close Siarhei Vishniakou
After the input channel is closed, the messages that were previously written to the fd should still be readable. However, in some cases, if the fd is closed with data in the fd's buffer, the data may be discarded. This is confirmed by the attached unit test. Add an explanation of this to the InputChannel implementation, so that this behaviour is documented. Bug: 376713684 Test: TEST=libinput_tests; m $TEST && adb sync data && adb shell -t /data/nativetest64/$TEST/$TEST --gtest_filter="*ReceiveAfterClose*" --gtest_repeat=100000 --gtest_break_on_failure Test: TEST=libinput_tests; m $TEST && $ANDROID_HOST_OUT/nativetest64/$TEST/$TEST --gtest_filter="*ReceiveAfterClose*" --gtest_repeat=100000 --gtest_break_on_failure Flag: TEST_ONLY Change-Id: I67a6a6432c4756283c8f2cca3c01210a3bcdb42e
2025-03-20Merge "Check mDataPos to see if the Parcel needs to grow" into main Treehugger Robot
2025-03-20Merge "Surface: Add 'isBufferOwned' call" into main Jim Shargo
2025-03-20Merge "Revert "Include stack traces from keystore2 in bugreports on ↵ Todd Frederick
debuggable builds"" into main
2025-03-20Merge "InputDispatcher: Provide a way to transfer the entire gesture" into main Treehugger Robot
2025-03-20Revert^2 "Use TransactionState in SurfaceFlinger." Anton Ivanov
This reverts commit 4f8b4f371947dce3abece1c0d64af0b5c990e0c2. Reason for revert: Rolling forward with fix for use-after-move in SurfaceFlinger::setTransactionState Flag: EXEMPT refactor Bug: 385156191 Test: presubmit Change-Id: I66d45b1e125dcb50cbe7a3cfcefa8eaa02705419
2025-03-20Revert "Include stack traces from keystore2 in bugreports on debuggable builds" Todd Frederick
This reverts commit e846316a9f4e4cf35064267362e6d626e94d6a74. Dumping keystore is not allowed due to an SELinux neverallow rule most recently added in aosp/3311665. Bug: 376065666 Test: Check for denial after taking a bug report Flag: EXEMPT bugfix Change-Id: I0e6a2ff1578602f7616c12e8fc9253dddaa67779
2025-03-19Check mDataPos to see if the Parcel needs to grow Devin Moore
Flag: EXEMPT bug fix Ignore-AOSP-First: security fix Test: atest binderUnitTest Bug: 399155883 Change-Id: I38b755ca3381cfca3300292873f763823fbf169b
2025-03-19InputDispatcher: Provide a way to transfer the entire gesture Siarhei Vishniakou
In this CL, we modify the behaviour of 'transferTouchGesture' to also allow the caller to request that the entire gesture is transferred to the destination window. Suppose the original window receives ACTION_DOWN and calls transferTouchGesture to the destination window. Before this CL, the subsequent pointers would always be routed to the original window. In this CL, the caller of 'transferTouchGesture' can choose to transfer the 'entire gesture', meaning that subsequent pointers from the same gesture that land in the original window would automatically be sent to the destination window. Bug: 397979572 Test: TEST=inputflinger_tests; m $TEST && $ANDROID_HOST_OUT/nativetest64/$TEST/$TEST Flag: com.android.input.flags.allow_transfer_of_entire_gesture Change-Id: I760ddb8aa8f329d0b21788e6d6e83c802015ee1d
2025-03-19Merge "add ScreenPartStatus to displayIdentification" into main Daniel Zheng
2025-03-19add ScreenPartStatus to displayIdentification Daniel Zheng
composer Hal 3 adds screenPartStatus to the DisplayIdentification parcelable. The struct used in the client side should be updated to reflect this. Flag: EXEMPT refactor Test: th Change-Id: I8fb7ee099b2c54f3a4708aa19b3169d74e403e4b
2025-03-19Add border API to surface control YCairn Overturf
See go/sf-box-shadows-api for more details Bug: b/367464660 Flag: com.android.window.flags.enable_border_settings Test: atest SurfaceFlinger_test Change-Id: I1190edb97693004d9f46058fd0165451470a65b3
2025-03-18Merge "Add comment explaining BpBinder::linkToDeath invalid operation" into ↵ Treehugger Robot
main am: 4212ea291e am: 93806e1a53 Original change: https://android-review.googlesource.com/c/platform/frameworks/native/+/3524953 Change-Id: Idd2ba75fed12813dd397d8d1da2ef67f40f5d977 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2025-03-18Ensure libgui classes that are held by sp<> are only constructed with ↵ Anton Ivanov
sp<>::make. Exception is SurfaceComposerClient, which is widely references in vendor code. Test: presubmit Flag: EXEMPT refactor Bug: 393217449 Change-Id: Ib1039a1f7bc5676d32a4067361f131fc79efad4d
2025-03-18Merge "binderRpcTest: Fix sockaddr_storage handling" into main Devin Moore
2025-03-18Merge "Add comment explaining BpBinder::linkToDeath invalid operation" into main Treehugger Robot
2025-03-18Merge changes I60d42f38,I25857739 into main Treehugger Robot
* changes: Ensure Choreographer is held as sp<> in SurfaceControl. Harden construction sites of android::StrongPointer in frameworks/native
2025-03-18Merge "Revert^2 "Explicitly include libgui as a shared library in ↵ Treehugger Robot
libgui_test."" into main
2025-03-18Merge "Revert "Use TransactionState in SurfaceFlinger."" into main Treehugger Robot
2025-03-17Merge "libbinder: fix trusty cpp fd transport modes" into main am: ↵ Treehugger Robot
33da0c2368 am: 3ea3bd3ec2 Original change: https://android-review.googlesource.com/c/platform/frameworks/native/+/3547982 Change-Id: Ic37d6529d7060d0bc4982c7b88e5948d8eb34241 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2025-03-17Merge "libbinder: fix trusty cpp fd transport modes" into main Treehugger Robot
2025-03-17Revert^2 "Explicitly include libgui as a shared library in libgui_test." Anton Ivanov
This reverts commit d6198bed29ffb56054490820e73ee30a718930d1. Reason for revert: Removed flag defines from Android.bp that caused linker errors. Verified libgui_test behaves identically with and without this revert on cf_x86_64_auto-next-userdebug Change-Id: Ibbbd201253fa09c18491bffca0b540c2c13df8d3
2025-03-17Revert "Use TransactionState in SurfaceFlinger." Anton Ivanov
This reverts commit ed63243743079eb4531cb4a9e5ed4dccb7965d6d. Reason for revert: Droidmonitor created revert due to Jank regression b/403375000. Change-Id: I558338084124764026028b21667b45afb73b9e06
2025-03-17Ensure Choreographer is held as sp<> in SurfaceControl. Anton Ivanov
Test: ChoreographerTests pass Flag: EXEMPT refactor Bug: 393217449 Change-Id: I60d42f383d4b73b60de1a7a1bbeec5170b805ebb
2025-03-17Harden construction sites of android::StrongPointer in frameworks/native Anton Ivanov
Test: presubmit Flag: EXEMPT refactor Bug: 393217449 Change-Id: I25857739af659a52d2e33fc0e05799959f4024ab
2025-03-17libbinder: fix trusty cpp fd transport modes Gil Cukierman
This moves the call to set allowed file descriptor transport modes to a common method that is invoked on both cpp and rust construction of RpcServerTrusty objects. Test: atest -c frameworks/native/libs/binder/tests/ Bug: 393346719 Change-Id: I0ec04676ff36879ce2b40f5070214b1cbe21f2b0
2025-03-17Merge "Remove flags from cppflags of libgui_test." into main Treehugger Robot
2025-03-17Merge "Avoid copying Transaction objects unneccessarily." into main Anton Ivanov
2025-03-14Merge "Implement Send for RpcServerConnection" into main am: c643576c05 am: ↵ Hasini Gunasinghe
22795fe48f Original change: https://android-review.googlesource.com/c/platform/frameworks/native/+/3398707 Change-Id: I980c90b1f974c4b7d953611b7ab58ffe549c40db Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2025-03-14Merge "Implement Send for RpcServerConnection" into main Hasini Gunasinghe
2025-03-14Merge "[Lut] fix over-saturated issue on GPU composition" into main Sally Qi
2025-03-14Avoid copying Transaction objects unneccessarily. Anton Ivanov
Flag: EXEMPT refactor Bug: 385156191 Test: presubmit Change-Id: Ibd9d64bd7d41adbf5af0dacd660b6aaed6bc8741
2025-03-14Merge "Use EXPECT_NEAR for float value in tests" into main Arpit Singh
2025-03-14Merge "Add get() method to declare_binder_enum" into main am: 0fa903349f am: ↵ Treehugger Robot
709de11f8a Original change: https://android-review.googlesource.com/c/platform/frameworks/native/+/3483296 Change-Id: I4909414908686a9daf66617209fb844a5e8d8f94 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2025-03-14Merge "Add get() method to declare_binder_enum" into main Treehugger Robot
2025-03-14Surface: Add 'isBufferOwned' call Jim Shargo
Camera's StreamSplitter keeps track of buffers and won't attach them if they're already owned. This allows us to keep that behavior as we move over from IGBP to surfaces. Ideally, I think, IGBPs should fail to attach the same buffer multiple times. Even if not, this call should probably be to the IGBP instead of the Surface for more accuracy. But this replicates the old behavior. Bug: 340933206 Flag: com.android.graphics.libgui.flags.wb_stream_splitter Test: new test Change-Id: Idbb200202012c9eae2668616dcff277c925c3907
2025-03-14Merge "Introduce a dependency monitor for fences" into main Alec Mouri
2025-03-14Merge "Revert "libgui: Make bq_gl_fence_cleanup a bugfix flag"" into main Jim Shargo