[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH] virsh: add error return value

On 03/28/2012 02:24 PM, Alex Jia wrote:
* tools/virsh.c (cmdSnapshotList): Avoiding to continue to execute subsequent
   codes, the programming should return false when meeting a error.

For details, please see the following link:
RHBZ: https://bugzilla.redhat.com/show_bug.cgi?id=807545

Signed-off-by: Alex Jia<ajia redhat com>
  tools/virsh.c |    1 +
  1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/tools/virsh.c b/tools/virsh.c
index 5009b6b..7e74744 100644
--- a/tools/virsh.c
+++ b/tools/virsh.c
@@ -16374,6 +16374,7 @@ cmdSnapshotList(vshControl *ctl, const vshCmd *cmd)
          if (from) {
              vshError(ctl, "%s",
                       _("--roots and --from are mutually exclusive"));
+            return false;

Oh, you did post the patch for it. But the follow up patch for
the leaks should be buit on top of this then, it depends
on this patch.


But the fix is right, ACK

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]