[Rpm-maint] [rpm-software-management/rpm] Check that ‘einfo.offset’ is reasonable (#1494)
Panu Matilainen
notifications at github.com
Wed Feb 17 10:43:05 UTC 2021
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.
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.
--
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-780467828
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20210217/9d725f22/attachment.html>
More information about the Rpm-maint
mailing list