From: simon Date: Sun, 4 Aug 2013 19:33:43 +0000 (+0000) Subject: Sanitise freeing of DSA keys. X-Git-Url: https://git.distorted.org.uk/~mdw/sgt/putty/commitdiff_plain/e6d62ec35f603edc0571b5e14e8b216c5e4ca098 Sanitise freeing of DSA keys. git-svn-id: svn://svn.tartarus.org/sgt/putty@9984 cda61777-01e9-0310-a592-d414129be87e --- diff --git a/sshdss.c b/sshdss.c index 6487d75a..8c9f93e3 100644 --- a/sshdss.c +++ b/sshdss.c @@ -111,6 +111,7 @@ static void *dss_newkey(char *data, int len) dss->q = getmp(&data, &len); dss->g = getmp(&data, &len); dss->y = getmp(&data, &len); + dss->x = NULL; return dss; } @@ -118,10 +119,16 @@ static void *dss_newkey(char *data, int len) static void dss_freekey(void *key) { struct dss_key *dss = (struct dss_key *) key; - freebn(dss->p); - freebn(dss->q); - freebn(dss->g); - freebn(dss->y); + if (dss->p) + freebn(dss->p); + if (dss->q) + freebn(dss->q); + if (dss->g) + freebn(dss->g); + if (dss->y) + freebn(dss->y); + if (dss->x) + freebn(dss->x); sfree(dss); }