diff options
| author | Simon Glass <sjg@chromium.org> | 2019-07-08 14:25:34 -0600 |
|---|---|---|
| committer | Simon Glass <sjg@chromium.org> | 2019-07-24 12:54:08 -0700 |
| commit | 7f9e00a2a6a3aff06572fdf225e51556f85853f4 (patch) | |
| tree | 6d9a744c781ae9938a9d0b25125de34d519b66d9 | |
| parent | 5b463fc26ea5a932d40845ec4eac6ff7e968756f (diff) | |
| download | u-boot-7f9e00a2a6a3aff06572fdf225e51556f85853f4.zip u-boot-7f9e00a2a6a3aff06572fdf225e51556f85853f4.tar.gz u-boot-7f9e00a2a6a3aff06572fdf225e51556f85853f4.tar.bz2 | |
binman: Call ProcessUpdateContents() consistently
SetContents() should only be called to set the contents of an entry from
within the ObtainContents() call, since it has no guard against increasing
the size of the contents, thus triggering incorrect operation.
Change all such calls to use ProcessUpdateContents() instead.
Signed-off-by: Simon Glass <sjg@chromium.org>
| -rw-r--r-- | tools/binman/etype/blob_dtb.py | 2 | ||||
| -rw-r--r-- | tools/binman/etype/fdtmap.py | 2 | ||||
| -rw-r--r-- | tools/binman/etype/fmap.py | 2 | ||||
| -rw-r--r-- | tools/binman/etype/image_header.py | 2 |
4 files changed, 4 insertions, 4 deletions
diff --git a/tools/binman/etype/blob_dtb.py b/tools/binman/etype/blob_dtb.py index cc5b4a3..d80c3d7 100644 --- a/tools/binman/etype/blob_dtb.py +++ b/tools/binman/etype/blob_dtb.py @@ -30,4 +30,4 @@ class Entry_blob_dtb(Entry_blob): def ProcessContents(self): """Re-read the DTB contents so that we get any calculated properties""" _, data = state.GetFdtContents(self._filename) - self.SetContents(data) + self.ProcessContentsUpdate(data) diff --git a/tools/binman/etype/fdtmap.py b/tools/binman/etype/fdtmap.py index cdeb491..ddac148 100644 --- a/tools/binman/etype/fdtmap.py +++ b/tools/binman/etype/fdtmap.py @@ -106,4 +106,4 @@ class Entry_fdtmap(Entry): This is necessary since new data may have been written back to it during processing, e.g. the image-pos properties. """ - self.SetContents(self._GetFdtmap()) + self.ProcessContentsUpdate(self._GetFdtmap()) diff --git a/tools/binman/etype/fmap.py b/tools/binman/etype/fmap.py index e6b5c5c..45d6db1 100644 --- a/tools/binman/etype/fmap.py +++ b/tools/binman/etype/fmap.py @@ -62,4 +62,4 @@ class Entry_fmap(Entry): return True def ProcessContents(self): - self.SetContents(self._GetFmap()) + self.ProcessContentsUpdate(self._GetFmap()) diff --git a/tools/binman/etype/image_header.py b/tools/binman/etype/image_header.py index 9bc84ec..d6de58c 100644 --- a/tools/binman/etype/image_header.py +++ b/tools/binman/etype/image_header.py @@ -73,4 +73,4 @@ class Entry_image_header(Entry): This is necessary since image_pos is not available when ObtainContents() is called, since by then the entries have not been packed in the image. """ - self.SetContents(self._GetHeader()) + self.ProcessContentsUpdate(self._GetHeader()) |