Skip to content
New issue

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

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

Already on GitHub? # to your account

Connect when the server ready #1963

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lygstate
Copy link

==3059== Memcheck, a memory error detector
==3059== Copyright (C) 2002-2015, and GNU GPL'd, by Julian Seward et al.
==3059== Using Valgrind-3.11.0 and LibVEX; rerun with -h for copyright info
==3059== Command: /root/work_space/iotjs/build/x86_64-linux/debug/bin/iotjs /root/work_space/iotjs/test/run_pass/test_net_8.js
==3059==
AssertionError: {"actual":"1","expected":"12","operator":"=="}
AssertionError: {"actual":1,"expected":0,"operator":"=="}
==3059==
==3059== HEAP SUMMARY:
==3059== in use at exit: 0 bytes in 0 blocks
==3059== total heap usage: 131 allocs, 131 frees, 314,345 bytes allocated
==3059==

According to this, the server on close event triggered multiple times by close function

wait all four connection to end

==3059== Memcheck, a memory error detector
==3059== Copyright (C) 2002-2015, and GNU GPL'd, by Julian Seward et al.
==3059== Using Valgrind-3.11.0 and LibVEX; rerun with -h for copyright info
==3059== Command: /root/work_space/iotjs/build/x86_64-linux/debug/bin/iotjs /root/work_space/iotjs/test/run_pass/test_net_8.js
==3059==
AssertionError: {"actual":"1","expected":"12","operator":"=="}
AssertionError: {"actual":1,"expected":0,"operator":"=="}
==3059==
==3059== HEAP SUMMARY:
==3059==     in use at exit: 0 bytes in 0 blocks
==3059==   total heap usage: 131 allocs, 131 frees, 314,345 bytes allocated
==3059==

According to this, the server on close event triggered multiple times by close function

wait all four connection to end

IoT.js-DCO-1.0-Signed-off-by: Yonggang Luo luoyonggang@gmail.com
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant