[PATCH] ntdll: Avoid leaking "drive" on error paths from get_mountmgr_fs_info() (Valgrind).

[Zebediah Figura]

Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com>
---
 dlls/ntdll/file.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/dlls/ntdll/file.c b/dlls/ntdll/file.c
index b5a48db176..1b70e47367 100644
--- a/dlls/ntdll/file.c
+++ b/dlls/ntdll/file.c
@@ -2328,7 +2328,10 @@ static struct mountmgr_unix_drive *get_mountmgr_fs_info( HANDLE handle, int fd )
     InitializeObjectAttributes( &attr, &string, 0, NULL, NULL );
     if (NtOpenFile( &mountmgr, GENERIC_READ | SYNCHRONIZE, &attr, &io,
                     FILE_SHARE_READ | FILE_SHARE_WRITE, FILE_SYNCHRONOUS_IO_NONALERT ))
+    {
+        RtlFreeHeap( GetProcessHeap(), 0, drive );
         return NULL;
+    }
 
     status = NtDeviceIoControlFile( mountmgr, NULL, NULL, NULL, &io, IOCTL_MOUNTMGR_QUERY_UNIX_DRIVE,
                                     drive, sizeof(*drive), drive, 1024 );
-- 
2.27.0

[Marvin]

Hi,

While running your changed tests, I think I found new failures.
Being a bot and all I'm not very good at pattern recognition, so I might be
wrong, but could you please double-check?

Full results can be found at:
https://testbot.winehq.org/JobDetails.pl?Key=72927

Your paranoid android.


=== debiant (32 bit WoW report) ===

ntdll:
om.c:2118: Test failed: got 87