[asterisk-bugs] [Asterisk 0016945]: [patch] fix getting callerid name in imap_retrieve_file() (broken callerid number announcement/reply/...)

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=16945 
====================================================================== 
Reported By:                mneuhauser
Assigned To:                tilghman
====================================================================== 
Project:                    Asterisk
Issue ID:                   16945
Category:                   Applications/app_voicemail/IMAP
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     closed
Asterisk Version:           SVN 
JIRA:                       SWP-985 
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases):  1.4  
SVN Revision (number only!): 249845 
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2010-03-02 12:24 CST
Last Modified:              2010-06-23 13:41 CDT
====================================================================== 
Summary:                    [patch] fix getting callerid name in
imap_retrieve_file() (broken callerid number announcement/reply/...)
Description: 
The function imap_retrieve_file() uses get_header_by_tag() to retrieve
various headers from the mail in the IMAP storage and creates a temporary
file from these values. The get_header_by_tag() function uses a global
buffer to return its result but for the callerid, imap_retrieve_file()
calls this function twice, retrieving the number AND the name and only then
creates combined callerid, but at this point, both the name and the number
point into the global buffer which holds the last result, i.e., both are
the same string (the name, since this was retrieved last).

The fix just saves the number with ast_strdupa() before calling
get_header_by_tag() to retrieve the name.
====================================================================== 

---------------------------------------------------------------------- 
 (0123780) svnbot (reporter) - 2010-06-23 13:41
 https://issues.asterisk.org/view.php?id=16945#c123780 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 272148

_U  trunk/

------------------------------------------------------------------------
r272148 | tilghman | 2010-06-23 13:41:17 -0500 (Wed, 23 Jun 2010) | 12
lines

Recorded merge of revisions 272147 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r272147 | tilghman | 2010-06-23 13:40:28 -0500 (Wed, 23 Jun 2010) | 5
lines
  
  Backport part of revision 136715 to fix callerid in voicemail text files
(IMAP only).
  
  (closes issue https://issues.asterisk.org/view.php?id=16945)
   Reported by: mneuhauser
........

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

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

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-06-23 13:41 svnbot         Checkin                                      
2010-06-23 13:41 svnbot         Note Added: 0123780                          
======================================================================