utils.c - bug in pafish_check_mac_vendor
asafcyi opened this issue · comments
Hi,
I've noticed that in "pafish_check_mac_vendor" function in utils.c there is a potential problem:
palist pointer is advanced (line 167: palist = palist->Next;) and freed using LocalFree (lines: 163, 169).
It caused the program to crash.
I recommend to keep a pointer to the start of the allocated buffer and free it before leaving the function.
Thanks for this great tool!
Hi,
I've just noticed the same exists for "pafish_check_adapter_name" function in utils.c.
BR,
Asaf.
Hello there!
Thanks for your report, good catch! It has been fixed in the devel branch, it will be merged into master for the next release.
Cheers!
Merged into master and released!