Merge pull request #42 from mbrancato/master

Add additional check for empty SID parameter
2016-08-26 12:31:06 -04:00 · 2016-08-26 12:31:06 -04:00 · 47bf7e4926
parent 12556c0161 565aa58b51
commit 47bf7e4926
1 changed files with 3 additions and 1 deletions
--- a/PowerShell/BloodHound.ps1
+++ b/PowerShell/BloodHound.ps1
@ -14555,7 +14555,9 @@ function Export-BloodHoundCSV {
            }
        }
        elseif($Object.PSObject.TypeNames -contains 'PowerView.GPOLocalGroup') {
-            $MemberSimpleName = Convert-SidToName -SID $Object.ObjectSID | Convert-ADName -InputType 'NT4' -OutputType 'Canonical'
+            if(![string]::IsNullOrEmpty($Object.SID)){
+                $MemberSimpleName = Convert-SidToName -SID $Object.ObjectSID | Convert-ADName -InputType 'NT4' -OutputType 'Canonical'
+            }

            if($MemberSimpleName) {
                $MemberDomain = $MemberSimpleName.Split('/')[0]