[SECNET PATCH 10/12] site.c: Don't overwrite `st->sharedsecret' if it's null.

[Ian Jackson]

From: Mark Wooding <mdw at distorted.org.uk>

In this case, `st->sharesecretlen' is zero, but this is still undefined
behaviour.

Signed-off-by: Mark Wooding <mdw at distorted.org.uk>
---
 site.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/site.c b/site.c
index 0da21ef..04e3a0d 100644
--- a/site.c
+++ b/site.c
@@ -1456,7 +1456,7 @@ static void enter_state_run(struct site *st)
     FILLZERO(st->remoteN);
     dispose_transform(&st->new_transform);
     memset(st->dhsecret,0,st->dh->len);
-    memset(st->sharedsecret,0,st->sharedsecretlen);
+    if (st->sharedsecret) memset(st->sharedsecret,0,st->sharedsecretlen);
     set_link_quality(st);
 
     if (st->keepalive && !current_valid(st))
-- 
2.11.0