[asterisk-bugs] [Asterisk 0012942]: [patch] Dialog state NOTIFY for hold uses badly formed XML

[noreply at bugs.digium.com]

The following issue has been SUBMITTED. 
====================================================================== 
http://bugs.digium.com/view.php?id=12942 
====================================================================== 
Reported By:                gowen72
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   12942
Category:                   Channels/chan_sip/Interoperability
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     new
Asterisk Version:           SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  1.4  
SVN Revision (number only!): 126116 
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             06-27-2008 19:09 CDT
Last Modified:              06-27-2008 19:09 CDT
====================================================================== 
Summary:                    [patch] Dialog state NOTIFY for hold uses badly
formed XML
Description: 
For DIALOG_INFO_XML subscriptions, asterisk generates malformed XML for the
hold state NOTIFY events.

The problem is that the XML tag added to indicate the hold state (the
sip.rendering attribute) is not being terminated.  Since there is no value
for this tag, it should be self-terminated, in the same way you are
supposed to use "<hr/>" for a horizontal ruler in HTML.

Here is what is being sent:

  <param pname="+sip.rendering" pvalue="no">

But it shoud be:

  <param pname="+sip.rendering" pvalue="no"/>


Note the "/" character prior the closing ">".


====================================================================== 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
06-27-08 19:09  gowen72        SVN Revision (number only!) => 126116          
======================================================================