[Home]

Summary:ASTERISK-20982: xmpp Segfault when delete node using cli
Reporter:Marcello Ceschia (marcelloceschia)Labels:
Date Opened:2013-01-24 07:02:00.000-0600Date Closed:2013-02-11 09:03:58.000-0600
Priority:MajorRegression?
Status:Closed/CompleteComponents:Resources/res_xmpp
Versions:11.2.1 Frequency of
Occurrence
Constant
Related
Issues:
Environment:i686 running LinuxAttachments:( 0) delete-node-fix.diff
( 1) desc_backtrace.txt
( 2) mmlog
( 3) res_xmpp-backtrace.txt
( 4) res_xmpp-backtrace-full.txt
( 5) xmpp-delete-node.txt
Description:execute in asterisk cli:

xmpp delete node asterisk03 Custom:licht



backtrace:

[Edit by Rusty Newton - no debug in the description please - attaching to issue.]
Comments:By: Rusty Newton (rnewton) 2013-01-27 17:21:15.168-0600

Can this be reproduced every time that command is run?


https://wiki.asterisk.org/wiki/display/AST/Asterisk+Issue+Guidelines

https://wiki.asterisk.org/wiki/display/AST/Getting+a+Backtrace

Please *attach* both a bt and 'bt full' backtrace. Verify you have followed the compilation options required in the wiki page above. Please don't paste any debug into the Description field.

Thanks!

By: Rusty Newton (rnewton) 2013-01-31 14:40:35.628-0600

attaching desc_backtrace.txt from the Description.

By: Rusty Newton (rnewton) 2013-02-07 16:55:35.490-0600

Can you compile with the MALLOC_DEBUG compiler option and include the /var/log/asterisk/mmlog file after the crash is reproduced?



By: Marcello Ceschia (marcelloceschia) 2013-02-08 11:12:28.573-0600

mmlog do not contain any interesting part from xmpp

Comparing xmpp_pubsub_delete_node with xmpp_pubsub_create_node, which is nearly the same except create/delete and node content, shows that it is not necessary to delete pubsub, delete separately.

By: Matt Jordan (mjordan) 2013-02-11 08:53:55.459-0600

Yup, this patch is correct. {{iks_insert}} adds the created node as a child to the node passed in; {{iks_delete}} destroys the node and all of its children. Calling {{iks_delete}} on the other child nodes is invalid.

Thanks for the patch!