[asterisk-bugs] [Asterisk 0014876]: [patch] Missing \r\n in response to JabberSend manager action

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=14876 
====================================================================== 
Reported By:                srt
Assigned To:                seanbright
====================================================================== 
Project:                    Asterisk
Issue ID:                   14876
Category:                   Resources/res_jabber
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     closed
Asterisk Version:           SVN 
Regression:                 No 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 188018 
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2009-04-10 18:03 CDT
Last Modified:              2009-05-30 14:40 CDT
====================================================================== 
Summary:                    [patch] Missing \r\n in response to JabberSend
manager action
Description: 
The response to a JabberSend action is not properly terminated by "\r\n".
====================================================================== 

---------------------------------------------------------------------- 
 (0105796) svnbot (reporter) - 2009-05-30 14:40
 https://issues.asterisk.org/view.php?id=14876#c105796 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 198373

_U  branches/1.6.1/
U   branches/1.6.1/res/res_jabber.c

------------------------------------------------------------------------
r198373 | seanbright | 2009-05-30 14:40:59 -0500 (Sat, 30 May 2009) | 26
lines

Merged revisions 198371 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/trunk

................
  r198371 | seanbright | 2009-05-30 15:38:58 -0400 (Sat, 30 May 2009) | 19
lines
  
  Merged revisions 198370 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r198370 | seanbright | 2009-05-30 15:36:20 -0400 (Sat, 30 May 2009) |
12 lines
    
    Properly terminate AMI JabberSend response messages.
    
    The response message (either Error or Success) needs an extra trailing
\r\n
    after the fields to inform the client that the message is complete.
    
    (closes issue https://issues.asterisk.org/view.php?id=14876)
    Reported by: srt
    Patches:
          05302009_1.4_res_jabber.c.diff uploaded by seanbright (license
71)
          asterisk_14876.patch uploaded by srt (license 378)
          trunk-14876-2.diff uploaded by phsultan (license 73)
  ........
................

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=198373 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-05-30 14:40 svnbot         Checkin                                      
2009-05-30 14:40 svnbot         Note Added: 0105796                          
======================================================================