diff options
| author | Jae Hoon Kim <kimjae@chromium.org> | 2020-05-07 11:32:44 -0700 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2020-05-10 23:13:45 +0000 |
| commit | a3210e6c3b2a78dffdb8f5544ed80819b538a302 (patch) | |
| tree | b5812c8b6508129d0fdd3ce254c906ae9c2a986a /update_status_utils_unittest.cc | |
| parent | 065d78d6963ca13a38ad305bf751b09ec929cf51 (diff) | |
update_engine: NextPayload() resets URL index
This change fixes a bug/assumption that |PayloadState| used to make in
regards to URL index related to payloads. When a URL index is
incremented, there is no gurauntee that subsequent payloads will have
the same number of candidate URLs, hence it is critical to reset the URL
index back to 0 for subsequent payloads. This fix also allows candidate
URLs to not be skipped over for multi-package/payload request/responses.
The max number of times a URL is allowed to fail is reduced from 10 to 3
to allow preferred URLs to always be used as the intial URL for payloads.
BUG=chromium:928805
TEST=FEATURES=test emerge-$B update_engine # filter PayloadStateTest
Change-Id: I67732b2b7da08f580d1b554fd85eb06b3bf1f761
Reviewed-on: https://chromium-review.googlesource.com/c/aosp/platform/system/update_engine/+/2188552
Tested-by: Jae Hoon Kim <kimjae@chromium.org>
Commit-Queue: Andrew Lassalle <andrewlassalle@chromium.org>
Reviewed-by: Amin Hassani <ahassani@chromium.org>
Diffstat (limited to 'update_status_utils_unittest.cc')
0 files changed, 0 insertions, 0 deletions