This is the mail archive of the
elfutils-devel@sourceware.org
mailing list for the elfutils project.
Re: [PATCH] On elf_update, remember when we mmap()
- From: Ulf Hermann <ulf dot hermann at qt dot io>
- To: Mark Wielaard <mark at klomp dot org>
- Cc: <elfutils-devel at sourceware dot org>
- Date: Fri, 28 Apr 2017 10:51:02 +0200
- Subject: Re: [PATCH] On elf_update, remember when we mmap()
- Authentication-results: sourceware.org; auth=none
- Authentication-results: sourceware.org; dkim=none (message not signed) header.d=none;sourceware.org; dmarc=none action=none header.from=qt.io;
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=qtcompany.onmicrosoft.com; s=selector1-qt-io; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=MG4kBEuLk4shrrhDJU+BtfYXdblvsxX/CQTsO79AUFI=; b=WiO6eY6M+1Otu5oEyxwZlRdGQi0SrlQRQ9Mzcm1hNspCCFSqaOlotKzyiUA+fD1l4DDnHXQJZ0KSoerDZKU73IW3aoJsSWu3dlUBnLh717IZqvj070aKakv+J0S3TrdMunFzk9G5xODlYEunJjMw+u4BeCAoFzElGgvw9+wj+sY=
- References: <a878a574-ac2e-c371-862e-254b83670df9@qt.io> <20170427222345.GF2061@stream>
- Spamdiagnosticmetadata: NSPM
- Spamdiagnosticoutput: 1:99
On 04/28/2017 12:23 AM, Mark Wielaard wrote:
> On Thu, Apr 20, 2017 at 04:57:41PM +0200, Ulf Hermann wrote:
>> Otherwise we skip the munmap() later. This leaks resources.
>
> Oops. Good find. Applied to master.
>
> When configured --with-valgrind the tests are run under valgrind
> and memory leaks will fail the tests. But since this is mmap
> valgrind won't report it. How did you find it?
On windows you cannot rename or unlink files if there are still open views on it (and the win32 view mechanism is how I implement mmap on windows). So, the test cases fail then because the directories are not empty.
Ulf