BloodHoundAD / SharpHound2

The Old BloodHound C# Ingestor (Deprecated)
509 stars 113 forks source link

Add missing NetApiBufferFree #16

Closed paalbra closed 5 years ago

paalbra commented 6 years ago

Hi! Very nice project and application :)

I was looking through the code and found some apparently missing calls to NetApiBufferFree after calls to NetWkstaGetInfo and NetWkstaUserEnum.

I'm not really a trained .Net/C# developer, but I believe it's fine to add the NetApiBufferFree before return marshalled.computer_name; in Utils.cs?

rvazarkar commented 5 years ago

This looks correct. Merging it