From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:37964) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QcJS0-00068j-J4 for qemu-devel@nongnu.org; Thu, 30 Jun 2011 11:46:46 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1QcJRy-0006nB-SZ for qemu-devel@nongnu.org; Thu, 30 Jun 2011 11:46:44 -0400 Received: from mail-qy0-f180.google.com ([209.85.216.180]:39073) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QcJRy-0006n6-Ke for qemu-devel@nongnu.org; Thu, 30 Jun 2011 11:46:42 -0400 Received: by qyk30 with SMTP id 30so1397457qyk.4 for ; Thu, 30 Jun 2011 08:46:41 -0700 (PDT) Sender: Paolo Bonzini From: Paolo Bonzini Date: Thu, 30 Jun 2011 17:46:17 +0200 Message-Id: <1309448777-1447-5-git-send-email-pbonzini@redhat.com> In-Reply-To: <1309448777-1447-1-git-send-email-pbonzini@redhat.com> References: <1309448777-1447-1-git-send-email-pbonzini@redhat.com> Subject: [Qemu-devel] [RFC PATCH 4/4] Partially revert "savevm: fix corruption in vmstate_subsection_load()." List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org This reverts the additional check in commit eb60260d (but not the assertions). The new format does not require the check, and with the old format it traded one kind of bogus failure for a different kind of silent failure. Signed-off-by: Paolo Bonzini --- savevm.c | 4 ---- 1 files changed, 0 insertions(+), 4 deletions(-) diff --git a/savevm.c b/savevm.c index 654770a..6c726ec 100644 --- a/savevm.c +++ b/savevm.c @@ -1679,10 +1679,6 @@ static int vmstate_subsection_load(QEMUFile *f, const VMStateDescription *vmsd, { const VMStateSubsection *sub = vmsd->subsections; - if (!sub || !sub->needed) { - return 0; - } - while (qemu_peek_byte(f) == QEMU_VM_SUBSECTION) { char idstr[256]; int ret; -- 1.7.5.2