<p>Ehm, maybe we have a terminology problem here. When I talk about signature header immutable region, I mean HEADER_SIGNATURES region. That's not used for digests or signatures and when adding/removing signatures the signature header changes more fundamentally than just padding adjustments. So yes if you look at it in terms of what might change, only the build-time added digests and sizes etc should go into the region and actual signatures etc that might be added over time should go outside it, but rpm has "forever" reloaded the signature into a region after each signature add/remove and all. Padding only changes when signatures are added or removed, so technically it can just as well be in the region (or not). So yes the region doesn't mean a whole lot in the signature header...</p>
<p>As for the bug allegation, the details are right there in my previous posts: the code (copy-pasted from rpm5 as you must've noticed) calls headerReload() after adding the padding, which I read as "make it a single (immutable) region". And despite that, the padding ends up being outside the region. I'm not arguing whether it should be in the region or not, just observing that the result doesn't match what was indicated by the code, which to me seems like a possible bug someplace, which I'm pointing out as in "you might want to look at that", no ill-will / FUD / whatever intended. If I'm mistaken then by golly apologies for speculating the possibility of lurking bugs in a piece of software.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/rpm-software-management/rpm/issues/270#issuecomment-347774166">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ANb80-gN5Sjh_gk9sdFGdHSS806wwEN9ks5s7QWNgaJpZM4Oeptu">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ANb806dYb5h3POBDGXfa8zdU3rmwebtQks5s7QWNgaJpZM4Oeptu.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/rpm-software-management/rpm/issues/270#issuecomment-347774166"></link>
  <meta itemprop="name" content="View Issue"></meta>
</div>
<meta itemprop="description" content="View this Issue on GitHub"></meta>
</div>

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/rpm-software-management/rpm","title":"rpm-software-management/rpm","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/rpm-software-management/rpm"}},"updates":{"snippets":[{"icon":"PERSON","message":"@pmatilai in #270: Ehm, maybe we have a terminology problem here. When I talk about signature header immutable region, I mean HEADER_SIGNATURES region. That's not used for digests or signatures and when adding/removing signatures the signature header changes more fundamentally than just padding adjustments. So yes if you look at it in terms of what might change, only the build-time added digests and sizes etc should go into the region and actual signatures etc that might be added over time should go outside it, but rpm has \"forever\" reloaded the signature into a region after each signature add/remove and all. Padding only changes when signatures are added or removed, so technically it can just as well be in the region (or not). So yes the region doesn't mean a whole lot in the signature header...\r\n\r\nAs for the bug allegation, the details are right there in my previous posts: the code (copy-pasted from rpm5 as you must've noticed) calls headerReload() after adding the padding, which I read as \"make it a single (immutable) region\". And despite that, the padding ends up being outside the region. I'm not arguing whether it should be in the region or not, just observing that the result doesn't match what was indicated by the code, which to me seems like a possible bug someplace, which I'm pointing out as in \"you might want to look at that\", no ill-will / FUD / whatever intended. If I'm mistaken then by golly apologies for speculating the possibility of lurking bugs in a piece of software. "}],"action":{"name":"View Issue","url":"https://github.com/rpm-software-management/rpm/issues/270#issuecomment-347774166"}}}</script>