chiark / gitweb /
~ian / secnet.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 03a807e) | 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>
Mon, 30 Dec 2019 13:15:49 +0000 (13:15 +0000)
commit17f154096491925d9538831e029a10a6be839c83
tree473e4143cc4da5a0df7e7bd0524388629a846c72tree | snapshot
parent03a807e6c3898dd56203862d825a891ae9b74738commit | 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.