From: "Richard W.M. Jones" <rjones(a)redhat.com>
Although the code is technically correct (because the successful call
to $g->close won't overwrite $@ and because guestfs_close can never
fail) it's still confusing to assign $err = $@ and then not use $err.
Die with $err. It's possible something added here could overwrite $@
at some later time.
---
p2v/server/virt-p2v-server.pl | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/p2v/server/virt-p2v-server.pl b/p2v/server/virt-p2v-server.pl
index c15efb2..0121ab2 100755
--- a/p2v/server/virt-p2v-server.pl
+++ b/p2v/server/virt-p2v-server.pl
@@ -405,7 +405,7 @@ sub convert
my $err = $@;
$g->close() if defined($g);
- die($@);
+ die($err);
}
p2v_return_ok();
--
1.8.3.1