search for: stats_get_xml

...one (in /lib64/libc-2.12.so) ==13821== ==13821== 830,000 bytes in 830 blocks are definitely lost in loss record 262 of 263 ==13821== at 0x4A0515D: malloc (vg_replace_malloc.c:195) ==13821== by 0x3B08E2D6DF: xmlEncodeEntitiesReentrant (in /usr/lib64/libxml2. so.2.7.7) ==13821== by 0x40FAA9: stats_get_xml (stats.c:970) ==13821== by 0x40FBB0: stats_transform_xslt (stats.c:939) ==13821== by 0x419244: add_authenticated_listener (auth.c:422) ==13821== by 0x41955D: auth_add_listener (auth.c:511) ==13821== by 0x409D45: _handle_connection (connection.c:1150) ==13821== by 0x4200FD: _start_rou...

...is issue, I'd love to > > hear about it! > > xmlEncodeEntitiesReentrant() return values must be freed, so probably > not a libxml problem. > > The Debian Testing Icecast source (2.3.2-4) does seem to have a case > where they are not freed: a do-while loop in stats.c:stats_get_xml() can > leak when breaking early. This code is not present in -kh17 that I'm > using and all xmlEncodeEntitiesReentrant() calls are fine there. I'm co-maintaining the Debian package. I have no time to test it myself right now, but I'd be interested to have your feedback and fi...

Hi, I've been noticing a huge memory usage on Icecast 2.3.2 on multiple debian instances. They are using like 100 MB after a day and going on until they hit like 1 GB.. So I ran one in valgrind, and killed it after ~24h, here is the output: ==30481== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 38 from 3) ==30481== malloc/free: in use at exit: 44,448,584 bytes in 44,470 blocks.

...d love to >>> hear about it! >> xmlEncodeEntitiesReentrant() return values must be freed, so probably >> not a libxml problem. >> >> The Debian Testing Icecast source (2.3.2-4) does seem to have a case >> where they are not freed: a do-while loop in stats.c:stats_get_xml() can >> leak when breaking early. This code is not present in -kh17 that I'm >> using and all xmlEncodeEntitiesReentrant() calls are fine there. > > I'm co-maintaining the Debian package. I have no time to test it myself right > now, but I'd be interested to ha...

...we get an update on the mountpoint, force a @@ -653,7 +653,7 @@ static void command_stats(client_t *client, int response) { xmlDocPtr doc; - DEBUG0("Stats request, sending xml stats"); + DEBUG0("Stats request, sending xml stats, %s", client->con->ip); stats_get_xml(&doc); admin_send_response(doc, client, response, STATS_TRANSFORMED_REQUEST); @@ -665,7 +665,7 @@ static void command_list_mounts(client_t *client, int response) { xmlDocPtr doc; - DEBUG0("List mounts request"); + DEBUG0("List mounts request, %s", client-&...