diff options
author | Corinna Vinschen <corinna@vinschen.de> | 2008-01-19 15:14:45 +0100 |
---|---|---|
committer | Volker Lendecke <vl@samba.org> | 2008-01-19 19:30:16 +0100 |
commit | a0bd9d97a3a701fdb1f9a48ce925f63b786a8070 (patch) | |
tree | b5261d848d460038d61bc0d410552906514bfcb9 /source3/modules | |
parent | 70f88005c77cee0848a495da139144206b53eb3f (diff) | |
download | samba-a0bd9d97a3a701fdb1f9a48ce925f63b786a8070.tar.gz samba-a0bd9d97a3a701fdb1f9a48ce925f63b786a8070.tar.bz2 samba-a0bd9d97a3a701fdb1f9a48ce925f63b786a8070.zip |
Tiny memory leak in lib/version.c
Hi,
while implementing the extra_info version stuff, it occured to me that
samba_version_string() potentially allocates memory which is unused but
never free'd.
If SAMBA_VERSION_VENDOR_PATCH is defined, a second call to asprintf
takes place. The result is stored in tmp_version. Afterwards,
samba_version is set to tmp_version without free'ing samba_version
first. Looks like a simple free(samba_version) is missing. Patch
against 3.2-test below.
Ok, this only happens once over the lifetime of the application, so it's
no big deal, but I though it doesn't hurt to mention it.
Corinna
* lib/version.c (samba_version_string): Free samba_version
before setting to tmp_version.
(This used to be commit 373a23d48f2dd24e65dbf814ea58b4add2322128)
Diffstat (limited to 'source3/modules')
0 files changed, 0 insertions, 0 deletions