Re: [Sofia-sip-devel] Memory Leak from nua_destroy_handle().

2006-11-29 Thread Pekka Pessi 
On 11/30/06, Colin Whittaker <[EMAIL PROTECTED]> wrote:
> I running 1.12.3work4 and I am seeing an occasional memory leak during
> bulk calling tests.
> I assume the path to reproduce this requires some kind of error, but i
> have not isolated that yet.
> The backtrace to all the leaked memory comes from
> nua_session_usage_shutdown().
> I think this happens when the application calls nua_handle_destroy() for
> a handle that is in a dialog.

OK. The

> All the rest of the items allocated are below nua_creq_msg() or
> nta_outgoing_mcreate() in the call to nua_session_usage_shutdown().
> I am guessing the leak is caused by:
> nua_session.c line 1175:
>   if (cr->cr_orq)/* Wait for BYE to complete? */
> nua_creq_deinit(cr, NULL);
>
> So we don't deinit here. But, the handle does get freed.

I think there is an error there and the request message should be
freed in every case (even if the BYE is not sent). I fixed that for
1.12.4.

-- 
Pekka.Pessi mail at nokia.com

-
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
___
Sofia-sip-devel mailing list
Sofia-sip-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sofia-sip-devel

[Sofia-sip-devel] Memory Leak from nua_destroy_handle().

2006-11-29 Thread Colin Whittaker 
I running 1.12.3work4 and I am seeing an occasional memory leak during 
bulk calling tests.
I assume the path to reproduce this requires some kind of error, but i 
have not isolated that yet.
The backtrace to all the leaked memory comes from 
nua_session_usage_shutdown().
I think this happens when the application calls nua_handle_destroy() for 
a handle that is in a dialog.

Here is the backtrace for the first item allocated in this path:
su_home_new()+0x5c
msg_create()+0x2c
nta_msg_create()+0x44
nua_creq_msg()+0x1f8
nua_session_usage_shutdown()+352
nua_dialog_usage_refresh()+0x64
nh_call_pending()+128
nua_stack_destroy_handle()+0x28
nua_stack_signal()+0x40c
su_port_getmsgs()+0xb0
su_port_query()+180
su_root_step()+0x28

All the rest of the items allocated are below nua_creq_msg() or 
nta_outgoing_mcreate() in the call to nua_session_usage_shutdown().
I am guessing the leak is caused by:
nua_session.c line 1175:
  if (cr->cr_orq)/* Wait for BYE to complete? */
nua_creq_deinit(cr, NULL);

So we don't deinit here. But, the handle does get freed.

I'll try to get a SIP trace that causes this, but it may not be easy.

Any thoughts ?

Think we should fix this for 1.12.4 ?

Colin..



-
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
___
Sofia-sip-devel mailing list
Sofia-sip-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sofia-sip-devel

[Sofia-sip-devel] sofia-sip changes (2006-11-29)

2006-11-29 Thread Sofia-SIP Darcs Changes 
This posting was generated automatically from darcs repo
.

Wed Nov 29 12:03:12 EET 2006  [EMAIL PROTECTED]
  * Doxyfile.aliases: fixed @VERSION_1_12_4

M ./libsofia-sip-ua/docs/Doxyfile.aliases -1 +1

Wed Nov 29 12:01:30 EET 2006  [EMAIL PROTECTED]
  * nua_tag.h: NUTAG_APPL_METHOD is not yet in API.

M ./libsofia-sip-ua/nua/sofia-sip/nua_tag.h -37

Wed Nov 29 10:34:06 EET 2006  [EMAIL PROTECTED]
  * libsofia-sip-ua/Makefile.am: silenced some spurious warnings from doxygen 

M ./libsofia-sip-ua/Makefile.am -3 +4

Wed Nov 29 10:33:25 EET 2006  [EMAIL PROTECTED]
  * sres_blocking.c, su_osx_runloop.c: fixed doxygen docs

M ./libsofia-sip-ua/sresolv/sres_blocking.c -3 +3
M ./libsofia-sip-ua/su/su_osx_runloop.c -6 +7

Wed Nov 29 09:41:22 EET 2006  [EMAIL PROTECTED]
  * RELEASE: updated.

M ./RELEASE -5 +5

Wed Nov 29 01:44:17 EET 2006  [EMAIL PROTECTED]
  * Added #include  to test programs.

