diff options
author | seanbright <seanbright@f38db490-d61c-443f-a65b-d21fe96a405b> | 2008-12-01 23:53:18 +0000 |
---|---|---|
committer | seanbright <seanbright@f38db490-d61c-443f-a65b-d21fe96a405b> | 2008-12-01 23:53:18 +0000 |
commit | b6affabc9ef8c64b3f5a542507f2df2f71e9625a (patch) | |
tree | dbd7c7f9600c3707ce9d8335fbe060365469e498 | |
parent | 05f03ae252450919784d7188e1871417c0616a0a (diff) |
Merged revisions 160170-160172 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk
................
r160170 | seanbright | 2008-12-01 18:08:24 -0500 (Mon, 01 Dec 2008) | 1 line
Pay attention to the return value of system(), even if we basically ignore it.
................
r160171 | seanbright | 2008-12-01 18:18:48 -0500 (Mon, 01 Dec 2008) | 1 line
Silence a build warning. (chan_phone.c:810: warning: value computed is not used)
................
r160172 | seanbright | 2008-12-01 18:37:49 -0500 (Mon, 01 Dec 2008) | 10 lines
Merged revisions 159976 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4
........
r159976 | mvanbaak | 2008-12-01 11:08:36 -0500 (Mon, 01 Dec 2008) | 3 lines
Get rid of the useless format string and argument in the Bogus/ manager channelname.
Noted by kpfleming and name Bogus/manager suggested by eliel
........
................
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.6.1@160175 f38db490-d61c-443f-a65b-d21fe96a405b
-rw-r--r-- | channels/chan_phone.c | 2 | ||||
-rw-r--r-- | main/manager.c | 2 | ||||
-rw-r--r-- | utils/smsq.c | 4 |
3 files changed, 5 insertions, 3 deletions
diff --git a/channels/chan_phone.c b/channels/chan_phone.c index 5ac54694d..1cba39d0b 100644 --- a/channels/chan_phone.c +++ b/channels/chan_phone.c @@ -807,7 +807,7 @@ static int phone_write(struct ast_channel *ast, struct ast_frame *frame) we have to pad it to 24 bytes still. */ if (frame->datalen == 4) { if (p->silencesupression) { - memset(tmpbuf + 4, 0, sizeof(tmpbuf) - 4); + (void) memset(tmpbuf + 4, 0, sizeof(tmpbuf) - 4); memcpy(tmpbuf, frame->data.ptr, 4); expected = 24; res = phone_write_buf(p, tmpbuf, expected, maxfr, 0); diff --git a/main/manager.c b/main/manager.c index 39e652054..b4c91e050 100644 --- a/main/manager.c +++ b/main/manager.c @@ -1813,7 +1813,7 @@ static int action_getvar(struct mansession *s, const struct message *m) if (varname[strlen(varname) - 1] == ')') { if (!c) { - c = ast_channel_alloc(0, 0, "", "", "", "", "", 0, "Bogus/%p", SENTINEL); + c = ast_channel_alloc(0, 0, "", "", "", "", "", 0, "Bogus/manager"); if (c) { ast_func_read(c, (char *) varname, workspace, sizeof(workspace)); ast_channel_free(c); diff --git a/utils/smsq.c b/utils/smsq.c index d24845a54..fa71d7639 100644 --- a/utils/smsq.c +++ b/utils/smsq.c @@ -399,7 +399,9 @@ static void rxqcheck (char *dir, char *queue, char *process) setenv ("ud16", tmp, 1); } /* run the command */ - system (process); + if (system (process) == -1) { + fprintf(stderr, "Failed to fork process '%s'\n", process); + } } closedir (d); } |