{"action":"opened","issue":{"url":"https://api.github.com/repos/sysrepo/sysrepo/issues/581","repository_url":"https://api.github.com/repos/sysrepo/sysrepo","labels_url":"https://api.github.com/repos/sysrepo/sysrepo/issues/581/labels{/name}","comments_url":"https://api.github.com/repos/sysrepo/sysrepo/issues/581/comments","events_url":"https://api.github.com/repos/sysrepo/sysrepo/issues/581/events","html_url":"https://github.com/sysrepo/sysrepo/issues/581","id":192024954,"number":581,"title":"C++: Memory leak introduced in 51966f5d1d9fe7710d80f0d68dfd73b9d8d503e2","user":{"login":"jktjkt","id":2631925,"avatar_url":"https://avatars.githubusercontent.com/u/2631925?v=3","gravatar_id":"","url":"https://api.github.com/users/jktjkt","html_url":"https://github.com/jktjkt","followers_url":"https://api.github.com/users/jktjkt/followers","following_url":"https://api.github.com/users/jktjkt/following{/other_user}","gists_url":"https://api.github.com/users/jktjkt/gists{/gist_id}","starred_url":"https://api.github.com/users/jktjkt/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jktjkt/subscriptions","organizations_url":"https://api.github.com/users/jktjkt/orgs","repos_url":"https://api.github.com/users/jktjkt/repos","events_url":"https://api.github.com/users/jktjkt/events{/privacy}","received_events_url":"https://api.github.com/users/jktjkt/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2016-11-28T14:50:36Z","updated_at":"2016-11-28T14:50:36Z","closed_at":null,"body":"Since 51966f5d1d9fe7710d80f0d68dfd73b9d8d503e2, ASAN/LSAN once again reports a leak in ``Session::get_item``. My sysrepo is built with ``USE_SR_MEM_MGMT:BOOL=OFF``.\r\n```\r\nDirect leak of 32 byte(s) in 1 object(s) allocated from:\r\n    #0 0x50e135 in calloc /var/tmp/portage/sys-devel/llvm-3.9.0/work/llvm-3.9.0.src/projects/compiler-rt/lib/asan/asan_malloc_linux.cc:72\r\n    #1 0x7ff7fd62c3f6 in sr_calloc /home/jkt/work/prog/sysrepo/build/../src/common/sr_mem_mgmt.c:289:16\r\n    #2 0x7ff7fd6159ab in sr_dup_gpb_to_val_t /home/jkt/work/prog/sysrepo/build/../src/common/sr_protobuf.c:1530:11\r\n    #3 0x7ff7fd82002f in sr_get_item /home/jkt/work/prog/sysrepo/build/../src/clientlib/client_library.c:830:10\r\n    #4 0x7ff7fdbef01b in Session::get_item(char const*) /home/jkt/work/prog/sysrepo/build/../swig/cpp/src/Session.cpp:165:15\r\n    #5 0x557886 in ____C_A_T_C_H____T_E_S_T____17() /home/jkt/work/prog/cla-sysrepo/build/../tests/sysrepo-dummy-amp.cpp:130:37\r\n```\r\nI already fixed this earlier in commit f15dc9dc27a23ba91a4295a2b00e2da499993017 which is apparently reverted by this change.\r\n\r\n@mislavn , is my analysis in the commit message of f15dc9dc27a23ba91a4295a2b00e2da499993017 correct, or am I making a mistake somewhere? Could you please elaborate on the reasons for your recent change?"}}
{"actor":{"display_login":"jktjkt"}}