[asterisk-commits] russell: branch 1.4 r43700 - in /branches/1.4: ./ apps/app_voicemail.c

[asterisk-commits at lists.digium.com]

Author: russell
Date: Tue Sep 26 15:24:39 2006
New Revision: 43700

URL: http://svn.digium.com/view/asterisk?rev=43700&view=rev
Log:
Merged revisions 43699 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.2

........
r43699 | russell | 2006-09-26 16:23:15 -0400 (Tue, 26 Sep 2006) | 6 lines

When parsing the sections of voicemail.conf that contain mailbox definitions,
don't introduce a length limit on the definition by using a 256 byte temporary
storage buffer.  Instead, make the temporary buffer just as big as it needs
to be to hold the entire mailbox definition.
(fixes BE-68)

........

Modified:
    branches/1.4/   (props changed)
    branches/1.4/apps/app_voicemail.c

Propchange: branches/1.4/
------------------------------------------------------------------------------
Binary property 'branch-1.2-merged' - no diff available.

Modified: branches/1.4/apps/app_voicemail.c
URL: http://svn.digium.com/view/asterisk/branches/1.4/apps/app_voicemail.c?rev=43700&r1=43699&r2=43700&view=diff
==============================================================================
--- branches/1.4/apps/app_voicemail.c (original)
+++ branches/1.4/apps/app_voicemail.c Tue Sep 26 15:24:39 2006
@@ -6533,12 +6533,13 @@
 static int append_mailbox(char *context, char *mbox, char *data)
 {
 	/* Assumes lock is already held */
-	char tmp[256] = "";
+	char *tmp;
 	char *stringp;
 	char *s;
 	struct ast_vm_user *vmu;
 
-	ast_copy_string(tmp, data, sizeof(tmp));
+	tmp = ast_strdupa(data);
+
 	if ((vmu = find_or_create(context, mbox))) {
 		populate_defaults(vmu);