diff options
| author | Jae Hoon Kim <kimjae@chromium.org> | 2020-06-23 10:24:03 -0700 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2020-06-25 22:47:20 +0000 |
| commit | 76583910d6685e6e75c6edaa3f17795e5d85405f (patch) | |
| tree | ed3c456ff294199a50f077cf5b97b39ba8a35930 /scripts/paycheck_unittest.py | |
| parent | f4d1196edca97a666e20d16a334aa67f47bbcf0a (diff) | |
update_engine: Skip post DownloadAction exclusions
Post |DownloadAction| don't have direct reference to |Payload|s held
within the |PayloadState|. Hence it's required to halt exclusions for
|Action|s post |DownloadAction|.
This is done by setting the |payload_index_| within |PayloadState| >= to
the |candidate_urls_|/|response_.packages| size.
DCHECKs added where |payload_index_| is used as usage may cause out of
bounds indexing.
This change removes the dangling reference to the last |Payload|, as
previously |NextPayload()| kept |payload_index_| pointing to the last
|Payload| within |PayloadState|.
BUG=chromium:928805
TEST=FEATURES=test emerge-$B update_engine
Change-Id: I3f6a9a3cc26bb84f94506e45e1d6e906624e5dd7
Reviewed-on: https://chromium-review.googlesource.com/c/aosp/platform/system/update_engine/+/2261292
Tested-by: Jae Hoon Kim <kimjae@chromium.org>
Reviewed-by: Amin Hassani <ahassani@chromium.org>
Commit-Queue: Amin Hassani <ahassani@chromium.org>
Auto-Submit: Jae Hoon Kim <kimjae@chromium.org>
Diffstat (limited to 'scripts/paycheck_unittest.py')
0 files changed, 0 insertions, 0 deletions