Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Engine version 1.1.14202.0 fails calling DllMain() #36

Closed
earada opened this issue Sep 29, 2017 · 3 comments
Closed

Engine version 1.1.14202.0 fails calling DllMain() #36

earada opened this issue Sep 29, 2017 · 3 comments

Comments

@earada
Copy link
Contributor

earada commented Sep 29, 2017

After updating the engine to version 1.1.14202.0, mpclient isn't able to load DllMain.

root@093c610a27cc:/loadlibrary$ gdb -q ./mpclient
Reading symbols from ./mpclient...done.
(gdb) r /eicar.txt
Starting program: /loadlibrary/mpclient /eicar.txt
MultiByteToWideChar(): Unsupported Conversion Flags 0x1
MultiByteToWideChar(): Unsupported Conversion Flags 0x1
MultiByteToWideChar(): Unsupported Conversion Flags 0x1
MultiByteToWideChar(): Unsupported Conversion Flags 0x1
MultiByteToWideChar(): Unsupported Conversion Flags 0x1
MultiByteToWideChar(): Unsupported Conversion Flags 0x1

Program received signal SIGSEGV, Segmentation fault.
0xf6d35fbe in ?? ()
(gdb) bt
#0  0xf6d35fbe in ?? ()
#1  0xf6d360f3 in ?? ()
#2  0xf6d34e05 in ?? ()
#3  0xf6d32bf0 in ?? ()
#4  0xf6d32bc5 in ?? ()
#5  0xf6d32dff in ?? ()
#6  0xf6d32f06 in ?? ()
#7  0x56557013 in main (argc=<optimized out>, argv=<optimized out>, envp=<optimized out>) at mpclient.c:193
@taviso
Copy link
Owner

taviso commented Sep 29, 2017

Thanks for the excellent bug report, I think there are two bugs:

  • I hadn't labeled some of the locale functions with the correct calling convention (and neither did MSDN...grrr).
  • The latest build uses the function GetProcessTimes, so I needed to add that.

I think I made MultiByteToWideChar handle that conversion flag as well, although it would still have worked okay with some minor string mangling.

@taviso taviso closed this as completed in 00597b5 Sep 29, 2017
@taviso
Copy link
Owner

taviso commented Sep 29, 2017

Please try git pull and it should work with the new engine. Please let me know about any other issues you notice!

@earada
Copy link
Contributor Author

earada commented Sep 29, 2017

Thank you for fixing that quick. Pulled and tested, it works perfectly again!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants