[Home]

Summary:ASTERISK-02180: [patch] Make the ADSI feature download number and security code configurable
Reporter:sremington (sremington)Labels:
Date Opened:2004-08-02 22:19:51Date Closed:2008-01-15 15:07:35.000-0600
Priority:MajorRegression?No
Status:Closed/CompleteComponents:Applications/app_voicemail
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) 20040801_voicemail.patch
Description:This patch allows the ADSI FDN, security code, and version to be set in the voicemail.conf config file. This allows the user to change the FDN slot to download to without requiring a recompile. The version number is configurable too in case the admin wants to increment it to force CPE's to upgrade.

The sscanf / memcpy code is admittedly horrendous but it is the first solution I tried that worked. I'm sure there is a simpler, more elegant solution that somebody more clever than me can come up with.
Comments:By: Mark Spencer (markster) 2004-08-03 02:56:52

Do you already have a disclaimer on file?

By: sremington (sremington) 2004-08-03 06:59:54

Yes. I mailed one in a couple of weeks ago.

By: twisted (twisted) 2004-08-24 13:30:33

Does this still apply cleanly?  If so, markster, where do we stand on this one?

-Housekeeping

By: sremington (sremington) 2004-08-24 14:45:58

Just tested the patch on a clean CVS HEAD and it applied fine.

By: twisted (twisted) 2004-08-25 18:09:21

markster?

By: Mark Spencer (markster) 2004-09-15 16:33:16

Thanks, updated and merged into CVS.

By: Digium Subversion (svnbot) 2008-01-15 15:07:35.000-0600

Repository: asterisk
Revision: 3788

U   trunk/apps/app_voicemail.c
U   trunk/configs/voicemail.conf.sample

------------------------------------------------------------------------
r3788 | markster | 2008-01-15 15:07:34 -0600 (Tue, 15 Jan 2008) | 2 lines

Make ADSI in voicemail configurable (bug ASTERISK-2180)

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

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