[asterisk-commits] tilghman: trunk r167894 - in /trunk: ./ res/res_agi.c

[SVN commits to the Asterisk project]

Author: tilghman
Date: Thu Jan  8 16:37:20 2009
New Revision: 167894

URL: http://svn.digium.com/view/asterisk?view=rev&rev=167894
Log:
Merged revisions 167840 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r167840 | tilghman | 2009-01-08 16:08:56 -0600 (Thu, 08 Jan 2009) | 6 lines
  
  Don't truncate database results at 255 chars.
  (closes issue #14069)
   Reported by: evandro
   Patches: 
         20081214__bug14069.diff.txt uploaded by Corydon76 (license 14)
........

Modified:
    trunk/   (props changed)
    trunk/res/res_agi.c

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

Modified: trunk/res/res_agi.c
URL: http://svn.digium.com/view/asterisk/trunk/res/res_agi.c?view=diff&rev=167894&r1=167893&r2=167894
==============================================================================
--- trunk/res/res_agi.c (original)
+++ trunk/res/res_agi.c Thu Jan  8 16:37:20 2009
@@ -1849,16 +1849,32 @@
 static int handle_dbget(struct ast_channel *chan, AGI *agi, int argc, char **argv)
 {
 	int res;
-	char tmp[256];
+	struct ast_str *buf;
 
 	if (argc != 4)
 		return RESULT_SHOWUSAGE;
-	res = ast_db_get(argv[2], argv[3], tmp, sizeof(tmp));
+
+	if (!(buf = ast_str_create(16))) {
+		ast_agi_send(agi->fd, chan, "200 result=-1\n");
+		return RESULT_SUCCESS;
+	}
+
+	do {
+		res = ast_db_get(argv[2], argv[3], ast_str_buffer(buf), ast_str_size(buf));
+		if (ast_str_strlen(buf) < ast_str_size(buf) - 1) {
+			break;
+		}
+		if (ast_str_make_space(&buf, ast_str_size(buf) * 2)) {
+			break;
+		}
+	} while (1);
+	
 	if (res)
 		ast_agi_send(agi->fd, chan, "200 result=0\n");
 	else
-		ast_agi_send(agi->fd, chan, "200 result=1 (%s)\n", tmp);
-
+		ast_agi_send(agi->fd, chan, "200 result=1 (%s)\n", ast_str_buffer(buf));
+
+	ast_free(buf);
 	return RESULT_SUCCESS;
 }