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

Console seems a bit broken? #44

Open
mr-mop opened this issue Dec 7, 2017 · 0 comments
Open

Console seems a bit broken? #44

mr-mop opened this issue Dec 7, 2017 · 0 comments

Comments

@mr-mop
Copy link

mr-mop commented Dec 7, 2017

The console, http://duda.io/api/console.html https://web.archive.org/web/20190807111954/http://duda.io/api/console.html, enabling messages, interface map and, jemalloc statistics seems to have a problem when not built by dudac with jemalloc enabled and jemalloc stats enabled.

It looks like the line at 215 in https://github.com/monkey/duda/blob/dst-1/src/duda.c
was commented out to avoid a duplicate call further down at line 224?
I assume because duda, via dudac as is encouraged for using the dst-1 stack, it will enable jemalloc
and so expect to run the code at line 224?
This works if Jemalloc and Jealloc stats are enabled, but not if only Jemalloc is enabled.
I added an #else to the #if JEMALLOC block at lines 218-225 to call mk_list_add() and everything started to work as expected.

mr-mop added a commit to mr-mop/duda that referenced this issue Feb 13, 2019
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

1 participant