- 05 Apr, 2013 4 commits
-
-
Andrey Vagin authored
CID 996205 (#1-2 of 2): Resource leak (RESOURCE_LEAK) 14. leaked_storage: Variable "core" going out of scope leaks the storage it points to. Signed-off-by:
Andrey Vagin <avagin@openvz.org> Signed-off-by:
Pavel Emelyanov <xemul@parallels.com>
-
Andrey Vagin authored
Signed-off-by:
-
Andrey Vagin authored
CID 996206 (#3 of 3): Resource leak (RESOURCE_LEAK) 8. leaked_handle: Handle variable "sk" going out of scope leaks the handle. Signed-off-by:
-
Andrey Vagin authored
CID 996207 (#1 of 1): Out-of-bounds access (OVERRUN) 5. alloc_strlen: Allocating insufficient memory for the terminating null of the string. Signed-off-by:
-
- 04 Apr, 2013 20 commits
-
-
Cyrill Gorcunov authored
simfs is used in OpenVZ containers, so lets understand it and don't fail on its meeting. Signed-off-by:
Cyrill Gorcunov <gorcunov@openvz.org> Signed-off-by:
-
Cyrill Gorcunov authored
Signed-off-by:
-
Cyrill Gorcunov authored
No need to include all types here. And this allows to reuse this header outside of crtools code. Signed-off-by:
-
Alexander Kartashov authored
The macro PAGE_SIZE is declared in the following way in Debian: #define PAGE_SIZE (sysconf(_SC_PAGESIZE)) that makes it impossible to use the macro as the size of the array clone_args::stack since this generates the following error: cc -g -O2 -Wall -Werror -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=0 -iquote ../../lib/arch/arm/include -I../../lib session02.c ../../lib/libzdtmtst.a -o session02 session02.c:70:7: error: variably modified 'stack' at file scope Cc: Andrey Vagin <avagin@openvz.org> Signed-off-by:
Alexander Kartashov <alekskartashov@parallels.com> Acked-by:
Andrew Vagin <avagin@parallels.com> Signed-off-by:
-
Alexander Kartashov authored
Use a PRI* format specifier to convert an integer of known size to a string. Cc: Andrey Vagin <avagin@openvz.org> Signed-off-by:
-
Andrey Vagin authored
pstree.c:273:3: warning: Value stored to 'ret' is never read ret = 0; ^ ~ Signed-off-by: -
Andrey Vagin authored
files.c:159:21: error: comparison of unsigned expression < 0 is always false [-Werror,-Wtautological-compare] if (p->fown.signum < 0) { ~~~~~~~~~~~~~~ ^ ~ Signed-off-by: -
Andrey Vagin authored
cr-restore.c:1795:2: warning: Value stored to 'restore_task_vma_len' is never read restore_task_vma_len = 0; ^ ~ cr-restore.c:1796:2: warning: Value stored to 'restore_thread_vma_len' is never read restore_thread_vma_len = 0; Signed-off-by: -
Andrey Vagin authored
ipc_ns.c:464:3: warning: Memory is never released; potential leak of memory pointed to by 'values' return; ^ 1 warning generated. Signed-off-by: -
Andrey Vagin authored
pie/parasite.c:308:3: warning: Value stored to 'ret' is never read ret = -1; Signed-off-by: -
Andrey Vagin authored
Signed-off-by:
-
Andrey Vagin authored
Signed-off-by:
-
Andrey Vagin authored
Signed-off-by:
-
Andrey Vagin authored
Signed-off-by:
-
Andrey Vagin authored
Signed-off-by:
-
Andrey Vagin authored
Signed-off-by:
-
Andrey Vagin authored
clang requires that. In file included from arch/x86/cpu.c:9: arch/x86/include/asm/bitops.h:24:15: error: ambiguous instructions require an explicit suffix (could be 'btsw', 'btsl', or 'btsq') asm volatile("bts %1,%0" : ADDR : "Ir" (nr) : "memory"); Signed-off-by: -
Andrey Vagin authored
clang-analyzer sets the environment variable CC scan-build -plist-html --use-analyzer /usr/bin/clang make Signed-off-by:
-
Pavel Emelyanov authored
If any task has a sysvipc mapping we should make sure, that the ipc namespace is dumped as well. Otherwise after restore the task will die. Signed-off-by: -
Pavel Emelyanov authored
Absense of packet and netlink diag is OK at dump -- we can be on older kernel and will fail later if _really_ need these sockets. During check stage all diag is expected to be in place. Signed-off-by:
-
- 02 Apr, 2013 15 commits
-
-
Pavel Emelyanov authored
When dumping a unix socket we must make sure, that all connections are dumped or -x option is used. Currently only peers with non-matching ids are added into check. Fix this. Signed-off-by: -
Cyrill Gorcunov authored
To reuse them outside of crtools code. Signed-off-by:
-
Cyrill Gorcunov authored
This allows to use it in external projects outside of types.h header, moreover it's close to what kernel does too. Signed-off-by:
-
Cyrill Gorcunov authored
This diet types.h and allows to include BITS_PER_LONG only where really needed. It makes BITS_PER_LONG definition close to one used in kernel. Signed-off-by:
-
Cyrill Gorcunov authored
Lets stick to names used in kernel. Signed-off-by:
-
Cyrill Gorcunov authored
hlist is suitable for hash operations so extent list.h code. I need it for converter and this allow me to reuse this header. Signed-off-by:
-
Cyrill Gorcunov authored
This header consists of a number of constants and a few macro helper. No need to make it asm/types.h dependant. Instead only include <stdbool.h> for bool definition as required by forward fdinfo_per_id (I think this declaration should live in somewhere else place, not sure where yet, so I left it untouched). Signed-off-by:
-
Cyrill Gorcunov authored
Instead of bloating util.h lets move ERR_ helpers to own err.h header. This allow to reuse it where needed without util.h inclusion. Signed-off-by:
-
Cyrill Gorcunov authored
Signed-off-by:
-
Pavel Emelyanov authored
Multi-task/thread apps spoil log as simple logger does char-by-char output. Signed-off-by: -
Pavel Emelyanov authored
Signed-off-by: -
Pavel Emelyanov authored
This info is not in per-ID file, not per-PID, so need special care. Signed-off-by: -
Pavel Emelyanov authored
Show only image info relevant to given pid. Also reuse the introduced --pid argument for exec action. Signed-off-by: -
Pavel Emelyanov authored
Print delimiters and image names for easier reading of the output. Signed-off-by: -
Pavel Emelyanov authored
Threads' IDs were not initialized on img load. Signed-off-by:
-
- 01 Apr, 2013 1 commit
-
-
Andrey Vagin authored
It's required for per-test cleaning. Signed-off-by:
-
