Use a regex with a negative lookbehind to cope with CNs that contain …

…commas
rapid7 · Dec 21, 2015 · e8c8c54 · e8c8c54
1 parent b0fca76
commit e8c8c54
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/modules/post/windows/gather/enum_ad_managedby_groups.rb b/modules/post/windows/gather/enum_ad_managedby_groups.rb
@@ -94,7 +94,7 @@ def parse_results(results)
       end
       if datastore['RESOLVE_MANAGERS']
         begin
-          managedby_cn = result[2][:value].split(',')[0]
+          managedby_cn = result[2][:value].split(/,(?<!\\,)/)[0]
           m = query("(&(objectClass=user)(objectCategory=person)(#{managedby_cn}))", 1, ['sAMAccountName'])
           if !m.nil? && !m[:results].empty?
             row << m[:results][0][0][:value]