All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH BlueZ] unit/test-queue: Fix memory leak
@ 2014-06-27 10:52 Luiz Augusto von Dentz
  2014-06-27 11:08 ` Luiz Augusto von Dentz
  0 siblings, 1 reply; 2+ messages in thread
From: Luiz Augusto von Dentz @ 2014-06-27 10:52 UTC (permalink / raw)
  To: linux-bluetooth

From: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>

This fixes the following memory leak:

32 bytes in 1 blocks are definitely lost in loss record 20 of 35
   at 0x4C291D4: calloc (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
   by 0x400DC2: queue_new (queue.c:65)
   by 0x400AA5: test_foreach_remove_all (test-queue.c:92)
   by 0x4E9E5E0: ??? (in /usr/lib64/libglib-2.0.so.0.3800.2)
   by 0x4E9E7A5: ??? (in /usr/lib64/libglib-2.0.so.0.3800.2)
   by 0x4E9EB1A: g_test_run_suite (in /usr/lib64/libglib-2.0.so.0.3800.2)
   by 0x40083E: main (test-queue.c:109)
---
 unit/test-queue.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/unit/test-queue.c b/unit/test-queue.c
index af7dfd3..5cbe529 100644
--- a/unit/test-queue.c
+++ b/unit/test-queue.c
@@ -96,6 +96,7 @@ static void test_foreach_remove_all(void)
 	queue_push_tail(queue, UINT_TO_PTR(2));
 
 	queue_foreach(queue, foreach_remove_all, queue);
+	queue_destroy(queue, NULL);
 }
 
 int main(int argc, char *argv[])
-- 
1.9.3


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH BlueZ] unit/test-queue: Fix memory leak
  2014-06-27 10:52 [PATCH BlueZ] unit/test-queue: Fix memory leak Luiz Augusto von Dentz
@ 2014-06-27 11:08 ` Luiz Augusto von Dentz
  0 siblings, 0 replies; 2+ messages in thread
From: Luiz Augusto von Dentz @ 2014-06-27 11:08 UTC (permalink / raw)
  To: linux-bluetooth

Hi,

On Fri, Jun 27, 2014 at 1:52 PM, Luiz Augusto von Dentz
<luiz.dentz@gmail.com> wrote:
> From: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>
>
> This fixes the following memory leak:
>
> 32 bytes in 1 blocks are definitely lost in loss record 20 of 35
>    at 0x4C291D4: calloc (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
>    by 0x400DC2: queue_new (queue.c:65)
>    by 0x400AA5: test_foreach_remove_all (test-queue.c:92)
>    by 0x4E9E5E0: ??? (in /usr/lib64/libglib-2.0.so.0.3800.2)
>    by 0x4E9E7A5: ??? (in /usr/lib64/libglib-2.0.so.0.3800.2)
>    by 0x4E9EB1A: g_test_run_suite (in /usr/lib64/libglib-2.0.so.0.3800.2)
>    by 0x40083E: main (test-queue.c:109)
> ---
>  unit/test-queue.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/unit/test-queue.c b/unit/test-queue.c
> index af7dfd3..5cbe529 100644
> --- a/unit/test-queue.c
> +++ b/unit/test-queue.c
> @@ -96,6 +96,7 @@ static void test_foreach_remove_all(void)
>         queue_push_tail(queue, UINT_TO_PTR(2));
>
>         queue_foreach(queue, foreach_remove_all, queue);
> +       queue_destroy(queue, NULL);
>  }
>
>  int main(int argc, char *argv[])
> --
> 1.9.3

Pushed.


-- 
Luiz Augusto von Dentz

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2014-06-27 11:08 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-06-27 10:52 [PATCH BlueZ] unit/test-queue: Fix memory leak Luiz Augusto von Dentz
2014-06-27 11:08 ` Luiz Augusto von Dentz

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.