[PATCH 09/30] Reuse result of previous strchr.
Zdenek Kabelac <[email protected]>
| Newsgroups | dev.linux.lists.lvm-devel |
|---|---|
| Message-ID | <1ded298d3c909b9ebdcedcf9c275c201c9e4861c.1287994530.git.zkabelac@redhat.com> |
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 <[email protected]> --- 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