[Rpm-maint] [rpm-software-management/rpm] Check that ‘einfo.offset’ is reasonable (#1494)

[Demi Marie Obenour]

> Nah, sprinkling checks on checks on checks when nothing bad actually happens only makes the code less readable in the end. Again, forest from the trees. Thanks for the detailed analysis of the case though.

You’re welcome!

> I'd much rather spend time treating causes than syndroms, such as looking into eliminating those pesky signed integers from the code to the extent possible. The region offset is the _only_ signed data in the header entries, but signed ints are used for accessing them all over the place.

What might that look like here?  I’m a bit confused.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/1494#issuecomment-780704610
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20210217/d204ac21/attachment.html>