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

mlm_selftest.exe -t mlm_server crashes on exit in Windows #169

Open
opedroso opened this issue Apr 26, 2016 · 3 comments
Open

mlm_selftest.exe -t mlm_server crashes on exit in Windows #169

opedroso opened this issue Apr 26, 2016 · 3 comments

Comments

@opedroso
Copy link
Contributor

Got latest zeromq, built all in Windows using Dev2013.

When Malamute selftest runs, it crashes on exit, even thought actual test passed.

X:\3rdparty\malamute> builds\msvc\vs2013\DebugDEXE\mlm_selftest.exe -t mlm_server
Running malamute test 'mlm_server'...
 * mlm_server: I: 16-04-26 14:45:31 client address='(null)' - invalid command
OK
Assertion failed: Successful WSASTARTUP not yet performed (..\..\..\..\src\signaler.cpp:261)

It is interesting that it works for several of the executables built, but fails for these:

X:\3rdparty\malamute\builds\msvc\vs2013\DebugDEXE\mlm_selftest.exe
X:\3rdparty\malamute\builds\msvc\vs2013\ReleaseDEXE\mlm_selftest.exe
X:\3rdparty\malamute\builds\msvc\vs2013\x64\DebugDEXE\mlm_selftest.exe
X:\3rdparty\malamute\builds\msvc\vs2013\x64\ReleaseDEXE\mlm_selftest.exe

Commonality is these are all using LIBZMQ component as DLL.

@opedroso
Copy link
Contributor Author

Closing this issue, since this is a LIBZMQ problem:
zeromq/libzmq#1708

@opedroso
Copy link
Contributor Author

I did this and the problem went away.

x:\3rdparty\malamute>git diff --patch src/mlm_selftest.c
diff --git a/src/mlm_selftest.c b/src/mlm_selftest.c
index 381216f..22c00ed 100644
--- a/src/mlm_selftest.c
+++ b/src/mlm_selftest.c
@@ -141,7 +141,8 @@ main (int argc, char **argv)
     else
         test_runall (verbose);

-    return 0;
+       fcloseall();
+       return 0;
 }
 /*
 ################################################################################

I wonder if I should change the test or wait for the LIBZMQ issue to be fixed.

@opedroso opedroso reopened this Apr 27, 2016
@diorcety
Copy link

Put zsys_shutdown() at the end of the test solve the issue.

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