M ./libsofia-sip-ua/http/test_http.c -5 +6
M ./libsofia-sip-ua/ipt/torture_base64.c +1
M ./libsofia-sip-ua/sip/torture_sip.c -3 +5
M ./libsofia-sip-ua/su/torture_rbtree.c +1
M ./libsofia-sip-ua/su/torture_su_alloc.c +1
M ./libsofia-sip-ua/su/torture_su_bm.c +2
M ./libsofia-sip-ua/su/torture_su_port.c -1 +3
M ./libsofia-sip-ua/su/torture_su_root.c +1
M ./libsofia-sip-ua/su/torture_su_root_osx.c +1
M ./libsofia-sip-ua/su/torture_su_time.c +1

Wed Nov 29 01:39:05 EET 2006  [EMAIL PROTECTED]
  * RELEASE: updated.

M ./RELEASE -5 +21

Wed Nov 29 01:29:28 EET 2006  [EMAIL PROTECTED]
  * test_refer.c: not using assert(), found timing problem with Mr. C.

M ./libsofia-sip-ua/nua/test_refer.c -1 +1

Wed Nov 29 01:20:01 EET 2006  [EMAIL PROTECTED]
  * test_register.c: increase minimum refresh interval for REGISTER to 600 sec.

M ./libsofia-sip-ua/nua/test_register.c -1 +1

Wed Nov 29 00:02:34 EET 2006  [EMAIL PROTECTED]
  * tstdef.h: added tst_abort

M ./libsofia-sip-ua/nua/test_nua.c +2
M ./libsofia-sip-ua/su/sofia-sip/tstdef.h -13 +23

Tue Nov 28 23:41:56 EET 2006  [EMAIL PROTECTED]
  * nua: not using aor-specific contact if that aor is not specified.

M ./libsofia-sip-ua/nua/nua_register.c -2 +5
M ./libsofia-sip-ua/nua/test_basic_call.c -24 +32

Tue Nov 28 23:25:26 EET 2006  [EMAIL PROTECTED]
  * nua: fixed problem of using user-supplied Contact with nua_register()
  
  The user-supplied Contact was not stored with the registration.

M ./libsofia-sip-ua/nua/nua_dialog.h +2
M ./libsofia-sip-ua/nua/nua_register.c -10 +13
M ./libsofia-sip-ua/nua/nua_stack.c +3
M ./libsofia-sip-ua/nua/test_basic_call.c -16 +16
M ./libsofia-sip-ua/nua/test_register.c -4 +12

Tue Nov 28 22:35:48 EET 2006  [EMAIL PROTECTED]
  * torture_url.c: added tests for url_query_as_header_string().

M ./libsofia-sip-ua/url/torture_url.c -3 +9

Tue Nov 28 22:10:26 EET 2006  [EMAIL PROTECTED]
  * msg: added msg_as_string(). Updated msg_make() prototype.

M ./libsofia-sip-ua/msg/msg_parser.c -41 +125
M ./libsofia-sip-ua/msg/msg_parser_util.c -5 +16
M ./libsofia-sip-ua/msg/sofia-sip/msg.h -1 +5
M ./libsofia-sip-ua/msg/sofia-sip/msg_mclass.h -2 +6
M ./libsofia-sip-ua/msg/test_msg.c -1 +33

Tue Nov 28 21:56:04 EET 2006  [EMAIL PROTECTED]
  * auth_module.c: added auth_mod_name().

M ./libsofia-sip-ua/iptsec/auth_module.c +5
M ./libsofia-sip-ua/iptsec/sofia-sip/auth_module.h +2

Tue Nov 28 21:52:13 EET 2006  [EMAIL PROTECTED]
  * su, http, nth, sip, tport: removed some warnings.

M ./libsofia-sip-ua/http/test_http.c -1 +1
M ./libsofia-sip-ua/iptsec/auth_client.c -2 +2
M ./libsofia-sip-ua/nth/nth_server.c -1 +1
M ./libsofia-sip-ua/nth/sofia-sip/nth.h -1 +1
M ./libsofia-sip-ua/sip/validator.c -7 +7
M ./libsofia-sip-ua/su/sofia-sip/su_alloc.h -1 +1
M ./libsofia-sip-ua/su/sofia-sip/su_tag.h -4 +4
M ./libsofia-sip-ua/su/su_wait.c -3 +4
M ./libsofia-sip-ua/tport/test_tport.c -1 +1

Tue Nov 28 21:17:27 EET 2006  [EMAIL PROTECTED]
  * tport.c: tport_send_msg() returns -1 or 0.

M ./libsofia-sip-ua/tport/tport.c -4 +5

-
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
___
Sofia-sip-devel mailing list
Sofia-sip-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sofia-sip-devel