Fix crash in vmbackup's abort timer.

VmBackupDoAbort cleans up the global backup state, so we shouldn't access
its fields after calling that function.

Signed-off-by: Marcelo Vanzin <mvanzin@vmware.com>

diff --git a/open-vm-tools/services/plugins/vmbackup/stateMachine.c b/open-vm-tools/services/plugins/vmbackup/stateMachine.c
index d3ab6a28e56adb1c168288bb654400d943761351..c8b8e6c7c96e69c2c6f12fe076a47a22e249fce8 100644 (file)
--- a/open-vm-tools/services/plugins/vmbackup/stateMachine.c
+++ b/open-vm-tools/services/plugins/vmbackup/stateMachine.c
@@ -358,9 +358,9 @@ VmBackupAbortTimer(gpointer data)
 {
    ASSERT(gBackupState != NULL);
    g_warning("Aborting backup operation due to timeout.");
-   VmBackupDoAbort();
    g_source_unref(gBackupState->abortTimer);
    gBackupState->abortTimer = NULL;
+   VmBackupDoAbort();
    return FALSE;
 }