[Rpm-maint] [rpm-software-management/rpm] Harden against crafted inputs (#1471)

Demi Marie Obenour notifications at github.com
Mon Jan 11 09:45:51 UTC 2021


@DemiMarie commented on this pull request.



> @@ -169,8 +169,8 @@ rpmRC rpmpkgRead(struct rpmvs_s *vs, FD_t fd,
 	goto exit;
     }
 
-    /* Read the signature header. Might not be in a contiguous region. */
-    if (hdrblobRead(fd, 1, 0, RPMTAG_HEADERSIGNATURES, sigblob, &msg))
+    /* Read the signature header. Must be in a contiguous region. */
+    if (hdrblobRead(fd, 1, 1, RPMTAG_HEADERSIGNATURES, sigblob, &msg))

Closing this.  It turns out that the relevant code is at least somewhat robust (it passed fuzzing with libfuzzer for a couple of hours with no bugs found).  I still recommend removing it from the attack surface, but for now there are bigger wins to be had elsewhere.

-- 
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/1471#discussion_r554923997
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20210111/8a343b73/attachment.html>


More information about the Rpm-maint mailing list