[asterisk-commits] trunk - r7556 in /trunk: cli.c
include/asterisk/strings.h res/res_agi.c utils.c
asterisk-commits at lists.digium.com
asterisk-commits at lists.digium.com
Tue Dec 20 14:20:06 CST 2005
Author: russell
Date: Tue Dec 20 14:20:04 2005
New Revision: 7556
URL: http://svn.digium.com/view/asterisk?rev=7556&view=rev
Log:
- move the string join() function to utils.c since it is used in both cli.c and res_agi.c
- reimplement ast_join to be of linear effieciency instead of quadratic
- remove some useless checks for "if (e)"
- reorder checks for strings starting with '_' to avoid a useless call to ast_join()
- check array bounds when parsing arguments to AGI
(issue #5868)
Modified:
trunk/cli.c
trunk/include/asterisk/strings.h
trunk/res/res_agi.c
trunk/utils.c
Modified: trunk/cli.c
URL: http://svn.digium.com/view/asterisk/trunk/cli.c?rev=7556&r1=7555&r2=7556&view=diff
==============================================================================
--- trunk/cli.c (original)
+++ trunk/cli.c Tue Dec 20 14:20:04 2005
@@ -1019,17 +1019,8 @@
static void join(char *dest, size_t destsize, char *w[], int tws)
{
- int x;
- /* Join words into a string */
- if (!dest || destsize < 1) {
- return;
- }
- dest[0] = '\0';
- for (x=0;w[x];x++) {
- if (x)
- strncat(dest, " ", destsize - strlen(dest) - 1);
- strncat(dest, w[x], destsize - strlen(dest) - 1);
- }
+ ast_join(dest, destsize, w);
+
if (tws && !ast_strlen_zero(dest))
strncat(dest, " ", destsize - strlen(dest) - 1);
}
Modified: trunk/include/asterisk/strings.h
URL: http://svn.digium.com/view/asterisk/trunk/include/asterisk/strings.h?rev=7556&r1=7555&r2=7556&view=diff
==============================================================================
--- trunk/include/asterisk/strings.h (original)
+++ trunk/include/asterisk/strings.h Tue Dec 20 14:20:04 2005
@@ -165,6 +165,7 @@
}
)
+
/*!
\brief Build a string in a buffer, designed to be called repeatedly
@@ -209,6 +210,18 @@
* Returns 0 if val is a NULL pointer, -1 if "false", and 0 otherwise.
*/
int ast_false(const char *val);
+
+/*
+ \brief Join an array of strings into a single string.
+ \param s the resulting string buffer
+ \param len the length of the result buffer, s
+ \param w an array of strings to join
+
+ This function will join all of the strings in the array 'w' into a single
+ string. It will also place a space in the result buffer in between each
+ string from 'w'.
+*/
+void ast_join(char *s, size_t len, char * const w[]);
/* The realloca lets us ast_restrdupa(), but you can't mix any other ast_strdup calls! */
Modified: trunk/res/res_agi.c
URL: http://svn.digium.com/view/asterisk/trunk/res/res_agi.c?rev=7556&r1=7555&r2=7556&view=diff
==============================================================================
--- trunk/res/res_agi.c (original)
+++ trunk/res/res_agi.c Tue Dec 20 14:20:04 2005
@@ -1627,22 +1627,6 @@
{ { "wait", "for", "digit", NULL }, handle_waitfordigit, "Waits for a digit to be pressed", usage_waitfordigit },
};
-static void join(char *s, size_t len, char *w[])
-{
- int x;
-
- /* Join words into a string */
- if (!s) {
- return;
- }
- s[0] = '\0';
- for (x=0; w[x]; x++) {
- if (x)
- strncat(s, " ", len - strlen(s) - 1);
- strncat(s, w[x], len - strlen(s) - 1);
- }
-}
-
static int help_workhorse(int fd, char *match[])
{
char fullcmd[80];
@@ -1650,20 +1634,17 @@
int x;
struct agi_command *e;
if (match)
- join(matchstr, sizeof(matchstr), match);
+ ast_join(matchstr, sizeof(matchstr), match);
for (x=0;x<sizeof(commands)/sizeof(commands[0]);x++) {
- if (!commands[x].cmda[0]) break;
e = &commands[x];
- if (e)
- join(fullcmd, sizeof(fullcmd), e->cmda);
+ if (!e->cmda[0])
+ break;
/* Hide commands that start with '_' */
- if (fullcmd[0] == '_')
+ if ((e->cmda[0])[0] == '_')
continue;
- if (match) {
- if (strncasecmp(matchstr, fullcmd, strlen(matchstr))) {
- continue;
- }
- }
+ ast_join(fullcmd, sizeof(fullcmd), e->cmda);
+ if (match && strncasecmp(matchstr, fullcmd, strlen(matchstr)))
+ continue;
ast_cli(fd, "%20.20s %s\n", fullcmd, e->summary);
}
return 0;
@@ -1801,10 +1782,9 @@
static int agi_handle_command(struct ast_channel *chan, AGI *agi, char *buf)
{
char *argv[MAX_ARGS];
- int argc = 0;
+ int argc = MAX_ARGS;
int res;
agi_command *c;
- argc = MAX_ARGS;
parse_args(buf, &argc, argv);
#if 0
@@ -1929,7 +1909,7 @@
if (find_command(argv + 2, -1)) {
return help_workhorse(fd, argv + 1);
} else {
- join(fullcmd, sizeof(fullcmd), argv+1);
+ ast_join(fullcmd, sizeof(fullcmd), argv+1);
ast_cli(fd, "No such command '%s'.\n", fullcmd);
}
}
@@ -1942,7 +1922,6 @@
static int handle_dumpagihtml(int fd, int argc, char *argv[]) {
struct agi_command *e;
char fullcmd[80];
- char *tempstr;
int x;
FILE *htmlfile;
@@ -1961,18 +1940,18 @@
fprintf(htmlfile, "<TABLE BORDER=\"0\" CELLSPACING=\"10\">\n");
for (x=0;x<sizeof(commands)/sizeof(commands[0]);x++) {
- char *stringp=NULL;
- if (!commands[x].cmda[0]) break;
+ char *stringp, *tempstr;
+
e = &commands[x];
- if (e)
- join(fullcmd, sizeof(fullcmd), e->cmda);
+ if (!e->cmda[0]) /* end ? */
+ break;
/* Hide commands that start with '_' */
- if (fullcmd[0] == '_')
+ if ((e->cmda[0])[0] == '_')
continue;
+ ast_join(fullcmd, sizeof(fullcmd), e->cmda);
fprintf(htmlfile, "<TR><TD><TABLE BORDER=\"1\" CELLPADDING=\"5\" WIDTH=\"100%%\">\n");
fprintf(htmlfile, "<TR><TH ALIGN=\"CENTER\"><B>%s - %s</B></TD></TR>\n", fullcmd,e->summary);
-
stringp=e->usage;
tempstr = strsep(&stringp, "\n");
@@ -1980,10 +1959,8 @@
fprintf(htmlfile, "<TR><TD ALIGN=\"CENTER\">%s</TD></TR>\n", tempstr);
fprintf(htmlfile, "<TR><TD ALIGN=\"CENTER\">\n");
- while ((tempstr = strsep(&stringp, "\n")) != NULL) {
- fprintf(htmlfile, "%s<BR>\n",tempstr);
-
- }
+ while ((tempstr = strsep(&stringp, "\n")) != NULL)
+ fprintf(htmlfile, "%s<BR>\n",tempstr);
fprintf(htmlfile, "</TD></TR>\n");
fprintf(htmlfile, "</TABLE></TD></TR>\n\n");
@@ -2016,9 +1993,8 @@
ast_copy_string(buf, data, sizeof(buf));
memset(&agi, 0, sizeof(agi));
- while ((stringp = strsep(&tmp, "|"))) {
+ while ((stringp = strsep(&tmp, "|")) && argc < MAX_ARGS-1)
argv[argc++] = stringp;
- }
argv[argc] = NULL;
LOCAL_USER_ADD(u);
Modified: trunk/utils.c
URL: http://svn.digium.com/view/asterisk/trunk/utils.c?rev=7556&r1=7555&r2=7556&view=diff
==============================================================================
--- trunk/utils.c (original)
+++ trunk/utils.c Tue Dec 20 14:20:04 2005
@@ -903,3 +903,22 @@
*dataPut = 0;
return dataPut;
}
+
+void ast_join(char *s, size_t len, char * const w[])
+{
+ int x, ofs = 0;
+ const char *src;
+
+ /* Join words into a string */
+ if (!s)
+ return;
+ for (x=0; ofs < len && w[x]; x++) {
+ if (x > 0)
+ s[ofs++] = ' ';
+ for (src = w[x]; *src && ofs < len; src++)
+ s[ofs++] = *src;
+ }
+ if (ofs == len)
+ ofs--;
+ s[ofs] = '\0';
+}
More information about the asterisk-commits
mailing list