[prev in list] [next in list] [prev in thread] [next in thread]
List: lvm2-commits
Subject: [lvm2-commits] master - libdm: improve check in dm_split_lvm_name
From: Zdenek Kabelac <zkabelac () fedoraproject ! org>
Date: 2013-04-21 21:17:58
Message-ID: 20130421211758.7D8E3903 () fedorahosted ! org
[Download RAW message or body]
Gitweb: http://git.fedorahosted.org/git/?p=lvm2.git;a=commitdiff;h=8f1dd00c3634d111be9a42524ab50b2f31962530
Commit: 8f1dd00c3634d111be9a42524ab50b2f31962530
Parent: a2b76a6f022920abd59072decfa7d3d76eeda913
Author: Zdenek Kabelac <zkabelac@redhat.com>
AuthorDate: Sun Apr 21 12:48:24 2013 +0200
Committer: Zdenek Kabelac <zkabelac@redhat.com>
CommitterDate: Sun Apr 21 23:10:43 2013 +0200
libdm: improve check in dm_split_lvm_name
We support both type of call - with or without mem pool.
So ensure we will not use NULL vgname also when mem is given.
---
libdm/libdm-string.c | 5 ++++-
1 files changed, 4 insertions(+), 1 deletions(-)
diff --git a/libdm/libdm-string.c b/libdm/libdm-string.c
index 5ef6334..9d667bb 100644
--- a/libdm/libdm-string.c
+++ b/libdm/libdm-string.c
@@ -92,7 +92,10 @@ static char *_unquote(char *component)
int dm_split_lvm_name(struct dm_pool *mem, const char *dmname,
char **vgname, char **lvname, char **layer)
{
- if (mem && !(*vgname = dm_pool_strdup(mem, dmname)))
+ if (mem)
+ *vgname = dm_pool_strdup(mem, dmname);
+
+ if (!*vgname)
return 0;
_unquote(*layer = _unquote(*lvname = _unquote(*vgname)));
_______________________________________________
lvm2-commits mailing list
lvm2-commits@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/lvm2-commits
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic