diff options
| author | 2024-10-31 22:03:01 -0700 | |
|---|---|---|
| committer | 2024-10-31 22:03:01 -0700 | |
| commit | 0f09c2f475d06e6802085755aa3fffe729f3a256 (patch) | |
| tree | b05c90cf12eb805b36edcc06dd13bef9c2032afb /libs/androidfw/ObbFile.cpp | |
| parent | 95cff9b40e17bc21a202be71d0cfa58515996bbd (diff) | |
appops: Finish started proxy op when chain fails
A more precise version of I92060d44e666fa6725411de5d714ac0d380f42ae
This fixes an issue where we finish the op which failed permission
checks... which causes refcount mismatches again.
Instead, ensure that we finish only the proxy ops which were
*successfully* started: acheiving this by pushing the cleanup into the
checkPerm loop which iterates through the attr chain.
Technically this should also be added for appop permissions, but focus
on runtime appops for now, since that is where the security issue is.
Test: CtsMediaAudioPermissionTestCases
Bug: 293603271
Flag: EXEMPT security
Change-Id: Ifced9449e47b09c7a1a9982c73d4871302a742a2
Diffstat (limited to 'libs/androidfw/ObbFile.cpp')
0 files changed, 0 insertions, 0 deletions