aboutsummaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2024-04-09storage: Add rust test for secure storage aidlHEADmastermainWeston Carvalho
Translate some of the unittests from `storage-unittest` into Rust tests calling the AIDL service. Tested: ./build-root/build-qemu-generic-arm64-test-debug/run --headless --boot-test "com.android.trusty.rust.storage_unittest_aidl.test" Bug: 300673823 Change-Id: Ib75e3afb3c179418da936c8e529e57b7bcb9efd8
2024-03-28storage: Implement storage aidl serviceWeston Carvalho
Implement all operations for `IDir`, `IFile`, `IStorageSession`, and `ISecureStorage`. Bug: 300673722 Change-Id: Ia47c867dc4c39921e7a1fde7524be08b8aaea9b5
2024-03-27[Metrics] Shift secure storage trusty metrics to use TIPC, and rename ↵Snehal
variables to match atom conventions Bug: 326964373 Bug: 328592145 Change-Id: Iee13de7faeddaf07a695934a80bc70ca2862d0d6
2024-03-19storage: Init service per peer and fsWeston Carvalho
Attach a peer to `SecureStorage` and track active filesystems so that the AIDL service can create active `storage_client_session`s. Bug: 300673722 Change-Id: I3283f39d033a98504350b5b2bed730e240e9eea7
2024-03-19storage: Add storage aidl serviceWeston Carvalho
Add placeholder implementation of the secure storage AIDL interface which always returns `EX_UNSUPPORTED_OPERATION`. Bug: 300673722 Change-Id: Ic9cd81b7916f04968070757c0176b309d1532a86
2024-03-18rpmb: fix unsigned int underflow in debug modeYurii Shutkin
Change-Id: Ic71729ffae5ee27ad165e440e021d5ee35feac34 Bug: None Test: run with RPMB_DEBUG defined
2024-03-15storage: Return error from rpmb_send on short readsArve Hjønnevåg
If the response packet is truncated, return an error code so we get a clearer error message than having the rpmb layer complain about the content in the uninialized repsonse data. Bug: 304582473 Change-Id: If201b5d0950371f8f276546e25087343419635e7
2024-03-06trusty:storage: add missing NULL check to prevent RAM dumpMike McTernan
file_get_info() may return NULL on a failed transaction, so check for this to avoid a possible NULL pointer access. Bug: 325770752 Test: build.py, run storage tests Change-Id: Id59edb940a6fa98f45d5d557572644d7a1f923e0
2024-03-06cleanup: remove reference to unexistent rss_limit_checker.cppYurii Shutkin
Change-Id: Ia6618e82db37e1995d35dfcfae41da3adcd74794 Bug: 305723343 Test: build
2024-03-05storage: Add storage benchmark to test-map am: a8fb26a9b6Raphael Herouart
Original change: https://android-review.googlesource.com/c/trusty/app/storage/+/2986294 Change-Id: I5d4a24b22512a01c641042677307915e4324b9c4 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-03-05storage: Add storage benchmark to test-mapRaphael Herouart
Make sure the com.android.trusty.storage.bench port is run by CI pipeline Bug: None Test: Build.py and verify "com.android.trusty.storage.bench" is in test-map.json Change-Id: Ib3c6c6fa788c03706c227f30d21fff720396c41a
2024-02-29test/storage-benchmark: [Bench] Secure Storage Benchmark am: 494556f723Raphael Herouart
Original change: https://android-review.googlesource.com/c/trusty/app/storage/+/2921652 Change-Id: If33e0da231c8eb0ed9ce5d7f07a52a8083c15da9 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-02-29test/storage-benchmark: [Bench] Secure Storage BenchmarkRaphael Herouart
Evaluate Secure Storage Performance Bug: 314131079 Change-Id: I9a0bf130ca1744ec6062dcd505f20847f59cb478
2024-02-22Allow C headers to be included in cpp files am: 65ae65b4d4Weston Carvalho
Original change: https://android-review.googlesource.com/c/trusty/app/storage/+/2870347 Change-Id: I47951a135221cce3d17129579b52f65ac2e357ab Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-02-22Don't require passing `src_name` if it's not used. am: c6d342ea46Weston Carvalho
Original change: https://android-review.googlesource.com/c/trusty/app/storage/+/2924290 Change-Id: I44d030327f375eafcbe0ae28ee3c9b5c61e6810a Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-02-22Move tipc-independent fns to client.h am: f52d4a8fc7Weston Carvalho
Original change: https://android-review.googlesource.com/c/trusty/app/storage/+/2806248 Change-Id: I5477f3c40f7813f66e614dc3d19dc8c18b41d13d Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-02-22Separate session init and destroy am: 815e02ec35Weston Carvalho
Original change: https://android-review.googlesource.com/c/trusty/app/storage/+/2806249 Change-Id: I9b18839a9d0ecd39df2a46ea6de19c665fc354af Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-02-22Split transaction end am: d200cb7fffWeston Carvalho
Original change: https://android-review.googlesource.com/c/trusty/app/storage/+/2806247 Change-Id: Ic5ad966096bf0711e32b35449402cf4a8ff66e59 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-02-22Allow C headers to be included in cpp filesWeston Carvalho
Not expecting any of the functions included in these headers to be called from C++ files, so not adding `__BEGIN_CDECLS` and `__END_CDECLS`. They are either transitively included (through `client.h`) or included by C++ for their struct definitions. * Make C-style cast of `void*` -> `struct mac*` explicit since cpp won't make it implicitly. * Add `storage_` prefix to functions to avoid collisions with OS headers. * `file_handle` conflicts with fcntl.h * `encrypt` conflicts with unistd.h Bug: 300673823 Change-Id: I29129321e503b6a7df108a0b826b5b11b517384d
2024-02-21Don't require passing `src_name` if it's not used.Weston Carvalho
Allow `storage_file_move` to be called without `src_name` if `src_already opened` indictates that the src file is already opened and we're finding it using the `handle`. Bug: 300673823 Change-Id: I21ae232057f983935c4178a969008533412efb2e
2024-02-21Move tipc-independent fns to client.hWeston Carvalho
Bug: 307999674 Change-Id: I42ce336f5a546d80f52461ce2c27a504fa320a86
2024-02-21Separate session init and destroyWeston Carvalho
Bug: 307999674 Change-Id: I870ab4efce28cba20aee12a4aad06604b00cb83e
2024-02-16Split storage_file_read am: 16717ea84fWeston Carvalho
Original change: https://android-review.googlesource.com/c/trusty/app/storage/+/2806246 Change-Id: I82c937f6648a7cbcaa7e5776b2ab51d5d038162d Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-02-16Split storage_file_list am: 9fcdfe8e34Weston Carvalho
Original change: https://android-review.googlesource.com/c/trusty/app/storage/+/2806245 Change-Id: I998af6f7fd92c2198d85a645987b157af6319b05 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-02-16Split storage_file_get_size am: ad08b154d6Weston Carvalho
Original change: https://android-review.googlesource.com/c/trusty/app/storage/+/2806244 Change-Id: I0949dcce7f16fc59d6139c2303c0d514b7fa484d Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-02-16Split storage_file_open am: 0b9f3528f8Weston Carvalho
Original change: https://android-review.googlesource.com/c/trusty/app/storage/+/2806243 Change-Id: Iaa5f9f7a5b43191e08936341306e7be570df2a98 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-02-16scudo: Set the ring buffer size using SCUDO_DEFAULT_OPTIONS am: aecd329097Andrei Homescu
Original change: https://android-review.googlesource.com/c/trusty/app/storage/+/2849068 Change-Id: Ie2c0f33fb2354f5211580e07068d4d91f247158d Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-02-16Split storage_file_set_size am: 295fba4b46Weston Carvalho
Original change: https://android-review.googlesource.com/c/trusty/app/storage/+/2806242 Change-Id: I4e35f4e3826bac2fd8aa18715b59cfad990fb22d Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-02-16Split storage_file_write am: b262752da8Weston Carvalho
Original change: https://android-review.googlesource.com/c/trusty/app/storage/+/2806241 Change-Id: I7b450fae85f04633ab84c765ed7c22b4668bd5ef Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-02-16Split storage_file_close am: 9d650493b4Weston Carvalho
Original change: https://android-review.googlesource.com/c/trusty/app/storage/+/2806240 Change-Id: I125f794f28800e94e9649cb86af5875de604cfb0 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-02-16Split storage_file_move am: c938ec2943Weston Carvalho
Original change: https://android-review.googlesource.com/c/trusty/app/storage/+/2806239 Change-Id: Ib75338e6bbbfed863fcd5534ac70c40705774ba9 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-02-16Split storage_file_delete am: 98d07d4e82Weston Carvalho
Original change: https://android-review.googlesource.com/c/trusty/app/storage/+/2806238 Change-Id: I2ba027eb6847b880d0ca001cbfed347188926f5d Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-02-16trusty: cleanup storage before running tests am: 9b634f6a1bYurii Shutkin
Original change: https://android-review.googlesource.com/c/trusty/app/storage/+/2781627 Change-Id: Id72132bed8c7fd6ea23629ea901b6750f3f67861 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-02-16Separate tipc part of storage_client_session am: 2e4e28489bWeston Carvalho
Original change: https://android-review.googlesource.com/c/trusty/app/storage/+/2806235 Change-Id: Id455fc1558135b1dac63e04c6045ab6563bb2729 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-02-16Fix format specifiers. am: 5e3e0812b6Weston Carvalho
Original change: https://android-review.googlesource.com/c/trusty/app/storage/+/2806236 Change-Id: I17edcae911a9a0fc6a027c29198ce2698fa54576 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-02-16Use TIPC library instead of wait_any am: 5271b319b1Weston Carvalho
Original change: https://android-review.googlesource.com/c/trusty/app/storage/+/2759885 Change-Id: I8a5d34d76ef946006d005d04454bf840716c040d Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-02-16scudo: split include wrappers into separate files to resolve static ↵Yurii Shutkin
functions conflict am: 30529c11e0 Original change: https://android-review.googlesource.com/c/trusty/app/storage/+/2792573 Change-Id: I8ce1a1c0bfe805f2b99baa7f81c4bdfee76cd9e7 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-02-16scudo: include rss_limit_checker.cpp only if exists am: f58e42e0a3Yurii Shutkin
Original change: https://android-review.googlesource.com/c/trusty/app/storage/+/2786253 Change-Id: I1d06c890e28c203180082111954d78712d563d01 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-02-16Add new file report_linux.cpp. am: 1d0e95899dChristopher Ferris
Original change: https://android-review.googlesource.com/c/trusty/app/storage/+/2787034 Change-Id: I1e99f89c9ca86c5a46b58a4626f4b2ecd94be28e Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-02-16storage: CONSTANTS is deprecated am: dd8ab633c2Raphael Herouart
Original change: https://android-review.googlesource.com/c/trusty/app/storage/+/2755130 Change-Id: I8435efd6a1512764b6fdb19a24cbea2fbc8b6093 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-02-16storage: update scudo config to work with upstream changes am: 9599fd2d08Frances Wingerter
Original change: https://android-review.googlesource.com/c/trusty/app/storage/+/2626777 Change-Id: Icadb2bc024117087b4b56cf8baa99cb94f225111 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-01-10Split transaction endWeston Carvalho
Bug: 307999674 Change-Id: Ice1e2fe455110e8af1db6c0f99f63c6112edcef7
2024-01-10Split storage_file_readmain-16kWeston Carvalho
Bug: 307999674 Change-Id: I99453fe130040b419446379459ce9dff031ee4fc
2024-01-10Split storage_file_listWeston Carvalho
Also fixes bug where the `max_count` argument was not being respected. Bug: 307999674 Change-Id: Icb72e3bfab8b40ca1010639986c7410f068a5171
2023-11-30Split storage_file_get_sizeWeston Carvalho
Bug: 307999674 Change-Id: I5ab6c33222318665a3ae0b4935d448d6cbc9b59e
2023-11-30Split storage_file_openWeston Carvalho
Bug: 307999674 Change-Id: Ie55c985a29c880c224db2364262dbcd34ab5fbb8
2023-11-28scudo: Set the ring buffer size using SCUDO_DEFAULT_OPTIONSAndrei Homescu
Bug: 285203365 Test: Run com.android.trusty.scudotest Change-Id: I0645464f1d5f85536a6a5e2c3aa829b6a51d3aa6
2023-11-16Split storage_file_set_sizeWeston Carvalho
Bug: 307999674 Change-Id: I1202f0ca871b6ab9151240598cdce61340e55910
2023-11-16Split storage_file_writeWeston Carvalho
Bug: 307999674 Change-Id: I1dfd8135ece3b362d2bf4bc25c9a31f620d928f5
2023-11-16Split storage_file_closeWeston Carvalho
Bug: 307999674 Change-Id: Ib7f316138e0a41b650022c65ff52aa16b0539e5c