[asterisk-commits] branch oej/sipregister - r8462 in
/team/oej/sipregister: ./ apps/ channels/ c...
asterisk-commits at lists.digium.com
asterisk-commits at lists.digium.com
Mon Jan 23 05:13:22 MST 2006
Author: oej
Date: Mon Jan 23 06:12:19 2006
New Revision: 8462
URL: http://svn.digium.com/view/asterisk?rev=8462&view=rev
Log:
Merged revisions 8270,8278,8282,8299-8300,8313,8315,8323-8324,8336,8338,8345-8346,8348,8356-8357,8359-8362,8368,8370-8372,8378-8381,8387,8393,8401-8404,8407-8411,8413,8420-8422,8426,8428,8432,8436,8438,8443,8446-8447 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk
................
r8270 | tilghman | 2006-01-19 19:58:45 +0100 (Thu, 19 Jan 2006) | 2 lines
Deprecate the use of Random in 1.3, 1.4
................
r8278 | tilghman | 2006-01-19 20:18:51 +0100 (Thu, 19 Jan 2006) | 3 lines
Merged revisions 8276 via svnmerge from
/branches/1.2
................
r8282 | oej | 2006-01-19 20:41:06 +0100 (Thu, 19 Jan 2006) | 2 lines
Enable "musicclass" setting for sip peers
................
r8299 | oej | 2006-01-19 23:09:18 +0100 (Thu, 19 Jan 2006) | 2 lines
Doxygen fixes
................
r8300 | oej | 2006-01-19 23:41:55 +0100 (Thu, 19 Jan 2006) | 3 lines
Add support for "musicclass" instead of "musiconhold" to make chan_zap compatible
with other channels.
................
r8313 | oej | 2006-01-20 01:08:36 +0100 (Fri, 20 Jan 2006) | 2 lines
Doxygen updates
................
r8315 | oej | 2006-01-20 01:18:42 +0100 (Fri, 20 Jan 2006) | 2 lines
More doxygen updates
................
r8323 | mogorman | 2006-01-20 02:07:46 +0100 (Fri, 20 Jan 2006) | 11 lines
Merged revisions 8320 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.2
........
r8320 | mogorman | 2006-01-19 19:00:46 -0600 (Thu, 19 Jan 2006) | 3 lines
solved problem with delayreject and iax trunking
bug 4291
........
................
r8324 | russell | 2006-01-20 02:25:12 +0100 (Fri, 20 Jan 2006) | 2 lines
fix version reporting (issue #6292)
................
r8336 | oej | 2006-01-20 15:32:30 +0100 (Fri, 20 Jan 2006) | 4 lines
- Add DOC file about caller ID presentation values
- Add callingpres to sip.conf
- Add reference to README.callingpres from zapata.conf
................
r8338 | kpfleming | 2006-01-20 18:27:51 +0100 (Fri, 20 Jan 2006) | 2 lines
eliminate compiler warnings from recent const-ification
................
r8345 | mattf | 2006-01-20 19:17:46 +0100 (Fri, 20 Jan 2006) | 2 lines
Update config. Apprently default doesn't always work now.
................
r8346 | mattf | 2006-01-20 19:18:40 +0100 (Fri, 20 Jan 2006) | 2 lines
Change chan_alsa to default open the first sound card device
................
r8348 | russell | 2006-01-20 19:40:24 +0100 (Fri, 20 Jan 2006) | 10 lines
Merged revisions 8347 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.2
........
r8347 | russell | 2006-01-20 13:34:42 -0500 (Fri, 20 Jan 2006) | 2 lines
fix invalid value of prev_q (issue #6302)
........
................
r8356 | russell | 2006-01-20 20:24:42 +0100 (Fri, 20 Jan 2006) | 11 lines
- move ast_strdupa from channel.h to utils.h
- attempt to log an error message if the __builtin_alloca inside of ast_strdupa
fails.
- document the fact that it is known and intended behavior for ast_strdupa to
cause Asterisk to crash if the alloca fails
- use __builtin_expect when checking for allocation failure in all of the
allocation wrappers
New Janitor Project! Anywhere that we check for a successful allocation after
a call to ast_strdupa is unnecessary and should be removed.
................
r8357 | mattf | 2006-01-20 22:36:33 +0100 (Fri, 20 Jan 2006) | 2 lines
Fix for formats so they give better output on failure conditions. (#6141)
................
r8359 | mattf | 2006-01-21 00:19:49 +0100 (Sat, 21 Jan 2006) | 2 lines
Fix comments in sip.conf (#6134)
................
r8360 | russell | 2006-01-21 00:23:00 +0100 (Sat, 21 Jan 2006) | 2 lines
formatting and doxygen fixes (issue #6300)
................
r8361 | mattf | 2006-01-21 00:28:37 +0100 (Sat, 21 Jan 2006) | 2 lines
fix for codec_gsm on ia64 (#6294)
................
r8362 | russell | 2006-01-21 01:42:25 +0100 (Sat, 21 Jan 2006) | 2 lines
remove some useless checks after calls to ast_strdupa
................
r8368 | russell | 2006-01-21 04:09:01 +0100 (Sat, 21 Jan 2006) | 2 lines
remove optimization where its benefits are negligible
................
r8370 | tilghman | 2006-01-21 06:05:45 +0100 (Sat, 21 Jan 2006) | 2 lines
Bug 6096 - callerid_parse cleanup
................
r8371 | tilghman | 2006-01-21 06:15:56 +0100 (Sat, 21 Jan 2006) | 2 lines
Bug 5515 - Devicestate and API documentation update
................
r8372 | tilghman | 2006-01-21 06:22:18 +0100 (Sat, 21 Jan 2006) | 2 lines
Bug 4872 - Make Asterisk paths available to AGIs via environmental variables
................
r8378 | russell | 2006-01-21 09:01:24 +0100 (Sat, 21 Jan 2006) | 2 lines
remove some useless checks of the result of ast_strdupa
................
r8379 | russell | 2006-01-21 09:13:12 +0100 (Sat, 21 Jan 2006) | 2 lines
remove lots of useless checks of the result of ast_strdupa
................
r8380 | russell | 2006-01-21 09:23:57 +0100 (Sat, 21 Jan 2006) | 2 lines
remove useless checks of the result of ast_strdupa
................
r8381 | russell | 2006-01-21 09:45:39 +0100 (Sat, 21 Jan 2006) | 2 lines
conversions to use allocation wrappers (issue #6277)
................
r8387 | russell | 2006-01-21 18:50:04 +0100 (Sat, 21 Jan 2006) | 3 lines
revert my pass through the tree to remove checks of the result of ast_strdupa
(revisions 8378 through 8381)
................
r8393 | russell | 2006-01-21 19:11:40 +0100 (Sat, 21 Jan 2006) | 3 lines
don't do the memcpy inside of ast_strdupa if we know that __builtin_alloca
was not successful
................
r8401 | russell | 2006-01-21 21:20:06 +0100 (Sat, 21 Jan 2006) | 3 lines
finish reverting my pass through the tree to remove checks of the result of
ast_strdupa, this one is revision 8362
................
r8402 | tilghman | 2006-01-21 21:32:17 +0100 (Sat, 21 Jan 2006) | 2 lines
Bug 5936 - Cannot AddQueueMember on realtime queue, if queue not yet loaded (different fix than 1.2)
................
r8403 | russell | 2006-01-21 21:57:06 +0100 (Sat, 21 Jan 2006) | 2 lines
on this pass, only remove duplicate log messages
................
r8404 | russell | 2006-01-21 21:58:57 +0100 (Sat, 21 Jan 2006) | 1 line
................
r8407 | russell | 2006-01-21 22:29:06 +0100 (Sat, 21 Jan 2006) | 2 lines
clean up formatting to conform to coding guidelines and fix some typos (issue #6260)
................
r8408 | russell | 2006-01-21 22:50:09 +0100 (Sat, 21 Jan 2006) | 5 lines
- conversion to allocation wrappers
- get rid of some compiler warnings due to usused variables and const-ification
- remove some unnecessary uses of malloc/strncpy/free and replace with ast_strdupa
(based on the patch from issue #6275)
................
r8409 | russell | 2006-01-21 22:57:01 +0100 (Sat, 21 Jan 2006) | 2 lines
oops, don't build app_sql_postgres by default ...
................
r8410 | russell | 2006-01-21 23:09:06 +0100 (Sat, 21 Jan 2006) | 5 lines
- conversions to allocation wrappers
- replace malloc/memset with ast_calloc
- replace malloc/ast_copy_string with ast_strdup
(based on patch from issue #6299)
................
r8411 | russell | 2006-01-22 00:05:19 +0100 (Sun, 22 Jan 2006) | 2 lines
const-ify some fields in the ast_exten and ast_include structures (issue #6270)
................
r8413 | russell | 2006-01-22 00:17:52 +0100 (Sun, 22 Jan 2006) | 1 line
................
r8420 | russell | 2006-01-22 03:06:33 +0100 (Sun, 22 Jan 2006) | 1 line
................
r8421 | russell | 2006-01-22 03:10:19 +0100 (Sun, 22 Jan 2006) | 3 lines
prevent writing outside of the provided workspace when calculating a
substring (issue #6271)
................
r8422 | russell | 2006-01-22 03:23:38 +0100 (Sun, 22 Jan 2006) | 2 lines
add 'show channeltype' CLI command (issue #6149, with small modifications)
................
r8426 | tilghman | 2006-01-22 08:18:02 +0100 (Sun, 22 Jan 2006) | 2 lines
Bug 6148 - Add PARKEDAT variable; also cleaned up application help to fit 80-column screen.
................
r8428 | tilghman | 2006-01-22 09:09:02 +0100 (Sun, 22 Jan 2006) | 2 lines
Bug 6262 - New function STRPTIME
................
r8432 | bweschke | 2006-01-22 16:08:51 +0100 (Sun, 22 Jan 2006) | 2 lines
Correct some scenarios where CALL_LIMIT could not be getting adjusted properly allowing chan_sip to send calls when it really shouldn't. Bug #6111
................
r8436 | tilghman | 2006-01-22 18:28:42 +0100 (Sun, 22 Jan 2006) | 2 lines
Bug 6312 - Macro janitor
................
r8438 | russell | 2006-01-22 18:53:14 +0100 (Sun, 22 Jan 2006) | 10 lines
Merged revisions 8437 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.2
........
r8437 | russell | 2006-01-22 12:47:13 -0500 (Sun, 22 Jan 2006) | 2 lines
fix MixMonitor crash (issue #6321, probably others)
........
................
r8443 | russell | 2006-01-22 19:42:06 +0100 (Sun, 22 Jan 2006) | 2 lines
eliminate some compiler warnings
................
r8446 | russell | 2006-01-22 20:04:37 +0100 (Sun, 22 Jan 2006) | 1 line
................
r8447 | russell | 2006-01-22 20:09:50 +0100 (Sun, 22 Jan 2006) | 2 lines
fix memory leak from not freeing the list of queue members when freeing a queue
................
Added:
team/oej/sipregister/doc/README.callingpres
- copied unchanged from r8447, trunk/doc/README.callingpres
Modified:
team/oej/sipregister/ (props changed)
team/oej/sipregister/apps/Makefile
team/oej/sipregister/apps/app_authenticate.c
team/oej/sipregister/apps/app_chanspy.c
team/oej/sipregister/apps/app_curl.c
team/oej/sipregister/apps/app_db.c
team/oej/sipregister/apps/app_dial.c
team/oej/sipregister/apps/app_dictate.c
team/oej/sipregister/apps/app_directory.c
team/oej/sipregister/apps/app_disa.c
team/oej/sipregister/apps/app_exec.c
team/oej/sipregister/apps/app_externalivr.c
team/oej/sipregister/apps/app_festival.c
team/oej/sipregister/apps/app_hasnewvoicemail.c
team/oej/sipregister/apps/app_image.c
team/oej/sipregister/apps/app_macro.c
team/oej/sipregister/apps/app_meetme.c
team/oej/sipregister/apps/app_mixmonitor.c
team/oej/sipregister/apps/app_osplookup.c
team/oej/sipregister/apps/app_page.c
team/oej/sipregister/apps/app_parkandannounce.c
team/oej/sipregister/apps/app_playback.c
team/oej/sipregister/apps/app_privacy.c
team/oej/sipregister/apps/app_queue.c
team/oej/sipregister/apps/app_random.c
team/oej/sipregister/apps/app_read.c
team/oej/sipregister/apps/app_readfile.c
team/oej/sipregister/apps/app_record.c
team/oej/sipregister/apps/app_sayunixtime.c
team/oej/sipregister/apps/app_senddtmf.c
team/oej/sipregister/apps/app_sendtext.c
team/oej/sipregister/apps/app_setcallerid.c
team/oej/sipregister/apps/app_skel.c
team/oej/sipregister/apps/app_sms.c
team/oej/sipregister/apps/app_sql_postgres.c
team/oej/sipregister/apps/app_stack.c
team/oej/sipregister/apps/app_talkdetect.c
team/oej/sipregister/apps/app_transfer.c
team/oej/sipregister/apps/app_url.c
team/oej/sipregister/apps/app_userevent.c
team/oej/sipregister/apps/app_verbose.c
team/oej/sipregister/apps/app_voicemail.c
team/oej/sipregister/apps/app_while.c
team/oej/sipregister/apps/app_zapras.c
team/oej/sipregister/ast_expr2.fl
team/oej/sipregister/ast_expr2.y
team/oej/sipregister/callerid.c
team/oej/sipregister/channel.c
team/oej/sipregister/channels/chan_agent.c
team/oej/sipregister/channels/chan_alsa.c
team/oej/sipregister/channels/chan_iax2.c
team/oej/sipregister/channels/chan_local.c
team/oej/sipregister/channels/chan_mgcp.c
team/oej/sipregister/channels/chan_sip.c
team/oej/sipregister/channels/chan_zap.c
team/oej/sipregister/codecs/codec_a_mu.c
team/oej/sipregister/codecs/codec_adpcm.c
team/oej/sipregister/codecs/codec_alaw.c
team/oej/sipregister/codecs/codec_g723_1.c
team/oej/sipregister/codecs/codec_g726.c
team/oej/sipregister/codecs/codec_gsm.c
team/oej/sipregister/codecs/codec_ilbc.c
team/oej/sipregister/codecs/codec_lpc10.c
team/oej/sipregister/codecs/codec_speex.c
team/oej/sipregister/codecs/codec_ulaw.c
team/oej/sipregister/codecs/gsm/Makefile
team/oej/sipregister/config.c
team/oej/sipregister/configs/alsa.conf.sample
team/oej/sipregister/configs/sip.conf.sample
team/oej/sipregister/configs/zapata.conf.sample
team/oej/sipregister/devicestate.c
team/oej/sipregister/doc/README.variables
team/oej/sipregister/enum.c
team/oej/sipregister/formats/format_pcm.c
team/oej/sipregister/formats/format_pcm_alaw.c
team/oej/sipregister/funcs/func_cdr.c
team/oej/sipregister/funcs/func_cut.c
team/oej/sipregister/funcs/func_logic.c
team/oej/sipregister/funcs/func_math.c
team/oej/sipregister/funcs/func_md5.c
team/oej/sipregister/funcs/func_odbc.c
team/oej/sipregister/funcs/func_rand.c
team/oej/sipregister/funcs/func_strings.c
team/oej/sipregister/include/asterisk/astosp.h
team/oej/sipregister/include/asterisk/channel.h
team/oej/sipregister/include/asterisk/utils.h
team/oej/sipregister/logger.c
team/oej/sipregister/pbx.c
team/oej/sipregister/res/res_agi.c
team/oej/sipregister/res/res_crypto.c
team/oej/sipregister/res/res_features.c
team/oej/sipregister/res/res_indications.c
team/oej/sipregister/res/res_monitor.c
team/oej/sipregister/res/res_musiconhold.c
team/oej/sipregister/res/res_osp.c
team/oej/sipregister/udptl.c
Propchange: team/oej/sipregister/
------------------------------------------------------------------------------
--- svnmerge-blocked (original)
+++ svnmerge-blocked Mon Jan 23 06:12:19 2006
@@ -1,1 +1,1 @@
-/branches/1.2:7490,7497,7517,7529,7546,7550,7552,7557,7580,7586,7595,7605,7641,7663,7706,7738,7771,7792,7812,7870-7871,7898-7900,7915,7960,7965,7970,7976,8047,8112
+/branches/1.2:7490,7497,7517,7529,7546,7550,7552,7557,7580,7586,7595,7605,7641,7663,7706,7738,7771,7792,7812,7870-7871,7898-7900,7915,7960,7965,7970,7976,8047,8112,8394,8412,8418,8445
Propchange: team/oej/sipregister/
------------------------------------------------------------------------------
--- svnmerge-integrated (original)
+++ svnmerge-integrated Mon Jan 23 06:12:19 2006
@@ -1,1 +1,1 @@
-/trunk:1-8266
+/trunk:1-8461
Modified: team/oej/sipregister/apps/Makefile
URL: http://svn.digium.com/view/asterisk/team/oej/sipregister/apps/Makefile?rev=8462&r1=8461&r2=8462&view=diff
==============================================================================
--- team/oej/sipregister/apps/Makefile (original)
+++ team/oej/sipregister/apps/Makefile Mon Jan 23 06:12:19 2006
@@ -96,7 +96,7 @@
$(CC) $(SOLINK) -o $@ ${CYGSOLINK} $< ${CYGSOLIB} $(CURLLIBS)
app_sql_postgres.o: app_sql_postgres.c
- $(CC) -pipe -I/usr/local/pgsql/include $(CFLAGS) -c -o app_sql_postgres.o app_sql_postgres.c
+ $(CC) -pipe -I$(CROSS_COMPILE_TARGET)/usr/local/pgsql/include -I$(CROSS_COMPILE_TARGET)/usr/include/postgresql $(CFLAGS) -c -o app_sql_postgres.o app_sql_postgres.c
app_sql_postgres.so: app_sql_postgres.o
$(CC) $(SOLINK) -o $@ ${CYGSOLINK} $< ${CYGSOLIB} -L/usr/local/pgsql/lib -lpq
Modified: team/oej/sipregister/apps/app_authenticate.c
URL: http://svn.digium.com/view/asterisk/team/oej/sipregister/apps/app_authenticate.c?rev=8462&r1=8461&r2=8462&view=diff
==============================================================================
--- team/oej/sipregister/apps/app_authenticate.c (original)
+++ team/oej/sipregister/apps/app_authenticate.c Mon Jan 23 06:12:19 2006
@@ -128,9 +128,7 @@
}
}
- argcopy = ast_strdupa(data);
- if (!argcopy) {
- ast_log(LOG_ERROR, "Out of memory!\n");
+ if (!(argcopy = ast_strdupa(data))) {
LOCAL_USER_REMOVE(u);
return -1;
}
Modified: team/oej/sipregister/apps/app_chanspy.c
URL: http://svn.digium.com/view/asterisk/team/oej/sipregister/apps/app_chanspy.c?rev=8462&r1=8461&r2=8462&view=diff
==============================================================================
--- team/oej/sipregister/apps/app_chanspy.c (original)
+++ team/oej/sipregister/apps/app_chanspy.c Mon Jan 23 06:12:19 2006
@@ -383,10 +383,8 @@
struct ast_flags flags;
signed char zero_volume = 0;
- if (!(args = ast_strdupa((char *)data))) {
- ast_log(LOG_ERROR, "Out of memory!\n");
+ if (!(args = ast_strdupa(data)))
return -1;
- }
LOCAL_USER_ADD(u);
Modified: team/oej/sipregister/apps/app_curl.c
URL: http://svn.digium.com/view/asterisk/team/oej/sipregister/apps/app_curl.c?rev=8462&r1=8461&r2=8462&view=diff
==============================================================================
--- team/oej/sipregister/apps/app_curl.c (original)
+++ team/oej/sipregister/apps/app_curl.c Mon Jan 23 06:12:19 2006
@@ -128,9 +128,7 @@
LOCAL_USER_ACF_ADD(u);
- info = ast_strdupa(data);
- if (!info) {
- ast_log(LOG_ERROR, "Out of memory\n");
+ if (!(info = ast_strdupa(data))) {
LOCAL_USER_REMOVE(u);
return buf;
}
Modified: team/oej/sipregister/apps/app_db.c
URL: http://svn.digium.com/view/asterisk/team/oej/sipregister/apps/app_db.c?rev=8462&r1=8461&r2=8462&view=diff
==============================================================================
--- team/oej/sipregister/apps/app_db.c (original)
+++ team/oej/sipregister/apps/app_db.c Mon Jan 23 06:12:19 2006
@@ -75,9 +75,7 @@
LOCAL_USER_ADD(u);
- argv = ast_strdupa(data);
- if (!argv) {
- ast_log(LOG_ERROR, "Memory allocation failed\n");
+ if (!(argv = ast_strdupa(data))) {
LOCAL_USER_REMOVE(u);
return 0;
}
@@ -121,9 +119,7 @@
LOCAL_USER_ADD(u);
- argv = ast_strdupa(data);
- if (!argv) {
- ast_log (LOG_ERROR, "Memory allocation failed\n");
+ if (!(argv = ast_strdupa(data))) {
LOCAL_USER_REMOVE(u);
return 0;
}
Modified: team/oej/sipregister/apps/app_dial.c
URL: http://svn.digium.com/view/asterisk/team/oej/sipregister/apps/app_dial.c?rev=8462&r1=8461&r2=8462&view=diff
==============================================================================
--- team/oej/sipregister/apps/app_dial.c (original)
+++ team/oej/sipregister/apps/app_dial.c Mon Jan 23 06:12:19 2006
@@ -787,7 +787,6 @@
LOCAL_USER_ADD(u);
if (!(parse = ast_strdupa(data))) {
- ast_log(LOG_WARNING, "Memory allocation failure\n");
LOCAL_USER_REMOVE(u);
return -1;
}
@@ -1627,9 +1626,7 @@
LOCAL_USER_ADD(u);
- announce = ast_strdupa(data);
- if (!announce) {
- ast_log(LOG_ERROR, "Out of memory!\n");
+ if (!(announce = ast_strdupa(data))) {
LOCAL_USER_REMOVE(u);
return -1;
}
Modified: team/oej/sipregister/apps/app_dictate.c
URL: http://svn.digium.com/view/asterisk/team/oej/sipregister/apps/app_dictate.c?rev=8462&r1=8461&r2=8462&view=diff
==============================================================================
--- team/oej/sipregister/apps/app_dictate.c (original)
+++ team/oej/sipregister/apps/app_dictate.c Mon Jan 23 06:12:19 2006
@@ -110,11 +110,8 @@
snprintf(dftbase, sizeof(dftbase), "%s/dictate", ast_config_AST_SPOOL_DIR);
if (!ast_strlen_zero(data)) {
- parse = ast_strdupa(data);
- if (!parse) {
- ast_log(LOG_ERROR, "Out of memory!\n");
+ if (!(parse = ast_strdupa(data)))
return -1;
- }
AST_STANDARD_APP_ARGS(args, parse);
} else
args.argc = 0;
Modified: team/oej/sipregister/apps/app_directory.c
URL: http://svn.digium.com/view/asterisk/team/oej/sipregister/apps/app_directory.c?rev=8462&r1=8461&r2=8462&view=diff
==============================================================================
--- team/oej/sipregister/apps/app_directory.c (original)
+++ team/oej/sipregister/apps/app_directory.c Mon Jan 23 06:12:19 2006
@@ -431,10 +431,7 @@
LOCAL_USER_ADD(u);
- parse = ast_strdupa(data);
-
- if (!parse) {
- ast_log(LOG_ERROR, "Out of memory!\n");
+ if (!(parse = ast_strdupa(data))) {
LOCAL_USER_REMOVE(u);
return -1;
}
Modified: team/oej/sipregister/apps/app_disa.c
URL: http://svn.digium.com/view/asterisk/team/oej/sipregister/apps/app_disa.c?rev=8462&r1=8461&r2=8462&view=diff
==============================================================================
--- team/oej/sipregister/apps/app_disa.c (original)
+++ team/oej/sipregister/apps/app_disa.c Mon Jan 23 06:12:19 2006
@@ -161,9 +161,7 @@
ast_log(LOG_DEBUG, "Digittimeout: %d\n", digittimeout);
ast_log(LOG_DEBUG, "Responsetimeout: %d\n", firstdigittimeout);
- tmp = ast_strdupa(data);
- if (!tmp) {
- ast_log(LOG_ERROR, "Out of memory\n");
+ if (!(tmp = ast_strdupa(data))) {
LOCAL_USER_REMOVE(u);
return -1;
}
Modified: team/oej/sipregister/apps/app_exec.c
URL: http://svn.digium.com/view/asterisk/team/oej/sipregister/apps/app_exec.c?rev=8462&r1=8461&r2=8462&view=diff
==============================================================================
--- team/oej/sipregister/apps/app_exec.c (original)
+++ team/oej/sipregister/apps/app_exec.c Mon Jan 23 06:12:19 2006
@@ -73,8 +73,7 @@
/* Check and parse arguments */
if (data) {
- s = ast_strdupa((char *)data);
- if (s) {
+ if ((s = ast_strdupa(data))) {
appname = strsep(&s, "(");
if (s) {
endargs = strrchr(s, ')');
@@ -91,10 +90,8 @@
res = -1;
}
}
- } else {
- ast_log(LOG_ERROR, "Out of memory\n");
+ } else
res = -1;
- }
}
LOCAL_USER_REMOVE(u);
Modified: team/oej/sipregister/apps/app_externalivr.c
URL: http://svn.digium.com/view/asterisk/team/oej/sipregister/apps/app_externalivr.c?rev=8462&r1=8461&r2=8462&view=diff
==============================================================================
--- team/oej/sipregister/apps/app_externalivr.c (original)
+++ team/oej/sipregister/apps/app_externalivr.c Mon Jan 23 06:12:19 2006
@@ -271,9 +271,7 @@
goto exit;
}
- buf = ast_strdupa(data);
- if (!buf) {
- ast_log(LOG_ERROR, "Out of memory!\n");
+ if (!(buf = ast_strdupa(data))) {
LOCAL_USER_REMOVE(u);
return -1;
}
Modified: team/oej/sipregister/apps/app_festival.c
URL: http://svn.digium.com/view/asterisk/team/oej/sipregister/apps/app_festival.c?rev=8462&r1=8461&r2=8462&view=diff
==============================================================================
--- team/oej/sipregister/apps/app_festival.c (original)
+++ team/oej/sipregister/apps/app_festival.c Mon Jan 23 06:12:19 2006
@@ -339,9 +339,7 @@
festivalcommand = "(tts_textasterisk \"%s\" 'file)(quit)\n";
}
- data = ast_strdupa(vdata);
- if (!data) {
- ast_log(LOG_ERROR, "Out of memery\n");
+ if (!(data = ast_strdupa(vdata))) {
ast_config_destroy(cfg);
LOCAL_USER_REMOVE(u);
return -1;
Modified: team/oej/sipregister/apps/app_hasnewvoicemail.c
URL: http://svn.digium.com/view/asterisk/team/oej/sipregister/apps/app_hasnewvoicemail.c?rev=8462&r1=8461&r2=8462&view=diff
==============================================================================
--- team/oej/sipregister/apps/app_hasnewvoicemail.c (original)
+++ team/oej/sipregister/apps/app_hasnewvoicemail.c Mon Jan 23 06:12:19 2006
@@ -129,9 +129,7 @@
LOCAL_USER_ADD(u);
- input = ast_strdupa((char *)data);
- if (! input) {
- ast_log(LOG_ERROR, "Out of memory error\n");
+ if (!(input = ast_strdupa(data))) {
LOCAL_USER_REMOVE(u);
return -1;
}
@@ -193,9 +191,7 @@
buf[0] = '\0';
- argsstr = ast_strdupa(data);
- if (!argsstr) {
- ast_log(LOG_ERROR, "Out of memory");
+ if (!(argsstr = ast_strdupa(data))) {
LOCAL_USER_REMOVE(u);
return buf;
}
Modified: team/oej/sipregister/apps/app_image.c
URL: http://svn.digium.com/view/asterisk/team/oej/sipregister/apps/app_image.c?rev=8462&r1=8461&r2=8462&view=diff
==============================================================================
--- team/oej/sipregister/apps/app_image.c (original)
+++ team/oej/sipregister/apps/app_image.c Mon Jan 23 06:12:19 2006
@@ -79,7 +79,6 @@
LOCAL_USER_ADD(u);
if (!(parse = ast_strdupa(data))) {
- ast_log(LOG_WARNING, "Memory Error!\n");
LOCAL_USER_REMOVE(u);
return -1;
}
Modified: team/oej/sipregister/apps/app_macro.c
URL: http://svn.digium.com/view/asterisk/team/oej/sipregister/apps/app_macro.c?rev=8462&r1=8461&r2=8462&view=diff
==============================================================================
--- team/oej/sipregister/apps/app_macro.c (original)
+++ team/oej/sipregister/apps/app_macro.c Mon Jan 23 06:12:19 2006
@@ -307,9 +307,7 @@
LOCAL_USER_ADD(u);
- expr = ast_strdupa(data);
- if (!expr) {
- ast_log(LOG_ERROR, "Out of Memory!\n");
+ if (!(expr = ast_strdupa(data))) {
LOCAL_USER_REMOVE(u);
return -1;
}
Modified: team/oej/sipregister/apps/app_meetme.c
URL: http://svn.digium.com/view/asterisk/team/oej/sipregister/apps/app_meetme.c?rev=8462&r1=8461&r2=8462&view=diff
==============================================================================
--- team/oej/sipregister/apps/app_meetme.c (original)
+++ team/oej/sipregister/apps/app_meetme.c Mon Jan 23 06:12:19 2006
@@ -1778,11 +1778,8 @@
if (strcasecmp(var->name, "conf"))
continue;
- parse = ast_strdupa(var->value);
- if (!parse) {
- ast_log(LOG_ERROR, "Out of Memory!\n");
+ if (!(parse = ast_strdupa(var->value)))
return NULL;
- }
AST_STANDARD_APP_ARGS(args, parse);
if (!strcasecmp(args.confno, confno)) {
@@ -1817,7 +1814,7 @@
return cnf;
}
-/*--- count_exec: The MeetmeCount application */
+/*! \brief The MeetmeCount application */
static int count_exec(struct ast_channel *chan, void *data)
{
struct localuser *u;
@@ -1838,9 +1835,7 @@
LOCAL_USER_ADD(u);
- localdata = ast_strdupa(data);
- if (!localdata) {
- ast_log(LOG_ERROR, "Out of memory!\n");
+ if (!(localdata = ast_strdupa(data))) {
LOCAL_USER_REMOVE(u);
return -1;
}
@@ -1867,7 +1862,7 @@
return res;
}
-/*--- conf_exec: The meetme() application */
+/*! \brief The meetme() application */
static int conf_exec(struct ast_channel *chan, void *data)
{
int res=-1;
@@ -1987,8 +1982,6 @@
}
}
}
- } else {
- ast_log(LOG_ERROR, "Out of memory\n");
}
}
var = var->next;
@@ -2137,7 +2130,7 @@
return NULL;
}
-/*--- admin_exec: The MeetMeadmin application */
+/*! \brief The MeetMeadmin application */
/* MeetMeAdmin(confno, command, caller) */
static int admin_exec(struct ast_channel *chan, void *data) {
char *params;
Modified: team/oej/sipregister/apps/app_mixmonitor.c
URL: http://svn.digium.com/view/asterisk/team/oej/sipregister/apps/app_mixmonitor.c?rev=8462&r1=8461&r2=8462&view=diff
==============================================================================
--- team/oej/sipregister/apps/app_mixmonitor.c (original)
+++ team/oej/sipregister/apps/app_mixmonitor.c Mon Jan 23 06:12:19 2006
@@ -334,7 +334,6 @@
LOCAL_USER_ADD(u);
if (!(parse = ast_strdupa(data))) {
- ast_log(LOG_WARNING, "Memory Error!\n");
LOCAL_USER_REMOVE(u);
return -1;
}
Modified: team/oej/sipregister/apps/app_osplookup.c
URL: http://svn.digium.com/view/asterisk/team/oej/sipregister/apps/app_osplookup.c?rev=8462&r1=8461&r2=8462&view=diff
==============================================================================
--- team/oej/sipregister/apps/app_osplookup.c (original)
+++ team/oej/sipregister/apps/app_osplookup.c Mon Jan 23 06:12:19 2006
@@ -138,9 +138,7 @@
LOCAL_USER_ADD(u);
- temp = ast_strdupa(data);
- if (!temp) {
- ast_log(LOG_ERROR, "Out of memory!\n");
+ if (!(temp = ast_strdupa(data))) {
LOCAL_USER_REMOVE(u);
return -1;
}
@@ -201,9 +199,7 @@
LOCAL_USER_ADD(u);
- temp = ast_strdupa(data);
- if (!temp) {
- ast_log(LOG_ERROR, "Out of memory!\n");
+ if (!(temp = ast_strdupa(data))) {
LOCAL_USER_REMOVE(u);
return -1;
}
@@ -275,9 +271,7 @@
LOCAL_USER_ADD(u);
- temp = ast_strdupa(data);
- if (!temp) {
- ast_log(LOG_ERROR, "Out of memory!\n");
+ if (!(temp = ast_strdupa(data))) {
LOCAL_USER_REMOVE(u);
return -1;
}
Modified: team/oej/sipregister/apps/app_page.c
URL: http://svn.digium.com/view/asterisk/team/oej/sipregister/apps/app_page.c?rev=8462&r1=8461&r2=8462&view=diff
==============================================================================
--- team/oej/sipregister/apps/app_page.c (original)
+++ team/oej/sipregister/apps/app_page.c Mon Jan 23 06:12:19 2006
@@ -163,9 +163,7 @@
return -1;
};
- options = ast_strdupa(data);
- if (!options) {
- ast_log(LOG_ERROR, "Out of memory\n");
+ if (!(options = ast_strdupa(data))) {
LOCAL_USER_REMOVE(u);
return -1;
}
Modified: team/oej/sipregister/apps/app_parkandannounce.c
URL: http://svn.digium.com/view/asterisk/team/oej/sipregister/apps/app_parkandannounce.c?rev=8462&r1=8461&r2=8462&view=diff
==============================================================================
--- team/oej/sipregister/apps/app_parkandannounce.c (original)
+++ team/oej/sipregister/apps/app_parkandannounce.c Mon Jan 23 06:12:19 2006
@@ -59,12 +59,21 @@
static char *descrip =
" ParkAndAnnounce(announce:template|timeout|dial|[return_context]):\n"
-"Park a call into the parkinglot and announce the call over the console.\n"
-"announce template: colon separated list of files to announce, the word PARKED\n"
-" will be replaced by a say_digits of the ext the call is parked in\n"
-"timeout: time in seconds before the call returns into the return context.\n"
-"dial: The app_dial style resource to call to make the announcement. Console/dsp calls the console.\n"
-"return_context: the goto style label to jump the call back into after timeout. default=prio+1\n";
+"Park a call into the parkinglot and announce the call to another channel.\n"
+"\n"
+"announce template: Colon-separated list of files to announce. The word PARKED\n"
+" will be replaced by a say_digits of the extension in which\n"
+" the call is parked.\n"
+"timeout: Time in seconds before the call returns into the return\n"
+" context.\n"
+"dial: The app_dial style resource to call to make the\n"
+" announcement. Console/dsp calls the console.\n"
+"return_context: The goto-style label to jump the call back into after\n"
+" timeout. Default <priority+1>.\n"
+"\n"
+"The variable ${PARKEDAT} will contain the parking extension into which the\n"
+"call was placed. Use with the Local channel to allow the dialplan to make\n"
+"use of this information.\n";
STANDARD_LOCAL_USER;
@@ -79,6 +88,7 @@
char *working, *context, *exten, *priority, *dial, *dialtech, *dialstr;
char *template, *tpl_working, *tpl_current;
char *tmp[100];
+ char buf[13];
int looptemp=0,i=0;
char *s,*orig_s;
@@ -183,8 +193,10 @@
/* Now place the call to the extention */
+ snprintf(buf, sizeof(buf), "%d", lot);
memset(&oh, 0, sizeof(oh));
oh.parent_channel = chan;
+ oh.vars = ast_variable_new("_PARKEDAT", buf);
dchan = __ast_request_and_dial(dialtech, AST_FORMAT_SLINEAR, dialstr,30000, &outstate, chan->cid.cid_num, chan->cid.cid_name, &oh);
if(dchan) {
Modified: team/oej/sipregister/apps/app_playback.c
URL: http://svn.digium.com/view/asterisk/team/oej/sipregister/apps/app_playback.c?rev=8462&r1=8461&r2=8462&view=diff
==============================================================================
--- team/oej/sipregister/apps/app_playback.c (original)
+++ team/oej/sipregister/apps/app_playback.c Mon Jan 23 06:12:19 2006
@@ -89,11 +89,8 @@
return -1;
}
- tmp = ast_strdupa(data);
- if (!tmp) {
- ast_log(LOG_ERROR, "Out of memory!\n");
+ if (!(tmp = ast_strdupa(data)))
return -1;
- }
LOCAL_USER_ADD(u);
AST_STANDARD_APP_ARGS(args, tmp);
Modified: team/oej/sipregister/apps/app_privacy.c
URL: http://svn.digium.com/view/asterisk/team/oej/sipregister/apps/app_privacy.c?rev=8462&r1=8461&r2=8462&view=diff
==============================================================================
--- team/oej/sipregister/apps/app_privacy.c (original)
+++ team/oej/sipregister/apps/app_privacy.c Mon Jan 23 06:12:19 2006
@@ -117,9 +117,7 @@
if (!ast_strlen_zero((char *)data))
{
- parse = ast_strdupa(data);
- if (!parse) {
- ast_log(LOG_ERROR, "Out of memory!\n");
+ if (!(parse = ast_strdupa(data))) {
LOCAL_USER_REMOVE(u);
return -1;
}
Modified: team/oej/sipregister/apps/app_queue.c
URL: http://svn.digium.com/view/asterisk/team/oej/sipregister/apps/app_queue.c?rev=8462&r1=8461&r2=8462&view=diff
==============================================================================
--- team/oej/sipregister/apps/app_queue.c (original)
+++ team/oej/sipregister/apps/app_queue.c Mon Jan 23 06:12:19 2006
@@ -367,11 +367,10 @@
struct member *members; /*!< Head of the list of members */
struct queue_ent *head; /*!< Head of the list of callers */
- struct ast_call_queue *next; /*!< Next call queue */
+ AST_LIST_ENTRY(ast_call_queue) list; /*!< Next call queue */
};
-static struct ast_call_queue *queues = NULL;
-AST_MUTEX_DEFINE_STATIC(qlock);
+static AST_LIST_HEAD_STATIC(queues, ast_call_queue);
static int set_member_paused(char *queuename, char *interface, int paused);
@@ -481,8 +480,8 @@
}
if (option_debug)
ast_log(LOG_DEBUG, "Device '%s/%s' changed to state '%d' (%s)\n", technology, loc, sc->state, devstate2str(sc->state));
- ast_mutex_lock(&qlock);
- for (q = queues; q; q = q->next) {
+ AST_LIST_LOCK(&queues);
+ AST_LIST_TRAVERSE(&queues, q, list) {
ast_mutex_lock(&q->lock);
cur = q->members;
while(cur) {
@@ -508,7 +507,7 @@
}
ast_mutex_unlock(&q->lock);
}
- ast_mutex_unlock(&qlock);
+ AST_LIST_UNLOCK(&queues);
free(sc);
return NULL;
}
@@ -788,29 +787,48 @@
}
}
+static void free_members(struct ast_call_queue *q, int all)
+{
+ /* Free non-dynamic members */
+ struct member *curm, *next, *prev = NULL;
+
+ for (curm = q->members; curm; curm = next) {
+ next = curm->next;
+ if (all || !curm->dynamic) {
+ if (prev)
+ prev->next = next;
+ else
+ q->members = next;
+ free(curm);
+ } else
+ prev = curm;
+ }
+}
+
+static void destroy_queue(struct ast_call_queue *q)
+{
+ free_members(q, 1);
+ ast_mutex_destroy(&q->lock);
+ free(q);
+}
/*!\brief Reload a single queue via realtime.
\return Return the queue, or NULL if it doesn't exist.
- \note Should be called with the global qlock locked.
- When found, the queue is returned with q->lock locked. */
+ \note Should be called with the global qlock locked. */
static struct ast_call_queue *find_queue_by_name_rt(const char *queuename, struct ast_variable *queue_vars, struct ast_config *member_config)
{
struct ast_variable *v;
- struct ast_call_queue *q, *prev_q;
+ struct ast_call_queue *q;
struct member *m, *prev_m, *next_m;
char *interface;
char *tmp, *tmp_name;
char tmpbuf[64]; /* Must be longer than the longest queue param name. */
/* Find the queue in the in-core list (we will create a new one if not found). */
- q = queues;
- prev_q = NULL;
- while (q) {
+ AST_LIST_TRAVERSE(&queues, q, list) {
if (!strcasecmp(q->name, queuename)) {
break;
}
- q = q->next;
- prev_q = q;
}
/* Static queues override realtime. */
@@ -821,6 +839,7 @@
ast_mutex_unlock(&q->lock);
return NULL;
} else {
+ ast_mutex_unlock(&q->lock);
return q;
}
}
@@ -841,13 +860,9 @@
/* Delete if unused (else will be deleted when last caller leaves). */
if (!q->count) {
/* Delete. */
- if (!prev_q) {
- queues = q->next;
- } else {
- prev_q->next = q->next;
- }
+ AST_LIST_REMOVE(&queues, q, list);
ast_mutex_unlock(&q->lock);
- free(q);
+ destroy_queue(q);
} else
ast_mutex_unlock(&q->lock);
}
@@ -861,8 +876,7 @@
ast_mutex_lock(&q->lock);
clear_queue(q);
q->realtime = 1;
- q->next = queues;
- queues = q;
+ AST_LIST_INSERT_HEAD(&queues, q, list);
}
init_queue(q); /* Ensure defaults for all parameters not set explicitly. */
@@ -914,13 +928,60 @@
m = next_m;
}
+ ast_mutex_unlock(&q->lock);
+
return q;
}
-static int join_queue(char *queuename, struct queue_ent *qe, enum queue_result *reason)
+static struct ast_call_queue *load_realtime_queue(char *queuename)
{
struct ast_variable *queue_vars = NULL;
struct ast_config *member_config = NULL;
+ struct ast_call_queue *q;
+
+ /* Find the queue in the in-core list first. */
+ AST_LIST_LOCK(&queues);
+ AST_LIST_TRAVERSE(&queues, q, list) {
+ if (!strcasecmp(q->name, queuename)) {
+ break;
+ }
+ }
+ AST_LIST_UNLOCK(&queues);
+
+ if (!q) {
+ /*! \note Load from realtime before taking the global qlock, to avoid blocking all
+ queue operations while waiting for the DB.
+
+ This will be two separate database transactions, so we might
+ see queue parameters as they were before another process
+ changed the queue and member list as it was after the change.
+ Thus we might see an empty member list when a queue is
+ deleted. In practise, this is unlikely to cause a problem. */
+
+ queue_vars = ast_load_realtime("queues", "name", queuename, NULL);
+ if (queue_vars) {
+ member_config = ast_load_realtime_multientry("queue_members", "interface LIKE", "%", "queue_name", queuename, NULL);
+ if (!member_config) {
+ ast_log(LOG_ERROR, "no queue_members defined in your config (extconfig.conf).\n");
+ return NULL;
+ }
+ }
+
+ AST_LIST_LOCK(&queues);
+
+ q = find_queue_by_name_rt(queuename, queue_vars, member_config);
+ if (member_config)
+ ast_config_destroy(member_config);
+ if (queue_vars)
+ ast_variables_destroy(queue_vars);
+
+ AST_LIST_UNLOCK(&queues);
+ }
+ return q;
+}
+
+static int join_queue(char *queuename, struct queue_ent *qe, enum queue_result *reason)
+{
struct ast_call_queue *q;
struct queue_ent *cur, *prev = NULL;
int res = -1;
@@ -928,35 +989,12 @@
int inserted = 0;
enum queue_member_status stat;
- /*! \note Load from realtime before taking the global qlock, to avoid blocking all
- queue operations while waiting for the DB.
-
- This will be two separate database transactions, so we might
- see queue parameters as they were before another process
- changed the queue and member list as it was after the change.
- Thus we might see an empty member list when a queue is
- deleted. In practise, this is unlikely to cause a problem. */
- queue_vars = ast_load_realtime("queues", "name", queuename, NULL);
- if (queue_vars) {
- member_config = ast_load_realtime_multientry("queue_members", "interface LIKE", "%", "queue_name", queuename, NULL);
- if (!member_config) {
- ast_log(LOG_ERROR, "no queue_members defined in your config (extconfig.conf).\n");
- return res;
- }
- }
-
- ast_mutex_lock(&qlock);
- q = find_queue_by_name_rt(queuename, queue_vars, member_config);
- /* Note: If found, find_queue_by_name_rt() returns with q->lock locked. */
- if(member_config)
- ast_config_destroy(member_config);
- if(queue_vars)
- ast_variables_destroy(queue_vars);
-
- if (!q) {
- ast_mutex_unlock(&qlock);
+ q = load_realtime_queue(queuename);
+ if (!q)
return res;
- }
+
+ AST_LIST_LOCK(&queues);
+ ast_mutex_lock(&q->lock);
/* This is our one */
stat = get_member_status(q, qe->max_penalty);
@@ -1004,50 +1042,8 @@
#endif
}
ast_mutex_unlock(&q->lock);
- ast_mutex_unlock(&qlock);
+ AST_LIST_UNLOCK(&queues);
return res;
-}
-
-static void free_members(struct ast_call_queue *q, int all)
-{
- /* Free non-dynamic members */
- struct member *curm, *next, *prev;
-
- curm = q->members;
- prev = NULL;
- while(curm) {
- next = curm->next;
- if (all || !curm->dynamic) {
- if (prev)
- prev->next = next;
- else
- q->members = next;
- free(curm);
- } else
- prev = curm;
- curm = next;
- }
-}
-
-static void destroy_queue(struct ast_call_queue *q)
-{
- struct ast_call_queue *cur, *prev = NULL;
-
- ast_mutex_lock(&qlock);
- for (cur = queues; cur; cur = cur->next) {
- if (cur == q) {
- if (prev)
- prev->next = cur->next;
- else
- queues = cur->next;
- } else {
- prev = cur;
- }
- }
- ast_mutex_unlock(&qlock);
[... 7506 lines stripped ...]
More information about the asterisk-commits
mailing list