Fixed bug in Convert-ADName that output an array including null values instead of a string

HarmJ0y · HarmJ0y · commit 6c113b7956f8 · 2017-01-11T14:03:46.000-05:00
diff --git a/Recon/PowerView.ps1 b/Recon/PowerView.ps1
@@ -1563,18 +1563,18 @@ https://gallery.technet.microsoft.com/scriptcenter/Translating-Active-5c80dd67
                     )
                 }
                 catch {
-                    Write-Verbose "[Convert-ADName] Error initialiting translation for '$Identity' using alternate credentials : $_"
+                    Write-Verbose "[Convert-ADName] Error initializing translation for '$Identity' using alternate credentials : $_"
                 }
             }
             else {
                 try {
-                    Invoke-Method $Translate 'Init' (
+                    $Null = Invoke-Method $Translate 'Init' (
                         $ADSInitType,
                         $InitName
                     )
                 }
                 catch {
-                    Write-Verbose "[Convert-ADName] Error initialiting translation for '$Identity' : $_"
+                    Write-Verbose "[Convert-ADName] Error initializing translation for '$Identity' : $_"
                 }
             }
 
@@ -1583,7 +1583,7 @@ https://gallery.technet.microsoft.com/scriptcenter/Translating-Active-5c80dd67
 
             try {
                 # 8 = Unknown name type -> let the server do the work for us
-                Invoke-Method $Translate 'Set' (8, $TargetIdentity)
+                $Null = Invoke-Method $Translate 'Set' (8, $TargetIdentity)
                 Invoke-Method $Translate 'Get' ($ADSOutputType)
             }
             catch [System.Management.Automation.MethodInvocationException] {

Original file line number	Diff line number	Diff line change
`@@ -1563,18 +1563,18 @@ https://gallery.technet.microsoft.com/scriptcenter/Translating-Active-5c80dd67`
`1563`	`1563`	`)`
`1564`	`1564`	`}`
`1565`	`1565`	`catch {`
`1566`		`- Write-Verbose "[Convert-ADName] Error initialiting translation for '$Identity' using alternate credentials : $_"`
	`1566`	`+ Write-Verbose "[Convert-ADName] Error initializing translation for '$Identity' using alternate credentials : $_"`
`1567`	`1567`	`}`
`1568`	`1568`	`}`
`1569`	`1569`	`else {`
`1570`	`1570`	`try {`
`1571`		`- Invoke-Method $Translate 'Init' (`
	`1571`	`+ $Null = Invoke-Method $Translate 'Init' (`
`1572`	`1572`	`$ADSInitType,`
`1573`	`1573`	`$InitName`
`1574`	`1574`	`)`
`1575`	`1575`	`}`
`1576`	`1576`	`catch {`
`1577`		`- Write-Verbose "[Convert-ADName] Error initialiting translation for '$Identity' : $_"`
	`1577`	`+ Write-Verbose "[Convert-ADName] Error initializing translation for '$Identity' : $_"`
`1578`	`1578`	`}`
`1579`	`1579`	`}`
`1580`	`1580`
`@@ -1583,7 +1583,7 @@ https://gallery.technet.microsoft.com/scriptcenter/Translating-Active-5c80dd67`
`1583`	`1583`
`1584`	`1584`	`try {`
`1585`	`1585`	`# 8 = Unknown name type -> let the server do the work for us`
`1586`		`- Invoke-Method $Translate 'Set' (8, $TargetIdentity)`
	`1586`	`+ $Null = Invoke-Method $Translate 'Set' (8, $TargetIdentity)`
`1587`	`1587`	`Invoke-Method $Translate 'Get' ($ADSOutputType)`
`1588`	`1588`	`}`
`1589`	`1589`	`catch [System.Management.Automation.MethodInvocationException] {`