diff options
author | Simon Glass <sjg@chromium.org> | 2020-10-26 17:40:05 -0600 |
---|---|---|
committer | Simon Glass <sjg@chromium.org> | 2020-10-29 14:42:59 -0600 |
commit | e6bed4f1812c1666eeab794694c4b3765f41c143 (patch) | |
tree | 639acab5b7366af485b6029e69cdc30312529de4 /tools | |
parent | 72628cdf58ec2737637b41ec3dad4a30fe5090e0 (diff) | |
download | u-boot-e6bed4f1812c1666eeab794694c4b3765f41c143.zip u-boot-e6bed4f1812c1666eeab794694c4b3765f41c143.tar.gz u-boot-e6bed4f1812c1666eeab794694c4b3765f41c143.tar.bz2 |
binman: Expand the error message for breaching a section
Add in a few more details to this error message to make it easier to see
what is going on.
Signed-off-by: Simon Glass <sjg@chromium.org>
Diffstat (limited to 'tools')
-rw-r--r-- | tools/binman/etype/section.py | 10 | ||||
-rw-r--r-- | tools/binman/ftest.py | 5 |
2 files changed, 9 insertions, 6 deletions
diff --git a/tools/binman/etype/section.py b/tools/binman/etype/section.py index 3277504..a3e37c3 100644 --- a/tools/binman/etype/section.py +++ b/tools/binman/etype/section.py @@ -220,10 +220,12 @@ class Entry_section(Entry): if (entry.offset < self._skip_at_start or entry.offset + entry.size > self._skip_at_start + self.size): - entry.Raise("Offset %#x (%d) is outside the section starting " - "at %#x (%d)" % - (entry.offset, entry.offset, self._skip_at_start, - self._skip_at_start)) + entry.Raise('Offset %#x (%d) size %#x (%d) is outside the ' + "section '%s' starting at %#x (%d) " + 'of size %#x (%d)' % + (entry.offset, entry.offset, entry.size, entry.size, + self._node.path, self._skip_at_start, + self._skip_at_start, self.size, self.size)) if entry.offset < offset and entry.size: entry.Raise("Offset %#x (%d) overlaps with previous entry '%s' " "ending at %#x (%d)" % diff --git a/tools/binman/ftest.py b/tools/binman/ftest.py index adc1603..4f7e226 100644 --- a/tools/binman/ftest.py +++ b/tools/binman/ftest.py @@ -970,8 +970,9 @@ class TestFunctional(unittest.TestCase): """Test that the end-at-4gb property checks for offset boundaries""" with self.assertRaises(ValueError) as e: self._DoTestFile('028_pack_4gb_outside.dts') - self.assertIn("Node '/binman/u-boot': Offset 0x0 (0) is outside " - "the section starting at 0xffffffe0 (4294967264)", + self.assertIn("Node '/binman/u-boot': Offset 0x0 (0) size 0x4 (4) " + "is outside the section '/binman' starting at " + '0xffffffe0 (4294967264) of size 0x20 (32)', str(e.exception)) def testPackX86Rom(self): |