]> git.ipfire.org Git - thirdparty/lxc.git/commit
lxc/utils: bugfix freed pointer return value 358/head
authorSilvio Fricke <silvio.fricke@gmail.com>
Fri, 14 Nov 2014 19:56:12 +0000 (20:56 +0100)
committerSilvio Fricke <silvio.fricke@gmail.com>
Wed, 19 Nov 2014 17:14:27 +0000 (18:14 +0100)
commita1c9a9c97840b1b6b4e7a1ea9cf75aeb2f815659
tree4e4bfbab9d75cea28d11ab52f6786e34fdcaa939
parent40580b462409d37bf73e2fdda9b7843e807e545e
lxc/utils: bugfix freed pointer return value

We allocate a pointer and save this address in a static variable. After
this we freed this pointer and return.

Here a cuttout of a valgrind report:

[...]
==11568== Invalid read of size 1
==11568==    at 0x4C2D524: strlen (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==11568==    by 0x5961C9B: puts (in /usr/lib/libc-2.20.so)
==11568==    by 0x400890: main (lxc_config.c:73)
==11568==  Address 0x6933e21 is 1 bytes inside a block of size 32 free'd
==11568==    at 0x4C2B200: free (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==11568==    by 0x4E654F2: lxc_global_config_value (utils.c:415)
==11568==    by 0x4E92177: lxc_get_global_config_item (lxccontainer.c:2287)
==11568==    by 0x400883: main (lxc_config.c:71)
[...]

Signed-off-by: Silvio Fricke <silvio.fricke@gmail.com>
src/lxc/utils.c