@DemiMarie requested changes on this pull request.


> +     if (sigalg->setmpi(sigalg, i, p))
+           break;

@pmatilai good point.  In fact, I would argue that *not* adding the check would 
make this PR a regression security wise.  Would it be possible to include #1705 
in this PR?  It is tiny and passes the regression test suite, and can be 
replaced with a better solution later.

-- 
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/1795#pullrequestreview-782356055
_______________________________________________
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint

Reply via email to