Revert "comply with VTS requirements" am: f272a85a53
Original change: https://googleplex-android-review.googlesource.com/c/device/google/gs-common/+/22328025 Change-Id: I8ecec5a6b63447e3768567437dd8531e2a511ef6 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
commit
cff535bd9a
6 changed files with 1 additions and 16 deletions
|
@ -154,11 +154,7 @@ ndk::ScopedAStatus Dumpstate::dumpstateBoard(const std::vector<::ndk::ScopedFile
|
||||||
int64_t in_timeoutMillis) {
|
int64_t in_timeoutMillis) {
|
||||||
// Unused arguments.
|
// Unused arguments.
|
||||||
(void) in_timeoutMillis;
|
(void) in_timeoutMillis;
|
||||||
|
(void) in_mode;
|
||||||
if (in_mode < IDumpstateDevice::DumpstateMode::FULL || in_mode > IDumpstateDevice::DumpstateMode::PROTO) {
|
|
||||||
ALOGE("Invalid mode: %d\n", in_mode);
|
|
||||||
return ndk::ScopedAStatus::fromExceptionCodeWithMessage(EX_ILLEGAL_ARGUMENT, "Invalid mode");
|
|
||||||
}
|
|
||||||
|
|
||||||
if (in_fds.size() < 1) {
|
if (in_fds.size() < 1) {
|
||||||
ALOGE("no FDs\n");
|
ALOGE("no FDs\n");
|
||||||
|
@ -175,7 +171,6 @@ ndk::ScopedAStatus Dumpstate::dumpstateBoard(const std::vector<::ndk::ScopedFile
|
||||||
|
|
||||||
if (in_fds.size() < 2) {
|
if (in_fds.size() < 2) {
|
||||||
ALOGE("no FD for dumpstate_board binary\n");
|
ALOGE("no FD for dumpstate_board binary\n");
|
||||||
dumpTextSection(fd, "");
|
|
||||||
} else {
|
} else {
|
||||||
int fd_bin = in_fds[1].get();
|
int fd_bin = in_fds[1].get();
|
||||||
dumpLogSection(fd, fd_bin);
|
dumpLogSection(fd, fd_bin);
|
||||||
|
|
|
@ -1,4 +1,3 @@
|
||||||
# dumpstate packing directory
|
|
||||||
type radio_vendor_data_file, file_type, data_file_type;
|
type radio_vendor_data_file, file_type, data_file_type;
|
||||||
userdebug_or_eng(`
|
userdebug_or_eng(`
|
||||||
typeattribute radio_vendor_data_file mlstrustedobject;
|
typeattribute radio_vendor_data_file mlstrustedobject;
|
||||||
|
|
|
@ -1,4 +1,3 @@
|
||||||
# generic dumpstate for pixel
|
|
||||||
/vendor/bin/hw/android\.hardware\.dumpstate-service u:object_r:hal_dumpstate_default_exec:s0
|
/vendor/bin/hw/android\.hardware\.dumpstate-service u:object_r:hal_dumpstate_default_exec:s0
|
||||||
|
|
||||||
/data/vendor/radio(/.*)? u:object_r:radio_vendor_data_file:s0
|
/data/vendor/radio(/.*)? u:object_r:radio_vendor_data_file:s0
|
||||||
|
|
|
@ -1,7 +1,5 @@
|
||||||
# required permission to use tar to pack dumpsate_board.bin
|
|
||||||
allow hal_dumpstate_default vendor_toolbox_exec:file execute_no_trans;
|
allow hal_dumpstate_default vendor_toolbox_exec:file execute_no_trans;
|
||||||
allow hal_dumpstate_default radio_vendor_data_file:dir create_dir_perms;
|
allow hal_dumpstate_default radio_vendor_data_file:dir create_dir_perms;
|
||||||
allow hal_dumpstate_default radio_vendor_data_file:file create_file_perms;
|
allow hal_dumpstate_default radio_vendor_data_file:file create_file_perms;
|
||||||
allow hal_dumpstate_default shell_data_file:file getattr;
|
allow hal_dumpstate_default shell_data_file:file getattr;
|
||||||
set_prop(hal_dumpstate_default, vendor_logger_prop)
|
|
||||||
|
|
||||||
|
|
|
@ -1,3 +0,0 @@
|
||||||
# verbose property
|
|
||||||
vendor_internal_prop(vendor_logger_prop)
|
|
||||||
|
|
|
@ -1,3 +0,0 @@
|
||||||
# verbose property name
|
|
||||||
persist.vendor.verbose_logging_enabled u:object_r:vendor_logger_prop:s0
|
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue