Age | Commit message (Collapse) | Author |
|
into sc-v2-platform-release.
Change-Id: I365e4101295c2e30b69a2d54162854fb0abee740
|
|
We have never offered the native API to register mode watchers that are
invoked for foregroundness changes when the raw mode is MODE_FOREGROUND.
Test: Add logging to verify invocation
Bug: 247768581
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:949cb3d098fd98715826fc92ea3c26a51aa2d976)
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:130ff33dfeb3c0702636db090399f114fde86174)
Merged-In: I89af46de557fbfc31d69613367a4e26a5222430a
Change-Id: I89af46de557fbfc31d69613367a4e26a5222430a
|
|
into sc-v2-platform-release.
Change-Id: I7bdc15365fa8f285cdf96283a59188ee0015d753
|
|
This adds a unit test and fix for the bug reported by libfuzzer.
Changes made:
* Expose GPUService as testable code.
* Update main_gpuservice.cpp to use the new GpuService now located at
gpuservice/GpuService.h
* Make initializer threads members of GpuService
* Join the threads in destructor to prevent heap-use-after-free.
* Add unit test that waits 3 seconds after deallocation to ensure no
wrong access is made.
Bug: 282919145
Test: Added unit test and ran on device with ASAN
(cherry picked from commit 3c00cbc0f119c3f59325aa6d5061529feb58462b)
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:24a7874bb4093a1a6721a2002569512c43af5bdc)
Merged-In: I4d1d2d4658b575bf2c8f425f91f68f03114ad029
Change-Id: I4d1d2d4658b575bf2c8f425f91f68f03114ad029
|
|
into sc-v2-platform-release.
Change-Id: I38cad9a6135f313a67dbe06ce57d95f6a11f0dcd
|
|
Test: atest VtsHalSensorManagerV1_0TargetTest
Bug: 278013275
Bug: 269014004
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:49600b10aa5675d4e7e985203d69f252ead13e45)
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:d86376e25df2c601929a2fed3fed875cfe4e56ea)
Merged-In: I091f57de9570b0ace3a8da76f16fe0e83f0aa624
Change-Id: I091f57de9570b0ace3a8da76f16fe0e83f0aa624
|
|
'googleplex-android-review.googlesource.com/21548272', 'googleplex-android-review.googlesource.com/21548273'] into sc-v2-platform-release.
Change-Id: Ica67fc68280e715dcaf1fc77c3842f5bb3c54497
|
|
In order to ensure that clients don't leak their sensor manager
instance that we currently store in a static map, they need to be able
to remove their instance. Otherwise, this instance is never removed from
the list and will hang around until our SensorManage instance is
destroyed.
Bug: 269014004
Test: Run ./libsensorserviceaidl_fuzzer
Change-Id: I52185f74ae8d28b379440235ca6f03c5089081f5
(cherry picked from commit 9532f7c682fdd4b1e6e553cd6f61fc0cf2555902)
Merged-In: I52185f74ae8d28b379440235ca6f03c5089081f5
(cherry picked from commit on googleplex-android-review.googlesource.com host: 16af3e02e5864c3d97548f0acd05baf5e3341950)
Merged-In: I52185f74ae8d28b379440235ca6f03c5089081f5
|
|
After catching an error in Sensor::unflatten, there are memory leaks
caught by the fuzzer in the same test case.
Test: libsensorserviceaidl_fuzzer with testcase from bug
Bug: 269014004
Merged-In: I509cceb41f56ca117d9475f6f6674244560fe582
Change-Id: I509cceb41f56ca117d9475f6f6674244560fe582
(cherry picked from commit c95fa0f0e7c7b73746ff850b85a79fc5f92b784e)
Merged-In: I509cceb41f56ca117d9475f6f6674244560fe582
(cherry picked from commit on googleplex-android-review.googlesource.com host: cf83b16a403bfa8adf5a1490bbd14c8459e19140)
Merged-In: I509cceb41f56ca117d9475f6f6674244560fe582
|
|
Test: libsensorserviceaidl_fuzzer with testcase from bug
Bug: 269014004
Merged-In: I0e255c64243c38876fb657cbf942fc1613363216
Change-Id: I0e255c64243c38876fb657cbf942fc1613363216
(cherry picked from commit aeec1802f7befc8fbb18313ad3ac0969c3811870)
Merged-In: I0e255c64243c38876fb657cbf942fc1613363216
(cherry picked from commit on googleplex-android-review.googlesource.com host: 54aacdf96a7f57b97f8b6f1a69a8b39ae5ed4b18)
Merged-In: I0e255c64243c38876fb657cbf942fc1613363216
|
|
into sc-v2-platform-release.
Change-Id: I5866ab12f6d8e2d853211a286dd41a9fda01c96a
|
|
- This is part of fix of commit
Id9d9012d4ede9c8330f0ce1096bcb78e51b7c5df for backporting.
Bug: 248031255
Test: test using displaytoken app manually on the phone, test shell
screenrecord during using displaytoken; atest
android.hardware.camera2.cts.FastBasicsTest
Change-Id: Id9d9012d4ede9c8330f0ce1096bcb78e51b7c5df
Merged-In: Id9d9012d4ede9c8330f0ce1096bcb78e51b7c5df
(cherry picked from commit 3ea58dbc1d7a248160403f089b9998bf6694aae1)
Merged-In: Id9d9012d4ede9c8330f0ce1096bcb78e51b7c5df
|
|
Change-Id: Ie3e11a860bda4de97df8a7d240624a3cc965798b
|
|
To avoid it being initialised to true randomly.
Bug: 199483370
Bug: 196389741
Bug: 218605327
Change-Id: I75be2b1d305e22f8a71532b9f5b8ea6c469baaaa
Merged-In: I75be2b1d305e22f8a71532b9f5b8ea6c469baaaa
(cherry picked from commit edb375df82f6b48688d85c11dbca91adef088bc8)
(cherry picked from commit b4a2d3a77ab2fc8b7e3d88c528c71269651f0ddb)
(cherry picked from commit c058b4851d187d10a8f134c02c5cf1a22c1a9d39)
Merged-In: I75be2b1d305e22f8a71532b9f5b8ea6c469baaaa
|
|
This allows container surfaces to be considered visible in
InputDispatcher and prevent touches from passing through them. In
particular this is used by ActivityRecordInputSink to block touches that
would otherwise pass through the area available to the activity.
Bug: 194480991
Bug: 222292477
Test: Manually tested that blocking still works
Test: atest InputSurfacesTest
Change-Id: Iacfc952139311363b914b1d6bffc2b4190133d02
(cherry picked from commit a18588206c240b34128bb27e3ac875af17fce7f2)
(cherry picked from commit 8758cd1030dc9fd73cc98c09ce7b9c356757a659)
Merged-In: Iacfc952139311363b914b1d6bffc2b4190133d02
|
|
Change-Id: Ida05240506e8789e2541a76cd0c20a588e2559cd
|
|
stealReceiveChannel
Add a mutex to prevent a potential race condition.
Bug: 232541124
Test: See bug for details
Change-Id: Ia338f124c786bf12d6adba10a67b9048fe9c34a5
(cherry picked from commit 97a317064a76b8fc3a65bd980027f820fd4d53ae)
Merged-In: Ia338f124c786bf12d6adba10a67b9048fe9c34a5
|
|
Otherwise we could leak uninitialized memory to
an untrusted process.
Bug: 214999987
Test: Existing tests pass
Change-Id: I87993fbb920b3af938a311e6afc15383d571823a
(cherry picked from commit c2cf02c17af0d7e5f58b8a17ecb9cfcc91a4d2eb)
(cherry picked from commit 61a52db9d3a144a8ae4a6a495d61a3b59db80300)
Merged-In: I87993fbb920b3af938a311e6afc15383d571823a
|
|
sc-v2-platform-release
Change-Id: I04b77150dbff74ce1a1ec675c0f5c844d33a62fe
|
|
sso://googleplex-android/platform/frameworks/native into HEAD
Change-Id: I2a55b6478cc6339754f4c2a46eff874bd1647e5a
|
|
sc-qpr3-release
Change-Id: Id79b5ebbd47a3bca193982f22c919d6073dcb19e
|
|
Transaction#sanitize" into sc-dev am: 04caf0c75b am: c226dcf6a2 -s ours
am skip reason: subject contains skip directive
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/native/+/16811474
Change-Id: I4f83fc95854f9792212f575f20cea7319ba431b3
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
am: 1ff38ab351 am: 5882fa4ab0 -s ours
am skip reason: subject contains skip directive
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/native/+/16811474
Change-Id: Ib3b7345b0bdcdcc8cf9b4098826a6b1c3e0e569f
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
am: 04caf0c75b
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/native/+/16811474
Change-Id: I4583fd4a36654995c78a503436da43fa75d6205f
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/native/+/16811474
Change-Id: I2e02546aa4339b6145fa3ead4eb204252c4cf939
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
Transaction#sanitize" into sc-qpr1-dev am: 7c8d637adf -s ours
am skip reason: subject contains skip directive
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/native/+/16811471
Change-Id: I86b5c5f457929b8782da5e37af018a128b4e11ce
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
am: 842a412840 -s ours
am skip reason: subject contains skip directive
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/native/+/16811471
Change-Id: I047f02dcc35301e6fc5abf0888310223b19e3014
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
|
|
|
|
|
|
Various elements of the Transaction interface require
a permission in order to apply. In particular the setTrustedOverlay
and setInputWindowInfo fields. These permission checks are
implemented by checking the PID and the UID of the process which
sent the transaction. Unfortunately widespread use of transaction
merging makes this inadequate. At the moment
IWindowSession#finishDrawing seems to be the only boundary on which
transactions move from client to system processes, and so we expose
a sanitize method and use it from there to resolve the situation
in an easily backportable way.
Moving forward it likely make sense to move security sensitive
interfaces off of Transaction. Most of the things behind permissions
currently are not truly security sensitive, more of just a request
not to use them.
It was also considered to sanitize transactions at all process
boundaries through writeToParcel, however this could be disruptive
as previously permissioned processes (WM and SysUI) could freely
exchange transactions. As the change needs to be backportable the
lowest risk option was chosen.
Bug: 213644870
Test: Existing tests pass
Change-Id: I424f45bc30ea8e56e4c4493203ee0749eabf239c
(cherry picked from commit de6d7b467e572d384f2bc1bc788259340ebe2f93)
|
|
Various elements of the Transaction interface require
a permission in order to apply. In particular the setTrustedOverlay
and setInputWindowInfo fields. These permission checks are
implemented by checking the PID and the UID of the process which
sent the transaction. Unfortunately widespread use of transaction
merging makes this inadequate. At the moment
IWindowSession#finishDrawing seems to be the only boundary on which
transactions move from client to system processes, and so we expose
a sanitize method and use it from there to resolve the situation
in an easily backportable way.
Moving forward it likely make sense to move security sensitive
interfaces off of Transaction. Most of the things behind permissions
currently are not truly security sensitive, more of just a request
not to use them.
It was also considered to sanitize transactions at all process
boundaries through writeToParcel, however this could be disruptive
as previously permissioned processes (WM and SysUI) could freely
exchange transactions. As the change needs to be backportable the
lowest risk option was chosen.
Bug: 213644870
Test: Existing tests pass
Change-Id: I424f45bc30ea8e56e4c4493203ee0749eabf239c
(cherry picked from commit de6d7b467e572d384f2bc1bc788259340ebe2f93)
|
|
Various elements of the Transaction interface require
a permission in order to apply. In particular the setTrustedOverlay
and setInputWindowInfo fields. These permission checks are
implemented by checking the PID and the UID of the process which
sent the transaction. Unfortunately widespread use of transaction
merging makes this inadequate. At the moment
IWindowSession#finishDrawing seems to be the only boundary on which
transactions move from client to system processes, and so we expose
a sanitize method and use it from there to resolve the situation
in an easily backportable way.
Moving forward it likely make sense to move security sensitive
interfaces off of Transaction. Most of the things behind permissions
currently are not truly security sensitive, more of just a request
not to use them.
It was also considered to sanitize transactions at all process
boundaries through writeToParcel, however this could be disruptive
as previously permissioned processes (WM and SysUI) could freely
exchange transactions. As the change needs to be backportable the
lowest risk option was chosen.
Bug: 213644870
Test: Existing tests pass
Change-Id: I424f45bc30ea8e56e4c4493203ee0749eabf239c
(cherry picked from commit de6d7b467e572d384f2bc1bc788259340ebe2f93)
|
|
sc-qpr3-release
Change-Id: Ia99367d8e76f1ec7fb832dcd4fcdc09d360d95b8
|
|
Bug: 220733817
Fixes: 220733817
Test: presubmit
Change-Id: I98972b661fc36906d7d86b366c3f9537dcd17b12
|
|
sc-qpr3-release
Change-Id: I1bcf683394666238c81f9af25ec6a28e439029ce
|
|
Picked from aosp/1959201
Fix:220828052
Test: atest dumpstate_smoke_test:dumpstate_smoke_test.ZippedBugreportGenerationTest
Change-Id: Ic309423babba4e2d03ab7d95b9188faea79b3a41
Merged-In: I37e7707df3237f39d099484a26f195a85f16767a
|
|
sc-qpr3-release
Change-Id: I815ff2b5cd13cee0250eab8d135f322f7518be3f
|
|
cbe9c105fc am: 4648ad64f2 -s ours am: f2a0e26203 -s ours am: 36cf85e10a -s ours
am skip reason: Merged-In Ibce11ee7df5b5c8c226ebfab29574a99cd656f6d with SHA-1 9b0d13dc44 is already in history
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/native/+/16508594
Change-Id: Ie8c1ee80cfd1f7a657601aa76076eb356bbd3dd7
|
|
0f13fe552d am: f86cec5a48 -s ours am: d25e22dbce -s ours am: a22dfdd3c6 -s ours
am skip reason: Merged-In I71d7cf5064c8ce4626cff09b92e15ca38b39cbbe with SHA-1 212dcf44f6 is already in history
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/native/+/16508593
Change-Id: I3d43691a6b89d037499e9a647fd2132eaee2d9b5
|
|
sc-qpr3-release
Change-Id: Ib0b0f14676cf4f56cd93c7f7bfc45f7d2400e67e
|
|
cbe9c105fc am: 4648ad64f2 -s ours am: f2a0e26203 -s ours
am skip reason: Merged-In Ibce11ee7df5b5c8c226ebfab29574a99cd656f6d with SHA-1 9b0d13dc44 is already in history
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/native/+/16508594
Change-Id: Ie3e876d04fac6a851158472befd0a18f95309dee
|
|
0f13fe552d am: f86cec5a48 -s ours am: d25e22dbce -s ours
am skip reason: Merged-In I71d7cf5064c8ce4626cff09b92e15ca38b39cbbe with SHA-1 212dcf44f6 is already in history
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/native/+/16508593
Change-Id: Ifb3ba14f67a39494d5beb3929d515645908b31e7
|
|
cbe9c105fc am: 4648ad64f2 -s ours
am skip reason: Merged-In Ibce11ee7df5b5c8c226ebfab29574a99cd656f6d with SHA-1 9b0d13dc44 is already in history
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/native/+/16508594
Change-Id: Ic7146d0268a50f93c3822a874215643fa02e8b88
|
|
0f13fe552d am: f86cec5a48 -s ours
am skip reason: Merged-In I71d7cf5064c8ce4626cff09b92e15ca38b39cbbe with SHA-1 212dcf44f6 is already in history
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/native/+/16508593
Change-Id: Id263840eac339bb7decfd556092cc8a0efb7eae4
|
|
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/native/+/16508594
Change-Id: I165587ce81cbc15030bca8c385f1e63c80ef27a3
|
|
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/native/+/16508593
Change-Id: I3c51969c29fa3b501964651183930e7c7774b071
|
|
am: 0c3407429e -s ours am: dc0120ff41 -s ours am: 27cc85f3f3 -s ours
am skip reason: Merged-In I443741d5ab51a45d37fb865f11c433c436d96c1e with SHA-1 77daf700ce is already in history
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/native/+/16508592
Change-Id: Id695dbdde15afc6d821a79c0be1fb61f9a499566
|
|
am: 0c3407429e -s ours am: dc0120ff41 -s ours
am skip reason: Merged-In I443741d5ab51a45d37fb865f11c433c436d96c1e with SHA-1 77daf700ce is already in history
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/native/+/16508592
Change-Id: I2233c9af21ab8389ac0f1fc90eb4406430bb8be7
|
|
am: 0c3407429e -s ours
am skip reason: Merged-In I443741d5ab51a45d37fb865f11c433c436d96c1e with SHA-1 77daf700ce is already in history
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/native/+/16508592
Change-Id: I8a6b181d61577dde0d97b8340a44ade1a3ebc3f9
|