[lvm-devel] [PATCH 09/30] Reuse result of previous strchr.
Zdenek Kabelac
zkabelac at redhat.com
Mon Oct 25 08:24:16 UTC 2010
clang Argument with 'nonnull' attribute passed null
Reuse result of the last strchr call - it also makes sure,
'st' point is nonnull for next strchr call.
Signed-off-by: Zdenek Kabelac <zkabelac at redhat.com>
---
tools/toollib.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/tools/toollib.c b/tools/toollib.c
index 02c4962..5adc160 100644
--- a/tools/toollib.c
+++ b/tools/toollib.c
@@ -869,7 +869,7 @@ const char *extract_vgname(struct cmd_context *cmd, const char *lv_name)
while (*st == '/')
st++;
- if (!strchr(vg_name, '/') || strchr(st, '/')) {
+ if (!st || strchr(st, '/')) {
log_error("\"%s\": Invalid path for Logical Volume",
lv_name);
return 0;
--
1.7.3.1
More information about the lvm-devel
mailing list