Bug fix with ordinal loading in remote processes

clymb3r / PowerShell

Useful PowerShell scripts

1.22k stars 529 forks source link

Bug fix with ordinal loading in remote processes #4

Closed sixdub closed 9 years ago

sixdub commented 9 years ago

When performing remote loading of a DLL that had imports using ordinals, the code would crash prompting that NumBytesWritten was undefined. NumBytesWritten is properly defined in the logical case that a name is used but not an ordinal (line 1327). Added a definition of NumBytesWritten directly above the place where the error was occurring which then fixed the bug. screen shot 2014-12-28 at 4 04 45 pm

clymb3r commented 9 years ago

This is a good catch, thanks! Before I accept the pull request, could you please increment the version number contained in the scripts header comments from version 1.2 to 1.3?

sixdub commented 9 years ago

Incremented the version number. Thanks for the quick response and for the pull