All of lore.kernel.org
 help / color / mirror / Atom feed
From: Zdenek Kabelac <zkabelac@sourceware.org>
To: lvm-devel@redhat.com
Subject: main - cov: check setup_devices return code
Date: Fri, 23 Apr 2021 21:05:49 +0000 (GMT)	[thread overview]
Message-ID: <20210423210549.61071393BC13@sourceware.org> (raw)

Gitweb:        https://sourceware.org/git/?p=lvm2.git;a=commitdiff;h=d95b26fae0bdab7d306f0dff1d9328a697e041f3
Commit:        d95b26fae0bdab7d306f0dff1d9328a697e041f3
Parent:        ece80cd0fbce92d2ce248913fceffbf15e9d37f0
Author:        Zdenek Kabelac <zkabelac@redhat.com>
AuthorDate:    Thu Apr 22 12:34:16 2021 +0200
Committer:     Zdenek Kabelac <zkabelac@redhat.com>
CommitterDate: Fri Apr 23 23:00:55 2021 +0200

cov: check setup_devices return code

---
 tools/pvck.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tools/pvck.c b/tools/pvck.c
index 6bc1c091f..74663ec43 100644
--- a/tools/pvck.c
+++ b/tools/pvck.c
@@ -3187,8 +3187,8 @@ int pvck(struct cmd_context *cmd, int argc, char **argv)
 
 	if (argc == 1)
 		setup_device(cmd, argv[0]);
-	else
-		setup_devices(cmd);
+	else if (!setup_devices(cmd))
+		return ECMD_FAILED;
 
 	for (i = 0; i < argc; i++) {
 		pv_name = argv[i];



                 reply	other threads:[~2021-04-23 21:05 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20210423210549.61071393BC13@sourceware.org \
    --to=zkabelac@sourceware.org \
    --cc=lvm-devel@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.