|Summary:||ASTERISK-07080: [patch] [need disclaimer] memory leakage in callerid.c|
|Reporter:||Konstantin Prokazoff (oryx)||Labels:|
|Date Opened:||2006-06-02 01:49:00||Date Closed:||2006-06-06 11:04:52|
|Environment:||Attachments:||( 0) alamantia_patch01.diff|
|Description:||Malloc'ed memory in some cases willn't be freed at return from function.|
|Comments:||By: Serge Vecher (serge-v) 2006-06-02 08:31:42|
any more details that you can provide: a code snippet, perhaps? Why do you believe there is a memory leakage?
By: Konstantin Prokazoff (oryx) 2006-06-02 09:12:10
If you'll analyze source, you'll see.
I'll post patch asap.
By: Anthony LaMantia (alamantia) 2006-06-02 10:51:28
free(obuf); needs to be placed on lines
on all of these returns the allocated memory for obuf
short *buf = malloc(2 * len + cid->oldlen);
short *obuf = buf;
is never free()'ed
directly after the malloc call, it is checked to see if it has been executaed safley on line 267.
so at all of the return's after that check obuf should be free()'ed to prevent memory problems.
I will post a patch later today.
By: Anthony LaMantia (alamantia) 2006-06-02 10:58:24
i uploaded the .diff.
By: Anthony LaMantia (alamantia) 2006-06-02 11:24:17
let me update the diff from the latest SNV code.
By: BJ Weschke (bweschke) 2006-06-02 11:55:09
alamantia - Oryx - good catches. You guys have disclaimers on file?
By: Anthony LaMantia (alamantia) 2006-06-02 14:52:35
i dont have a disclaimer, but feel free to do what you want with the diff.
By: Russell Bryant (russell) 2006-06-04 16:16:13
Unfortunately, just making that statement is not enough. We need a disclaimer on file before we can review your proposed code changes. Thanks ...
By: Serge Vecher (serge-v) 2006-06-05 09:00:19
alamantia: see the bottom of this page for instructions http://bugs.digium.com/main_page.php . Thanks
By: Serge Vecher (serge-v) 2006-06-05 09:00:37
By: Brian West (bkw918) 2006-06-05 12:16:16
This patch shouldn't need a disclaimer at all unless someone magically took over the copyright to all "free" calls in the world.
By: Brian West (bkw918) 2006-06-05 12:16:57
He is adding THREE FREAKING calls to "free" and you need a disclaimer?
By: Tilghman Lesher (tilghman) 2006-06-06 11:04:52
Patch is simply functional and is not large enough to be copyrightable. Committed to 1.2, merged to trunk.