1. fix some exception when appverifier is enabled on win7 (PowerShell/Win32-OpenSSH#872)

2. enable sshdconfig tests on win7(PowerShell/Win32-OpenSSH#873)
3. fix for PowerShell/Win32-OpenSSH#874 does not return on win7 when no content in console )
4. remove loggin to console in Readthread because write hangs here since write thread already closed (PowerShell/Win32-OpenSSH#879)
5. fix VCTargetsPath

Author: bingbing8

contrib/win32/openssh/OpenSSHBuildHelper.psm1

@@ -217,7 +217,7 @@ function Start-OpenSSHBootstrap
         [Environment]::SetEnvironmentVariable('Path', $newMachineEnvironmentPath, 'MACHINE')
     }
 
-    $VCTargetsPath = "${env:ProgramFiles(x86)}\MSBuild\Microsoft.Cpp\v4.0\V140"
+    $VCTargetsPath = "${env:ProgramFiles(x86)}\MSBuild\Microsoft.Cpp\v4.0\V140\"
     if([Environment]::GetEnvironmentVariable('VCTargetsPath', 'MACHINE') -eq $null)
     {
         [Environment]::SetEnvironmentVariable('VCTargetsPath', $VCTargetsPath, 'MACHINE')

contrib/win32/win32compat/termio.c

@@ -103,12 +103,11 @@ ReadThread(_In_ LPVOID lpParameter)
 
 				if (!SetConsoleMode(GetStdHandle(STD_INPUT_HANDLE), dwAttributes))
 					error("SetConsoleMode on STD_INPUT_HANDLE failed with %d\n", GetLastError());
-			}
+			}			
 
 			if (!ReadFile(WINHANDLE(pio), pio->read_details.buf,
-				pio->read_details.buf_size, &read_status.transferred, NULL)) {
-				read_status.error = GetLastError();
-				debug("ReadThread - ReadFile failed %d, io:%p", GetLastError(), pio);
+				pio->read_details.buf_size, &read_status.transferred, NULL)) { 
+				read_status.error = GetLastError();				
 				return -1;
 			}
 
@@ -127,8 +126,7 @@ ReadThread(_In_ LPVOID lpParameter)
 	} else {
 		if (!ReadFile(WINHANDLE(pio), pio->read_details.buf,
 		    pio->read_details.buf_size, &read_status.transferred, NULL)) {
-			read_status.error = GetLastError();
-			debug("ReadThread - ReadFile failed %d, io:%p", GetLastError(), pio);
+			read_status.error = GetLastError();			
 			return -1;
 		}
 	}
@@ -257,7 +255,7 @@ syncio_close(struct w32_io* pio)
 	/* If io is pending, let worker threads exit. */
 	if (pio->read_details.pending) {
 		/* For console - the read thread is blocked so terminate it. */
-		if (FILETYPE(pio) == FILE_TYPE_CHAR && in_raw_mode)
+		if (FILETYPE(pio) == FILE_TYPE_CHAR)
 			TerminateThread(pio->read_overlapped.hEvent, 0);
 		else
 			WaitForSingleObject(pio->read_overlapped.hEvent, INFINITE);

contrib/win32/win32compat/w32fd.c

@@ -556,17 +556,18 @@ w32_io_process_fd_flags(struct w32_io* pio, int flags)
 
 	shi_flags = (flags & FD_CLOEXEC) ? 0 : HANDLE_FLAG_INHERIT;
 
-	if (SetHandleInformation(WINHANDLE(pio), HANDLE_FLAG_INHERIT, shi_flags) == FALSE) {
-		/* 
-		 * Ignore if handle is not valid yet. It will not be valid for 
-		 * UF_UNIX sockets that are not connected yet 
-		 */
-		if (GetLastError() != ERROR_INVALID_HANDLE) {
-			debug3("fcntl - SetHandleInformation failed  %d, io:%p",
-				GetLastError(), pio);
-			errno = EOTHER;
-			return -1;
-		}
+	HANDLE h = WINHANDLE(pio);
+	if (IS_INVALID_HANDLE(h)) {
+		/*
+		* Ignore if handle is not valid yet. It will not be valid for
+		* UF_UNIX sockets that are not connected yet
+		*/
+	}
+	else if (SetHandleInformation(h, HANDLE_FLAG_INHERIT, shi_flags) == FALSE) {
+		debug3("fcntl - SetHandleInformation failed  %d, io:%p",
+			GetLastError(), pio);
+		errno = EOTHER;
+		return -1;
 	}
 
 	pio->fd_flags = flags;

regress/pesterTests/SSHDConfig.tests.ps1

@@ -25,7 +25,7 @@ Describe "Tests of sshd_config" -Tags "CI" {
         Add-Type -AssemblyName System.DirectoryServices.AccountManagement
         $ContextName = $env:COMPUTERNAME
         $ContextType = [System.DirectoryServices.AccountManagement.ContextType]::Machine
-        $PrincipalContext = [System.DirectoryServices.AccountManagement.PrincipalContext]::new($ContextType, $ContextName)
+        $PrincipalContext = new-object -TypeName System.DirectoryServices.AccountManagement.PrincipalContext -ArgumentList @($ContextType, $ContextName)
         $IdentityType = [System.DirectoryServices.AccountManagement.IdentityType]::SamAccountName
 
         function Add-LocalUser
@@ -35,7 +35,7 @@ Describe "Tests of sshd_config" -Tags "CI" {
             if($user -eq $null)
             {
                 try {
-                    $user = [System.DirectoryServices.AccountManagement.UserPrincipal]::new($PrincipalContext,$UserName,$Password, $true)
+                    $user = new-object -TypeName System.DirectoryServices.AccountManagement.UserPrincipal -ArgumentList @($PrincipalContext,$UserName,$Password, $true)
                     $user.Save()
                 }
                 finally {
@@ -51,7 +51,7 @@ Describe "Tests of sshd_config" -Tags "CI" {
             if($group -eq $null)
             {
                 try {
-                    $group = [System.DirectoryServices.AccountManagement.GroupPrincipal]::new($PrincipalContext,$groupName)
+                    $group = new-object -TypeName System.DirectoryServices.AccountManagement.GroupPrincipal -ArgumentList @($PrincipalContext,$groupName)
                     $group.Save()
                 }
                 finally {