From b7d3e6d7d23d74a3b99e3a55a8d982d382f3930f Mon Sep 17 00:00:00 2001 From: Corinna Vinschen Date: Tue, 8 Jun 2010 15:25:09 +0000 Subject: * path.cc (symlink_info::check): Set h to NULL after closing handle in udf check. Remove overagressive bracketing. Don't set h to NULL without closing the handle. Fix using wrong handle in checking file system when using parent directory handle. --- winsup/cygwin/path.cc | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'winsup/cygwin/path.cc') diff --git a/winsup/cygwin/path.cc b/winsup/cygwin/path.cc index 8ebd7cce9..4173401e3 100644 --- a/winsup/cygwin/path.cc +++ b/winsup/cygwin/path.cc @@ -2308,6 +2308,7 @@ restart: if (!fs.is_udf ()) { NtClose (h); + h = NULL; status = STATUS_OBJECT_NAME_NOT_FOUND; } } @@ -2355,7 +2356,7 @@ restart: if (NT_SUCCESS (status) /* Check file system while we're having the file open anyway. This speeds up path_conv noticably (~10%). */ - && (fs.inited () || (fs.update (&upath, h))) + && (fs.inited () || fs.update (&upath, h)) && NT_SUCCESS (status = fs.has_buggy_basic_info () ? NtQueryAttributesFile (&attr, &fbi) : NtQueryInformationFile (h, &io, &fbi, sizeof fbi, @@ -2364,7 +2365,6 @@ restart: else { debug_printf ("%p = NtQueryInformationFile (%S)", status, &upath); - h = NULL; fileattr = INVALID_FILE_ATTRIBUTES; /* One of the inner path components is invalid, or the path contains @@ -2427,7 +2427,7 @@ restart: TRUE, &basename, TRUE); /* Take the opportunity to check file system while we're having the handle to the parent dir. */ - fs.update (&upath, h); + fs.update (&upath, dir); NtClose (dir); if (!NT_SUCCESS (status)) { -- cgit v1.2.3