From: Juergen Perlinger <perlinger@ntp.org>
Date: Mon, 28 Sep 2015 16:22:06 +0000 (+0200)
Subject: [Bug 2909] - Slow memory leak in CRYPTO_ASSOC
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=cbe6c846c31f3ecb9dfc9d350ce5840d54b7f0c2;p=thirdparty%2Fntp.git

[Bug 2909] - Slow memory leak in CRYPTO_ASSOC
 - added missing call to 'free()' in ntp_crypto.c.

bk: 5609692evz8-Q9aKG_bPsoQxQrt6Wg
---

diff --git a/ChangeLog b/ChangeLog
index 266244bf2..9ca95350e 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -20,6 +20,7 @@
 * [Bug 2890] Ignore ENOBUFS on routing netlink socket.  Konstantin Khlebnikov.
 * [Bug 2906] make check needs better support for pthreads.  Harlan Stenn.
 * [Bug 2907] dist* build targets require our libevent/ to be enabled.  HStenn.
+* [Bug 2909] added missing call to 'free()' in ntp_crypto.c. perlinger@ntp.org
 * libntp/emalloc.c: Remove explicit include of stdint.h.  Harlan Stenn.
 * Put Unity CPPFLAGS items in unity_config.h.  Harlan Stenn.
 * tests/ntpd/g_leapsec.cpp typo fix.  Harlan Stenn.
diff --git a/ntpd/ntp_crypto.c b/ntpd/ntp_crypto.c
index e9cd6c203..a7b693f2e 100644
--- a/ntpd/ntp_crypto.c
+++ b/ntpd/ntp_crypto.c
@@ -508,6 +508,7 @@ crypto_recv(
 					rval = XEVNT_ERR;
 					break;
 				}
+				free(peer->cmmd); /* will be set again! */
 			}
 			fp = emalloc(len);
 			memcpy(fp, ep, len);