From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:45352) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1fzHqW-0001BW-Ro for qemu-devel@nongnu.org; Mon, 10 Sep 2018 04:47:04 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1fzHeN-0002rZ-07 for qemu-devel@nongnu.org; Mon, 10 Sep 2018 04:34:27 -0400 Date: Mon, 10 Sep 2018 10:34:20 +0200 From: Kevin Wolf Message-ID: <20180910083420.GC4901@dhcp-200-186.str.redhat.com> References: <20180906142541.8238-1-berto@igalia.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20180906142541.8238-1-berto@igalia.com> Subject: Re: [Qemu-devel] [PATCH] block: Fix use after free error in bdrv_open_inherit() List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Alberto Garcia Cc: qemu-devel@nongnu.org, qemu-block@nongnu.org, Max Reitz , qemu-stable@nongnu.org Am 06.09.2018 um 16:25 hat Alberto Garcia geschrieben: > When a block device is opened with BDRV_O_SNAPSHOT and the > bdrv_append_temp_snapshot() call fails then the error code path tries > to unref the already destroyed 'options' QDict. > > This can be reproduced easily by setting TMPDIR to a location where > the QEMU process can't write: > > $ TMPDIR=/nonexistent $QEMU -drive driver=null-co,snapshot=on > > Signed-off-by: Alberto Garcia Thanks, applied to the block branch. But can we add the reproducer to some iotests case? Kevin