[asterisk-commits] tilghman: trunk r131044 - in /trunk: ./ apps/ channels/ main/
SVN commits to the Asterisk project
asterisk-commits at lists.digium.com
Tue Jul 15 13:25:34 CDT 2008
Author: tilghman
Date: Tue Jul 15 13:25:34 2008
New Revision: 131044
URL: http://svn.digium.com/view/asterisk?view=rev&rev=131044
Log:
Merged revisions 130959 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4
........
r130959 | tilghman | 2008-07-15 12:19:13 -0500 (Tue, 15 Jul 2008) | 8 lines
astman_send_error does not need a newline appended -- the API takes care of
that for us.
(closes issue #13068)
Reported by: gknispel_proformatique
Patches:
asterisk_1_4_astman_send.patch uploaded by gknispel (license 261)
asterisk_trunk_astman_send.patch uploaded by gknispel (license 261)
........
Modified:
trunk/ (props changed)
trunk/apps/app_voicemail.c
trunk/channels/chan_sip.c
trunk/main/manager.c
trunk/main/pbx.c
Propchange: trunk/
------------------------------------------------------------------------------
Binary property 'branch-1.4-merged' - no diff available.
Modified: trunk/apps/app_voicemail.c
URL: http://svn.digium.com/view/asterisk/trunk/apps/app_voicemail.c?view=diff&rev=131044&r1=131043&r2=131044
==============================================================================
--- trunk/apps/app_voicemail.c (original)
+++ trunk/apps/app_voicemail.c Tue Jul 15 13:25:34 2008
@@ -9338,7 +9338,7 @@
return RESULT_SUCCESS;
}
- astman_send_ack(s, m, "Voicemail user list will follow\r\n");
+ astman_send_ack(s, m, "Voicemail user list will follow");
AST_LIST_TRAVERSE(&users, vmu, list) {
char dirname[256];
Modified: trunk/channels/chan_sip.c
URL: http://svn.digium.com/view/asterisk/trunk/channels/chan_sip.c?view=diff&rev=131044&r1=131043&r2=131044
==============================================================================
--- trunk/channels/chan_sip.c (original)
+++ trunk/channels/chan_sip.c Tue Jul 15 13:25:34 2008
@@ -12905,7 +12905,7 @@
peer = astman_get_header(m, "Peer");
if (ast_strlen_zero(peer)) {
- astman_send_error(s, m, "Peer: <name> missing.\n");
+ astman_send_error(s, m, "Peer: <name> missing.");
return 0;
}
a[0] = "sip";
@@ -12951,7 +12951,7 @@
} else if (type == 0) {
ast_cli(fd, "Peer '%s' not found\n", argv[3]);
} else {
- astman_send_error(s, m, "Peer not found\n");
+ astman_send_error(s, m, "Peer not found");
}
return CLI_SUCCESS;
}
@@ -12964,7 +12964,7 @@
peer = astman_get_header(m, "Peer");
if (ast_strlen_zero(peer)) {
- astman_send_error(s, m, "Peer: <name> missing.\n");
+ astman_send_error(s, m, "Peer: <name> missing.");
return 0;
}
a[0] = "sip";
@@ -13036,7 +13036,7 @@
if (!ast_strlen_zero(id))
astman_append(s, "ActionID: %s\r\n", id);
} else {
- snprintf (cbuf, sizeof(cbuf), "Peer %s not found.\n", argv[3]);
+ snprintf (cbuf, sizeof(cbuf), "Peer %s not found.", argv[3]);
astman_send_error(s, m, cbuf);
return CLI_SUCCESS;
}
Modified: trunk/main/manager.c
URL: http://svn.digium.com/view/asterisk/trunk/main/manager.c?view=diff&rev=131044&r1=131043&r2=131044
==============================================================================
--- trunk/main/manager.c (original)
+++ trunk/main/manager.c Tue Jul 15 13:25:34 2008
@@ -1971,7 +1971,7 @@
}
if (!ast_strlen_zero(priority) && (sscanf(priority, "%d", &pi) != 1)) {
if ((pi = ast_findlabel_extension(NULL, context, exten, priority, NULL)) < 1) {
- astman_send_error(s, m, "Invalid priority\n");
+ astman_send_error(s, m, "Invalid priority");
return 0;
}
}
@@ -1984,14 +1984,14 @@
return 0;
}
if (ast_check_hangup(chan)) {
- astman_send_error(s, m, "Redirect failed, channel not up.\n");
+ astman_send_error(s, m, "Redirect failed, channel not up.");
ast_channel_unlock(chan);
return 0;
}
if (!ast_strlen_zero(name2))
chan2 = ast_get_channel_by_name_locked(name2);
if (chan2 && ast_check_hangup(chan2)) {
- astman_send_error(s, m, "Redirect failed, extra channel not up.\n");
+ astman_send_error(s, m, "Redirect failed, extra channel not up.");
ast_channel_unlock(chan);
ast_channel_unlock(chan2);
return 0;
@@ -2039,34 +2039,34 @@
int priority_int = 0;
if (ast_strlen_zero(name)) {
- astman_send_error(s, m, "No channel specified\n");
+ astman_send_error(s, m, "No channel specified");
return 0;
}
if (ast_strlen_zero(exten)) {
- astman_send_error(s, m, "No extension specified\n");
+ astman_send_error(s, m, "No extension specified");
return 0;
}
if (ast_strlen_zero(context)) {
- astman_send_error(s, m, "No context specified\n");
+ astman_send_error(s, m, "No context specified");
return 0;
}
if (ast_strlen_zero(priority)) {
- astman_send_error(s, m, "No priority specified\n");
+ astman_send_error(s, m, "No priority specified");
return 0;
}
if (sscanf(priority, "%d", &priority_int) != 1 && (priority_int = ast_findlabel_extension(NULL, context, exten, priority, NULL)) < 1) {
- astman_send_error(s, m, "Invalid Priority\n");
+ astman_send_error(s, m, "Invalid Priority");
return 0;
}
if (!(atxfer_feature = ast_find_call_feature("atxfer"))) {
- astman_send_error(s, m, "No attended transfer feature found\n");
+ astman_send_error(s, m, "No attended transfer feature found");
return 0;
}
if (!(chan = ast_get_channel_by_name_locked(name))) {
- astman_send_error(s, m, "Channel specified does not exist\n");
+ astman_send_error(s, m, "Channel specified does not exist");
return 0;
}
@@ -2080,7 +2080,7 @@
ast_queue_frame(chan, &f);
}
- astman_send_ack(s, m, "Atxfer successfully queued\n");
+ astman_send_ack(s, m, "Atxfer successfully queued");
ast_channel_unlock(chan);
return 0;
@@ -2288,19 +2288,19 @@
}
if (!ast_strlen_zero(priority) && (sscanf(priority, "%d", &pi) != 1)) {
if ((pi = ast_findlabel_extension(NULL, context, exten, priority, NULL)) < 1) {
- astman_send_error(s, m, "Invalid priority\n");
+ astman_send_error(s, m, "Invalid priority");
return 0;
}
}
if (!ast_strlen_zero(timeout) && (sscanf(timeout, "%d", &to) != 1)) {
- astman_send_error(s, m, "Invalid timeout\n");
+ astman_send_error(s, m, "Invalid timeout");
return 0;
}
ast_copy_string(tmp, name, sizeof(tmp));
tech = tmp;
data = strchr(tmp, '/');
if (!data) {
- astman_send_error(s, m, "Invalid channel\n");
+ astman_send_error(s, m, "Invalid channel");
return 0;
}
*data++ = '\0';
Modified: trunk/main/pbx.c
URL: http://svn.digium.com/view/asterisk/trunk/main/pbx.c?view=diff&rev=131044&r1=131043&r2=131044
==============================================================================
--- trunk/main/pbx.c (original)
+++ trunk/main/pbx.c Tue Jul 15 13:25:34 2008
@@ -5192,7 +5192,7 @@
/* try to lock contexts */
if (ast_rdlock_contexts()) {
- astman_send_error(s, m, "Failed to lock contexts\r\n");
+ astman_send_error(s, m, "Failed to lock contexts");
ast_log(LOG_WARNING, "Failed to lock contexts list for manager: listdialplan\n");
return -1;
}
@@ -5337,7 +5337,7 @@
if (context && !counters.context_existence) {
char errorbuf[BUFSIZ];
- snprintf(errorbuf, sizeof(errorbuf), "Did not find context %s\r\n", context);
+ snprintf(errorbuf, sizeof(errorbuf), "Did not find context %s", context);
astman_send_error(s, m, errorbuf);
return 0;
}
@@ -5345,9 +5345,9 @@
char errorbuf[BUFSIZ];
if (context)
- snprintf(errorbuf, sizeof(errorbuf), "Did not find extension %s@%s\r\n", exten, context);
+ snprintf(errorbuf, sizeof(errorbuf), "Did not find extension %s@%s", exten, context);
else
- snprintf(errorbuf, sizeof(errorbuf), "Did not find extension %s in any context\r\n", exten);
+ snprintf(errorbuf, sizeof(errorbuf), "Did not find extension %s in any context", exten);
astman_send_error(s, m, errorbuf);
return 0;
}
More information about the asterisk-commits
mailing list