chiark / gitweb /
~ian / secnet.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 3a1ad57) | 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, 15 Feb 2020 21:56:49 +0000 (21:56 +0000)
commitdffca6e2220c60519e52ac03438f37a0aa513293
treeb7cb818da90caa7d87b9701c5ae08480bd326fb1tree | snapshot
parent3a1ad577715b772da36b995f78089f4eeacf5a49commit | 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.