From 1e04a9f1502e70aea5d920bec69354cb85248ab5 Mon Sep 17 00:00:00 2001 From: Rom Walton Date: Sat, 22 Nov 2014 18:02:31 -0500 Subject: [PATCH] VBOX: Fix check-pointing. I broke it with my error path cleanup. --- samples/vboxwrapper/vbox_mscom_impl.cpp | 18 ++++++++---------- 1 file changed, 8 insertions(+), 10 deletions(-) diff --git a/samples/vboxwrapper/vbox_mscom_impl.cpp b/samples/vboxwrapper/vbox_mscom_impl.cpp index 0a485463c3..d31b63501c 100644 --- a/samples/vboxwrapper/vbox_mscom_impl.cpp +++ b/samples/vboxwrapper/vbox_mscom_impl.cpp @@ -1858,23 +1858,21 @@ int VBOX_VM::create_snapshot(double elapsed_time) { // Resume VM resume(); - if (ERR_EXEC != retval) goto CLEANUP; - // Set the suspended flag back to false before deleting the stale // snapshot poll(false); // Delete stale snapshot(s), if one exists - retval = cleanup_snapshots(false); - if (retval) return retval; + cleanup_snapshots(false); - fprintf( - stderr, - "%s Checkpoint completed.\n", - vboxwrapper_msg_prefix(buf, sizeof(buf)) - ); + if (BOINC_SUCCESS == retval) { + fprintf( + stderr, + "%s Checkpoint completed.\n", + vboxwrapper_msg_prefix(buf, sizeof(buf)) + ); + } -CLEANUP: return retval; }