[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH for-2.10] block/nfs: fix mutex assertion in nfs_
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-block] [PATCH for-2.10] block/nfs: fix mutex assertion in nfs_file_close() |
Date: |
Tue, 8 Aug 2017 13:59:44 +0200 |
User-agent: |
Mutt/1.8.3 (2017-05-23) |
Am 08.08.2017 um 00:29 hat Jeff Cody geschrieben:
> Commit c096358e747e88fc7364e40e3c354ee0bb683960 introduced assertion
> checks for when qemu_mutex() functions are called without the
> corresponding qemu_mutex_init() having initialized the mutex.
>
> This uncovered a latent bug in qemu's nfs driver - in
> nfs_client_close(), the NFSClient structure is overwritten with zeros,
> prior to the mutex being destroyed.
>
> Go ahead and destroy the mutex in nfs_client_close(), and change where
> we call qemu_mutex_init() so that it is correctly balanced.
>
> There are also a couple of memory leaks obscured by the memset, so this
> fixes those as well.
>
> Finally, we should be able to get rid of the memset(), as it isn't
> necessary.
>
> Signed-off-by: Jeff Cody <address@hidden>
Thanks, applied to the block branch.
Kevin