You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@kvrocks.apache.org by GitBox <gi...@apache.org> on 2022/07/01 02:25:40 UTC
[GitHub] [incubator-kvrocks] git-hulk opened a new issue, #702: Memory leak reports in tcl test cases
git-hulk opened a new issue, #702:
URL: https://github.com/apache/incubator-kvrocks/issues/702
### Search before asking
- [X] I had searched in the [issues](https://github.com/apache/incubator-kvrocks/issues) and found no similar issues.
### Version
unstable
### Minimal reproduce step
···
$ ./build.sh build -DENABLE_ASAN=ON
$ cd tests/tcl
$ ./runtest --dont-clean
···
### What did you expect to see?
No memory leak reported
### What did you see instead?
https://github.com/apache/incubator-kvrocks/runs/7133747333?check_suite_focus=true
```
=================================================================
==30270==ERROR: LeakSanitizer: detected memory leaks
Direct leak of 1023 byte(s) in 1 object(s) allocated from:
#0 0x7f0b2c2e5b40 in __interceptor_malloc (/usr/lib/x86_64-linux-gnu/libasan.so.4+0xdeb40)
#1 0x55ef1f258c72 in strbuf_init (/home/runner/work/incubator-kvrocks/incubator-kvrocks/tests/tcl/redis-server+0xa1ec72)
```
### Anything Else?
I noticed only the `cjson` library is using `strbuf_init`.
### Are you willing to submit a PR?
- [ ] I'm willing to submit a PR!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@kvrocks.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-kvrocks] git-hulk closed issue #702: Memory leak reports in tcl test cases
Posted by GitBox <gi...@apache.org>.
git-hulk closed issue #702: Memory leak reports in tcl test cases
URL: https://github.com/apache/incubator-kvrocks/issues/702
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@kvrocks.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-kvrocks] tisonkun commented on issue #702: Memory leak reports in tcl test cases
Posted by GitBox <gi...@apache.org>.
tisonkun commented on issue #702:
URL: https://github.com/apache/incubator-kvrocks/issues/702#issuecomment-1171861664
@git-hulk is it an issue on unstable branch or #697 only? We shouldn't create issue for further problem. It seems we may fix this issue at KvrocksLabs/luajit aspect?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@kvrocks.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-kvrocks] git-hulk commented on issue #702: Memory leak reports in tcl test cases
Posted by GitBox <gi...@apache.org>.
git-hulk commented on issue #702:
URL: https://github.com/apache/incubator-kvrocks/issues/702#issuecomment-1171867155
> @git-hulk is it an issue on unstable branch or #697 only? We shouldn't create issue for further problem. It seems we may fix this issue at KvrocksLabs/luajit aspect?
>
> In another word, I don't see what we can do before #697 merged and on unstable branch.
My bad. I think the Lua library is using `cjson`, so it should have the same problem as well. Will close this issue since it looks introduced by #697 and have an eye on this case.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@kvrocks.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org