[Rpm-maint] [rpm-software-management/rpm] Add OpenSSL crypto backend (#129)
Tomáš Mráz
notifications at github.com
Mon Feb 6 15:53:37 UTC 2017
t8m requested changes on this pull request.
The various _set0_ functions you add for compatibility with OpenSSL 1.0.2 do not properly handle multiple calls of the same function on the object - the old references are not freed. I understand that in the current patch this probably isn't a problem but it might happen in future. It might be better to release the old references before assigning new ones.
--
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/129#pullrequestreview-20297097
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20170206/9479dea6/attachment-0001.html>
More information about the Rpm-maint
mailing list