proxmox-mirrors/mirror_lxc
1
0
Fork 0
mirror of https://git.proxmox.com/git/mirror_lxc synced 2025-08-08 02:31:33 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

utils: lxc_deslashify() free memory

Browse Source 
Make sure we always free any memory that was allocated by the call to
lxc_normalize_path().

Signed-off-by: Christian Brauner <christian.brauner@canonical.com>
This commit is contained in:
Christian Brauner 2016-09-26 22:05:54 +02:00
parent 7be721c185
commit f85b16a104
No known key found for this signature in database
GPG Key ID: 8EB056D53EECB12D
1 changed files with 13 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
src/lxc/utils.c
View File

@ -718,6 +718,7 @@ char **lxc_normalize_path(const char *path)
bool lxc_deslashify(char **path) bool lxc_deslashify(char **path)
{ {
bool ret = false;
char *p; char *p;
char **parts = NULL; char **parts = NULL;
size_t n, len; size_t n, len;
@ -729,28 +730,33 @@ bool lxc_deslashify(char **path)
/* We'll end up here if path == "///" or path == "". */ /* We'll end up here if path == "///" or path == "". */
if (!*parts) { if (!*parts) {
len = strlen(*path); len = strlen(*path);
if (!len) if (!len) {
return true; ret = true;
goto out;
}
n = strcspn(*path, "/"); n = strcspn(*path, "/");
if (n == len) { if (n == len) {
p = strdup("/"); p = strdup("/");
if (!p) if (!p)
return false; goto out;
free(*path); free(*path);
*path = p; *path = p;
return true; ret = true;
goto out;
} }
} }
p = lxc_string_join("/", (const char **)parts, **path == '/'); p = lxc_string_join("/", (const char **)parts, **path == '/');
lxc_free_array((void **)parts, free);
if (!p) if (!p)
return false; goto out;
free(*path); free(*path);
*path = p; *path = p;
ret = true;
return true; out:
lxc_free_array((void **)parts, free);
return ret;
} }
char *lxc_append_paths(const char *first, const char *second) char *lxc_append_paths(const char *first, const char *second)