commit 4decfcd3b63c0bcc6ad9fc9b547a08320182ea13
author Kelvin Zhang <zhangkelvin@google.com> Wed Sep 07 03:21:25 2022 +0000
committer Kelvin Zhang <zhangkelvin@google.com> Wed Sep 07 03:23:25 2022 +0000
tree 9a56154ef9dde6319bab8d05c37eb16737643647
parent da4f72913fcbb0276aa9ca0fca55ce9afb2635c7

Fix a bug in total_data_length

If a partition only has 1 operation, it will have data_offset 0,
total_data_length will return 0, which is incorrect.

Bug: 227848550
Test: load an OTA with just 0 operation, make sure total_data_length > 0
Change-Id: I60ddb93c2f85647fa6baacbe791ee378092a5e0c

scripts/update_payload/payload.py

diff --git a/scripts/update_payload/payload.py b/scripts/update_payload/payload.py
index c63e5d1..cc15753 100644
--- a/scripts/update_payload/payload.py
+++ b/scripts/update_payload/payload.py
@@ -166,7 +166,7 @@
     # us total data length
     for partition in reversed(self.manifest.partitions):
       for op in reversed(partition.operations):
-        if op.data_offset > 0:
+        if op.data_length > 0:
           return op.data_offset + op.data_length
     return 0