tequilaOS/platform_system_security
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

[Compos Telemetry] Fix current_artifacts_ok in the pushed atom

Browse source 
In cases of no pending artifacts exist, current_artifacts_ok is not set
in the pushed atom. Fix this by checking the returned value of
checkArtifacts()

This is upstreaming ag/18580936

Bug: 233740445
Test: build succeds
Merged-In: I2d7ce37960c2c0f1adea15bde06d21947668b4a4

Change-Id: I2d7ce37960c2c0f1adea15bde06d21947668b4a4
This commit is contained in:
Shikha Panwar 2022-05-24 14:52:20 +00:00
parent e2c4f0fcf4
commit 4d7f9056c8
1 changed files with 5 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
ondevice-signing/odsign_main.cpp
View file

@ -374,7 +374,11 @@ art::odrefresh::ExitCode CheckCompOsPendingArtifacts(const SigningKey& signing_k
if (!directoryHasContent(kCompOsPendingArtifactsDir)) {
// No pending CompOS artifacts, all that matters is the current ones.
return checkArtifacts();
art::odrefresh::ExitCode odrefresh_status = checkArtifacts();
if (odrefresh_status == art::odrefresh::ExitCode::kOkay) {
compos_check_record->current_artifacts_ok = true;
}
return odrefresh_status;
}
compos_check_record->comp_os_pending_artifacts_exists = true;