chiark / gitweb /
~ian / secnet.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 81396f9) | patch
rsa1: rsa_loadpriv_core: Make assume_valid do the frees
authorIan Jackson <ijackson@chiark.greenend.org.uk>
Sun, 6 Oct 2019 21:12:06 +0000 (22:12 +0100)
committerIan Jackson <ijackson@chiark.greenend.org.uk>
Sat, 7 Dec 2019 17:08:51 +0000 (17:08 +0000)
commit029fec2038d1611f69fabd65bd2ee3fa28712fe7
treecc53bafe6f2898982eb3231bd065a783ce37c19ctree | snapshot
parent81396f94b4a2863fadbacdd1ffbf3f15fd45ee2dcommit | diff
rsa1: rsa_loadpriv_core: Make assume_valid do the frees

This regularises the exit path: now we always do these clears.  This
makes the function easier to think about and easier to amend in the
way we're just about to.

There is no overall functional change since these variables are
initialised with mpz_init right at the top, even on the assume_valid
path.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
rsa.c diff | blob | history
Ian's personal secnet repo on chiark, for work-in-progress stuff etc.