[Home]

Summary:ASTERISK-01535: [patch] Change strlen() to !ast_strlen_zero()
Reporter:Olle Johansson (oej)Labels:
Date Opened:2004-05-04 11:15:21Date Closed:2008-01-15 14:53:39.000-0600
Priority:MajorRegression?No
Status:Closed/CompleteComponents:Applications/app_voicemail
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) appsstrlen.txt
( 1) voicemail.txt
Description:This patch fixes two thing
* Change strlen to ast_strlen_zero
* Moves a gethostname into the right place to avoid unnecessary lookups :-)
Comments:By: Mark Spencer (markster) 2004-05-04 12:27:39

Actually that gethostname has to stay where it is, because it's used later on when creating the Message-ID.  Just fix it up and I"ll be happy to apply it.  Thanks!

By: Olle Johansson (oej) 2004-05-04 12:46:20

So much for trying to be clever :-)

By: Olle Johansson (oej) 2004-05-04 14:43:16

Two new patches uploaded. Fix to voicemail and another that goes into apps/app_adsiprog.c apps/app_agi.c apps/app_dial.c apps/app_authenticate.c

By: Mark Spencer (markster) 2004-05-04 15:05:15

Added to CVS, thanks!

By: Digium Subversion (svnbot) 2008-01-15 14:53:39.000-0600

Repository: asterisk
Revision: 2890

U   trunk/apps/app_adsiprog.c
U   trunk/apps/app_agi.c
U   trunk/apps/app_authenticate.c
U   trunk/apps/app_dial.c
U   trunk/channels/chan_sip.c

------------------------------------------------------------------------
r2890 | markster | 2008-01-15 14:53:39 -0600 (Tue, 15 Jan 2008) | 2 lines

strlen fixes and don't destroy SIP channels that still have pending things on them (bug ASTERISK-1535 et al)

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

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