2007-08-31 17:28:29

by Manuel Naranjo

[permalink] [raw]
Subject: [Bluez-devel] [PATCH] SDP lib memory leak

_______________________________________________
Bluez-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/bluez-devel


Attachments:
leak.patch (1.05 kB)
(No filename) (315.00 B)
(No filename) (164.00 B)
Download all attachments

2007-08-31 18:45:20

by Marcel Holtmann

[permalink] [raw]
Subject: Re: [Bluez-devel] [PATCH] SDP lib memory leak

Hi Manuel,

> I have made an application that makes use of the sdp lib. By
> running the app with valgrind [1] I found a memory leak.
> There's an static variable called bluetooth_base_uuid that's being
> initializated once but then never freed.
> I had added a method that should be called, only when the thread is
> about to end that frees that variable.
> Attached you can see the patch.

thanks for the patch, but I put a different patch in the CVS that avoids
any memory allocation at all.

Can you extract me a test program from your application, so I can double
check that it is doing the right thing on little and big endian systems
and we don't have any memory leaks at all anymore.

Regards

Marcel



-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems? Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/
_______________________________________________
Bluez-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/bluez-devel