Merge pull request #42 from mbrancato/master

Add additional check for empty SID parameter
SpecterOps · Aug 26, 2016 · 47bf7e4 · 47bf7e4
2 parents 12556c0 + 565aa58
commit 47bf7e4
Showing 1 changed file with 3 additions and 1 deletion.
diff --git 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]