major redesign of the channel spy infrastructure, increasing efficiency and reducing locking conflicts

(nearly) complete rewrite of app_muxmon, renaming the application to MixMonitor and fixing a large number of bugs and inconsistencies
update app_chanspy to use new spy infrastructure


git-svn-id: http://svn.digium.com/svn/asterisk/trunk@6884 f38db490-d61c-443f-a65b-d21fe96a405b

3 files changed, 580 insertions, 921 deletions

diff --git a/apps/app_chanspy.c b/apps/app_chanspy.c
index 036c5edef..d9138ed00 100755
--- a/apps/app_chanspy.c
+++ b/apps/app_chanspy.c
@@ -35,7 +35,6 @@ ASTERISK_FILE_VERSION(__FILE__, "$Revision$")
 #include "asterisk/channel.h"
 #include "asterisk/features.h"
 #include "asterisk/options.h"
-#include "asterisk/slinfactory.h"
 #include "asterisk/app.h"
 #include "asterisk/utils.h"
 #include "asterisk/say.h"
@@ -68,6 +67,8 @@ static const char *desc = "   Chanspy([<scanspec>][|<options>])\n\n"
 "(e.g. run Chanspy(Agent) and dial 1234# while spying to jump to channel Agent/1234)\n\n"
 "";
 
+static const char *chanspy_spy_type = "ChanSpy";
+
 #define OPTION_QUIET	 (1 << 0)	/* Quiet, no announcement */
 #define OPTION_BRIDGED   (1 << 1)	/* Only look at bridged calls */
 #define OPTION_VOLUME    (1 << 2)	/* Specify initial volume */
@@ -88,39 +89,10 @@ LOCAL_USER_DECL;
 struct chanspy_translation_helper {
 	/* spy data */
 	struct ast_channel_spy spy;
-	int volfactor;
 	int fd;
-	struct ast_slinfactory slinfactory[2];
+	int volfactor;
 };
 
-/* Prototypes */
-static struct ast_channel *local_get_channel_begin_name(char *name);
-static struct ast_channel *local_channel_walk(struct ast_channel *chan);
-static void spy_release(struct ast_channel *chan, void *data);
-static void *spy_alloc(struct ast_channel *chan, void *params);
-static struct ast_frame *spy_queue_shift(struct ast_channel_spy *spy, int qnum);
-static void ast_flush_spy_queue(struct ast_channel_spy *spy);
-static int spy_generate(struct ast_channel *chan, void *data, int len, int samples);
-static void start_spying(struct ast_channel *chan, struct ast_channel *spychan, struct ast_channel_spy *spy);
-static void stop_spying(struct ast_channel *chan, struct ast_channel_spy *spy);
-static int channel_spy(struct ast_channel *chan, struct ast_channel *spyee, int *volfactor, int fd);
-static int chanspy_exec(struct ast_channel *chan, void *data);
-
-
-#if 0
-static struct ast_channel *local_get_channel_by_name(char *name) 
-{
-	struct ast_channel *ret;
-	ast_mutex_lock(&modlock);
-	if ((ret = ast_get_channel_by_name_locked(name))) {
-		ast_mutex_unlock(&ret->lock);
-	}
-	ast_mutex_unlock(&modlock);
-
-	return ret;
-}
-#endif
-
 static struct ast_channel *local_channel_walk(struct ast_channel *chan) 
 {
 	struct ast_channel *ret;
@@ -149,247 +121,85 @@ static struct ast_channel *local_get_channel_begin_name(char *name)
 	return ret;
 }
 
-
-static void spy_release(struct ast_channel *chan, void *data) 
-{
-	struct chanspy_translation_helper *csth = data;
-
-	ast_slinfactory_destroy(&csth->slinfactory[0]);
-	ast_slinfactory_destroy(&csth->slinfactory[1]);
-
-	return;
-}
-
-static void *spy_alloc(struct ast_channel *chan, void *params) 
-{
-	struct chanspy_translation_helper *csth = params;
-	ast_slinfactory_init(&csth->slinfactory[0]);
-	ast_slinfactory_init(&csth->slinfactory[1]);
-	return params;
-}
-
-static struct ast_frame *spy_queue_shift(struct ast_channel_spy *spy, int qnum) 
-{
-	struct ast_frame *f;
-	
-	if (qnum < 0 || qnum > 1)
-		return NULL;
-
-	f = spy->queue[qnum];
-	if (f) {
-		spy->queue[qnum] = f->next;
-		return f;
-	}
-	return NULL;
-}
-
-
-static void ast_flush_spy_queue(struct ast_channel_spy *spy) 
+static void *spy_alloc(struct ast_channel *chan, void *data)
 {
-	struct ast_frame *f=NULL;
-	int x = 0;
-	ast_mutex_lock(&spy->lock);
-	for(x=0;x<2;x++) {
-		f = NULL;
-		while((f = spy_queue_shift(spy, x))) 
-			ast_frfree(f);
-	}
-	ast_mutex_unlock(&spy->lock);
+	/* just store the data pointer in the channel structure */
+	return data;
 }
 
-
-#if 0
-static int extract_audio(short *buf, size_t len, struct ast_trans_pvt *trans, struct ast_frame *fr, int *maxsamp)
+static void spy_release(struct ast_channel *chan, void *data)
 {
-	struct ast_frame *f;
-	int size, retlen = 0;
-	
-	if (trans) {
-		if ((f = ast_translate(trans, fr, 0))) {
-			size = (f->datalen > len) ? len : f->datalen;
-			memcpy(buf, f->data, size);
-			retlen = f->datalen;
-			ast_frfree(f);
-		} else {
-			/* your guess is as good as mine why this will happen but it seems to only happen on iax and appears harmless */
-			ast_log(LOG_DEBUG, "Failed to translate frame from %s\n", ast_getformatname(fr->subclass));
-		}
-	} else {
-		size = (fr->datalen > len) ? len : fr->datalen;
-		memcpy(buf, fr->data, size);
-		retlen = fr->datalen;
-	}
-
-	if (retlen > 0 && (size = retlen / 2)) {
-		if (size > *maxsamp) {
-			*maxsamp = size;
-		}
-	}
-	
-	return retlen;
-}
-
-
-static int spy_queue_ready(struct ast_channel_spy *spy)
-{
-	int res = 0;
-
-	ast_mutex_lock(&spy->lock);
-	if (spy->status == CHANSPY_RUNNING) {
-		res = (spy->queue[0] && spy->queue[1]) ? 1 : 0;
-	} else {
-		res = (spy->queue[0] || spy->queue[1]) ? 1 : -1;
-	}
-	ast_mutex_unlock(&spy->lock);
-	return res;
+	/* nothing to do */
 }
-#endif
 
 static int spy_generate(struct ast_channel *chan, void *data, int len, int samples) 
 {
-
 	struct chanspy_translation_helper *csth = data;
-	struct ast_frame frame, *f;
-	int len0 = 0, len1 = 0, samp0 = 0, samp1 = 0, x, vf, maxsamp;
-	short buf0[1280], buf1[1280], buf[1280];
+	struct ast_frame *f;
 		
-	if (csth->spy.status == CHANSPY_DONE) {
+	if (csth->spy.status != CHANSPY_RUNNING)
 		/* Channel is already gone more than likely */
 		return -1;
-	}
 
 	ast_mutex_lock(&csth->spy.lock);
-	while((f = csth->spy.queue[0])) {
-		csth->spy.queue[0] = f->next;
-		ast_slinfactory_feed(&csth->slinfactory[0], f);
-		ast_frfree(f);
-	}
-	ast_mutex_unlock(&csth->spy.lock);
-	ast_mutex_lock(&csth->spy.lock);
-	while((f = csth->spy.queue[1])) {
-		csth->spy.queue[1] = f->next;
-		ast_slinfactory_feed(&csth->slinfactory[1], f);
-		ast_frfree(f);
-	}
+	f = ast_channel_spy_read_frame(&csth->spy, samples);
 	ast_mutex_unlock(&csth->spy.lock);
 		
-	if (csth->slinfactory[0].size < len || csth->slinfactory[1].size < len) {
+	if (!f)
 		return 0;
-	}
-		
-	if ((len0 = ast_slinfactory_read(&csth->slinfactory[0], buf0, len))) {
-		samp0 = len0 / 2;
-	} 
-	if ((len1 = ast_slinfactory_read(&csth->slinfactory[1], buf1, len))) {
-		samp1 = len1 / 2;
-	}
-
-	maxsamp = (samp0 > samp1) ? samp0 : samp1;
-	vf = get_volfactor(csth->volfactor);
 		
-	for(x=0; x < maxsamp; x++) {
-		if (vf < 0) {
-			if (samp0) {
-				buf0[x] /= abs(vf);
-			}
-			if (samp1) {
-				buf1[x] /= abs(vf);
-			}
-		} else if (vf > 0) {
-			if (samp0) {
-				buf0[x] *= vf;
-			}
-			if (samp1) {
-				buf1[x] *= vf;
-			}
-		}
-		if (samp0 && samp1) {
-			if (x < samp0 && x < samp1) {
-				buf[x] = buf0[x] + buf1[x];
-			} else if (x < samp0) {
-				buf[x] = buf0[x];
-			} else if (x < samp1) {
-				buf[x] = buf1[x];
-			}
-		} else if (x < samp0) {
-			buf[x] = buf0[x];
-		} else if (x < samp1) {
-			buf[x] = buf1[x];
-		}
-	}
-		
-	memset(&frame, 0, sizeof(frame));
-	frame.frametype = AST_FRAME_VOICE;
-	frame.subclass = AST_FORMAT_SLINEAR;
-	frame.data = buf;
-	frame.samples = x;
-	frame.datalen = x * 2;
-
-	if (ast_write(chan, &frame)) {
+	if (ast_write(chan, f)) {
+		ast_frfree(f);
 		return -1;
 	}
 
-	if (csth->fd) {
-		write(csth->fd, buf1, len1);
-	}
+	if (csth->fd)
+		write(csth->fd, f->data, f->datalen);
+
+	ast_frfree(f);
 
 	return 0;
 }
 
 
 static struct ast_generator spygen = {
-	alloc: spy_alloc, 
-	release: spy_release, 
-	generate: spy_generate, 
+	.alloc = spy_alloc,
+	.release = spy_release,
+	.generate = spy_generate, 
 };
 
-static void start_spying(struct ast_channel *chan, struct ast_channel *spychan, struct ast_channel_spy *spy) 
+static int start_spying(struct ast_channel *chan, struct ast_channel *spychan, struct ast_channel_spy *spy) 
 {
-
-	struct ast_channel_spy *cptr=NULL;
+	int res;
 	struct ast_channel *peer;
 
-
-	ast_log(LOG_WARNING, "Attaching %s to %s\n", spychan->name, chan->name);
-
+	ast_log(LOG_NOTICE, "Attaching %s to %s\n", spychan->name, chan->name);
 
 	ast_mutex_lock(&chan->lock);
-	if (chan->spiers) {
-		for(cptr=chan->spiers;cptr && cptr->next;cptr=cptr->next);
-		cptr->next = spy;
-	} else {
-		chan->spiers = spy;
-	}
+	res = ast_channel_spy_add(chan, spy);
 	ast_mutex_unlock(&chan->lock);
-	if ( ast_test_flag(chan, AST_FLAG_NBRIDGE) && (peer = ast_bridged_channel(chan))) {
+
+	if (!res && ast_test_flag(chan, AST_FLAG_NBRIDGE) && (peer = ast_bridged_channel(chan))) {
 		ast_softhangup(peer, AST_SOFTHANGUP_UNBRIDGE);	
 	}
 
+	return res;
 }
 
 static void stop_spying(struct ast_channel *chan, struct ast_channel_spy *spy) 
 {
-	struct ast_channel_spy *cptr=NULL, *prev=NULL;
-
 	/* If our status has changed, then the channel we're spying on is gone....
 	   DON'T TOUCH IT!!!  RUN AWAY!!! */
 	if (spy->status != CHANSPY_RUNNING)
 		return;
 
+	if (!chan)
+		return;
+
 	ast_mutex_lock(&chan->lock);
-	for(cptr=chan->spiers; cptr; cptr=cptr->next) {
-		if (cptr == spy) {
-			if (prev) {
-				prev->next = cptr->next;
-				cptr->next = NULL;
-			} else
-				chan->spiers = NULL;
-		}
-		prev = cptr;
-	}
+	ast_channel_spy_remove(chan, spy);
 	ast_mutex_unlock(&chan->lock);
-
-}
+};
 
 /* Map 'volume' levels from -4 through +4 into
    decibel (dB) settings for channel drivers
@@ -414,35 +224,47 @@ static void set_volume(struct ast_channel *chan, struct chanspy_translation_help
 {
 	signed char volume_adjust = volfactor_map[csth->volfactor + 4];
 
-	if (!ast_channel_setoption(chan, AST_OPTION_TXGAIN, &volume_adjust, sizeof(volume_adjust), 0)) {
+	if (!ast_channel_setoption(chan, AST_OPTION_TXGAIN, &volume_adjust, sizeof(volume_adjust), 0))
 		csth->volfactor = 0;
-	}
 }
 
 static int channel_spy(struct ast_channel *chan, struct ast_channel *spyee, int *volfactor, int fd) 
 {
 	struct chanspy_translation_helper csth;
-	int running = 1, res = 0, x = 0;
+	int running, res = 0, x = 0;
 	char inp[24];
 	char *name=NULL;
 	struct ast_frame *f;
 
-	if (chan && !ast_check_hangup(chan) && spyee && !ast_check_hangup(spyee)) {
+	running = (chan && !ast_check_hangup(chan) && spyee && !ast_check_hangup(spyee));
+
+	if (running) {
 		memset(inp, 0, sizeof(inp));
 		name = ast_strdupa(spyee->name);
 		if (option_verbose >= 2)
 			ast_verbose(VERBOSE_PREFIX_2 "Spying on channel %s\n", name);
 
 		memset(&csth, 0, sizeof(csth));
+		ast_set_flag(&csth.spy, CHANSPY_FORMAT_AUDIO);
+		ast_set_flag(&csth.spy, CHANSPY_TRIGGER_NONE);
+		ast_set_flag(&csth.spy, CHANSPY_MIXAUDIO);
+		csth.spy.type = chanspy_spy_type;
 		csth.spy.status = CHANSPY_RUNNING;
+		csth.spy.read_queue.format = AST_FORMAT_SLINEAR;
+		csth.spy.write_queue.format = AST_FORMAT_SLINEAR;
 		ast_mutex_init(&csth.spy.lock);
 		csth.volfactor = *volfactor;
 		set_volume(chan, &csth);
-		
-		if (fd) {
-			csth.fd = fd;
-		}
-		start_spying(spyee, chan, &csth.spy);
+		csth.spy.read_vol_adjustment = csth.volfactor;
+		csth.spy.write_vol_adjustment = csth.volfactor;
+		csth.fd = fd;
+
+		if (start_spying(spyee, chan, &csth.spy))
+			running = 0;
+	}
+
+	if (running) {
+		running = 1;
 		ast_activate_generator(chan, &spygen, &csth);
 
 		while (csth.spy.status == CHANSPY_RUNNING &&
@@ -487,6 +309,8 @@ static int channel_spy(struct ast_channel *chan, struct ast_channel *spyee, int
 					}
 					csth.volfactor = *volfactor;
 					set_volume(chan, &csth);
+					csth.spy.read_vol_adjustment = csth.volfactor;
+					csth.spy.write_vol_adjustment = csth.volfactor;
 				}
 			} else if (res >= 48 && res <= 57) {
 				inp[x++] = res;
@@ -498,11 +322,12 @@ static int channel_spy(struct ast_channel *chan, struct ast_channel *spyee, int
 		if (option_verbose >= 2) {
 			ast_verbose(VERBOSE_PREFIX_2 "Done Spying on channel %s\n", name);
 		}
-		ast_flush_spy_queue(&csth.spy);
 	} else {
 		running = 0;
 	}
+
 	ast_mutex_destroy(&csth.spy.lock);
+
 	return running;
 }
 
diff --git a/apps/app_mixmonitor.c b/apps/app_mixmonitor.c
index ab8045835..c166c71aa 100755
--- a/apps/app_mixmonitor.c
+++ b/apps/app_mixmonitor.c
@@ -1,46 +1,64 @@
 /*
- * Asterisk -- A telephony toolkit for Linux.
- *
+ * Asterisk -- An open source telephony toolkit.
  *
  * Copyright (C) 2005, Anthony Minessale II
+ * Copyright (C) 2005, Digium, Inc.
+ *
+ * Mark Spencer <markster@digium.com>
+ * Kevin P. Fleming <kpfleming@digium.com>
  *
+ * Based on app_muxmon.c provided by
  * Anthony Minessale II <anthmct@yahoo.com>
  *
+ * See http://www.asterisk.org for more information about
+ * the Asterisk project. Please do not directly contact
+ * any of the maintainers of this project for assistance;
+ * the project provides a web site, mailing lists and IRC
+ * channels for your use.
+ *
  * This program is free software, distributed under the terms of
- * the GNU General Public License
+ * the GNU General Public License Version 2. See the LICENSE file
+ * at the top of the source tree.
  */
 
 /*! \file
  * \brief muxmon() - record a call natively
  */
 
-#include <asterisk/file.h>
-#include <asterisk/logger.h>
-#include <asterisk/channel.h>
-#include <asterisk/pbx.h>
-#include <asterisk/module.h>
-#include <asterisk/lock.h>
-#include <asterisk/cli.h>
-#include <asterisk/options.h>
-#include <asterisk/app.h>
-#include <asterisk/translate.h>
-#include <asterisk/slinfactory.h>
 #include <stdlib.h>
 #include <unistd.h>
 #include <string.h>
+
+#include "asterisk.h"
+
+ASTERISK_FILE_VERSION(__FILE__, "$Revision$")
+
+#include "asterisk/file.h"
+#include "asterisk/logger.h"
+#include "asterisk/channel.h"
+#include "asterisk/pbx.h"
+#include "asterisk/module.h"
+#include "asterisk/lock.h"
+#include "asterisk/cli.h"
+#include "asterisk/options.h"
+#include "asterisk/app.h"
+#include "asterisk/linkedlists.h"
+
 #define get_volfactor(x) x ? ((x > 0) ? (1 << x) : ((1 << abs(x)) * -1)) : 0
-#define minmax(x,y) x ? (x > y) ? y : ((x < (y * -1)) ? (y * -1) : x) : 0 
-
-static char *tdesc = "Native Channel Monitoring Module";
-static char *app = "MuxMon";
-static char *synopsis = "Record A Call Natively";
-static char *desc = ""
-"  MuxMon(<file>.<ext>[|<options>[|<command>]])\n\n"
-"Records The audio on the current channel to the specified file.\n\n"
-"Valid Options:\n"
-" b    - Only save audio to the file while the channel is bridged. Note: does\n"
-"        not include conferences\n"
-" a    - Append to the file instead of overwriting it.\n"
+
+static const char *tdesc = "Mixed Audio Monitoring Application";
+static const char *app = "MixMonitor";
+static const char *synopsis = "Record a call and mix the audio during the recording";
+static const char *desc = ""
+"  MixMonitor(<file>.<ext>[|<options>[|<command>]])\n\n"
+"Records the audio on the current channel to the specified file.\n"
+"If the filename is an absolute path, uses that path, otherwise\n"
+"creates the file in the configured monitoring directory from\n"
+"asterisk.conf.\n\n"
+"Valid options:\n"
+" a      - Append to the file instead of overwriting it.\n"
+" b      - Only save audio to the file while the channel is bridged.\n"
+"          Note: does not include conferences.\n"
 " v(<x>) - Adjust the heard volume by a factor of <x> (range -4 to 4)\n"	
 " V(<x>) - Adjust the spoken volume by a factor of <x> (range -4 to 4)\n"	
 " W(<x>) - Adjust the both heard and spoken volumes by a factor of <x>\n"
@@ -48,14 +66,16 @@ static char *desc = ""
 "<command> will be executed when the recording is over\n"
 "Any strings matching ^{X} will be unescaped to ${X} and \n"
 "all variables will be evaluated at that time.\n"
-"The variable MUXMON_FILENAME will contain the filename used to record.\n"
+"The variable MIXMONITOR_FILENAME will contain the filename used to record.\n"
 "";
 
 STANDARD_LOCAL_USER;
 
 LOCAL_USER_DECL;
 
-struct muxmon {
+static const char *mixmonitor_spy_type = "MixMonitor";
+
+struct mixmonitor {
 	struct ast_channel *chan;
 	char *filename;
 	char *post_process;
@@ -64,445 +84,341 @@ struct muxmon {
 	int writevol;
 };
 
-typedef enum {
-    MUXFLAG_RUNNING = (1 << 0),
+enum {
     MUXFLAG_APPEND = (1 << 1),
     MUXFLAG_BRIDGED = (1 << 2),
     MUXFLAG_VOLUME = (1 << 3),
     MUXFLAG_READVOLUME = (1 << 4),
-    MUXFLAG_WRITEVOLUME = (1 << 5)
-} muxflags;
-
+    MUXFLAG_WRITEVOLUME = (1 << 5),
+} mixmonitor_flags;
 
-AST_DECLARE_OPTIONS(muxmon_opts,{
-    ['a'] = { MUXFLAG_APPEND },
+AST_DECLARE_OPTIONS(mixmonitor_opts,{
+	['a'] = { MUXFLAG_APPEND },
 	['b'] = { MUXFLAG_BRIDGED },
 	['v'] = { MUXFLAG_READVOLUME, 1 },
 	['V'] = { MUXFLAG_WRITEVOLUME, 2 },
 	['W'] = { MUXFLAG_VOLUME, 3 },
 });
 
-
 static void stopmon(struct ast_channel *chan, struct ast_channel_spy *spy) 
 {
-	struct ast_channel_spy *cptr=NULL, *prev=NULL;
-	int count = 0;
-
-	if (chan) {
-		while(ast_mutex_trylock(&chan->lock)) {
-			if (chan->spiers == spy) {
-				chan->spiers = NULL;
-				return;
-			}
-			count++;
-			if (count > 10) {
-				return;
-			}
-			sched_yield();
-		}
-		
-		for(cptr=chan->spiers; cptr; cptr=cptr->next) {
-			if (cptr == spy) {
-				if (prev) {
-					prev->next = cptr->next;
-					cptr->next = NULL;
-				} else
-					chan->spiers = NULL;
-			}
-			prev = cptr;
-		}
+	/* If our status has changed, then the channel we're spying on is gone....
+	   DON'T TOUCH IT!!!  RUN AWAY!!! */
+	if (spy->status != CHANSPY_RUNNING)
+		return;
 
-		ast_mutex_unlock(&chan->lock);
-	}
+	if (!chan)
+		return;
+
+	ast_mutex_lock(&chan->lock);
+	ast_channel_spy_remove(chan, spy);
+	ast_mutex_unlock(&chan->lock);
 }
 
-static void startmon(struct ast_channel *chan, struct ast_channel_spy *spy) 
+static int startmon(struct ast_channel *chan, struct ast_channel_spy *spy) 
 {
-
-	struct ast_channel_spy *cptr=NULL;
 	struct ast_channel *peer;
+	int res;
 
-	if (chan) {
-		ast_mutex_lock(&chan->lock);
-		if (chan->spiers) {
-			for(cptr=chan->spiers;cptr->next;cptr=cptr->next);
-			cptr->next = spy;
-		} else {
-			chan->spiers = spy;
-		}
-		ast_mutex_unlock(&chan->lock);
+	if (!chan)
+		return -1;
+
+	ast_mutex_lock(&chan->lock);
+	res = ast_channel_spy_add(chan, spy);
+	ast_mutex_unlock(&chan->lock);
 		
-		if (ast_test_flag(chan, AST_FLAG_NBRIDGE) && (peer = ast_bridged_channel(chan))) {
-			ast_softhangup(peer, AST_SOFTHANGUP_UNBRIDGE);	
-		}
-	}
-}
+	if (!res && ast_test_flag(chan, AST_FLAG_NBRIDGE) && (peer = ast_bridged_channel(chan)))
+		ast_softhangup(peer, AST_SOFTHANGUP_UNBRIDGE);	
 
-static int spy_queue_translate(struct ast_channel_spy *spy,
-							   struct ast_slinfactory *slinfactory0,
-							   struct ast_slinfactory *slinfactory1)
-{
-	int res = 0;
-	struct ast_frame *f;
-	
-	ast_mutex_lock(&spy->lock);
-	while((f = spy->queue[0])) {
-		spy->queue[0] = f->next;
-		ast_slinfactory_feed(slinfactory0, f);
-		ast_frfree(f);
-	}
-	ast_mutex_unlock(&spy->lock);
-	ast_mutex_lock(&spy->lock);
-	while((f = spy->queue[1])) {
-		spy->queue[1] = f->next;
-		ast_slinfactory_feed(slinfactory1, f);
-		ast_frfree(f);
-	}
-	ast_mutex_unlock(&spy->lock);
 	return res;
 }
 
-static void *muxmon_thread(void *obj) 
-{
+#define SAMPLES_PER_FRAME 160
 
-	int len0 = 0, len1 = 0, samp0 = 0, samp1 = 0, framelen, maxsamp = 0, x = 0;
-	short buf0[1280], buf1[1280], buf[1280];
-	struct ast_frame frame;
-	struct muxmon *muxmon = obj;
+static void *mixmonitor_thread(void *obj) 
+{
+	struct mixmonitor *mixmonitor = obj;
 	struct ast_channel_spy spy;
 	struct ast_filestream *fs = NULL;
 	char *ext, *name;
 	unsigned int oflags;
-	struct ast_slinfactory slinfactory[2];
+	struct ast_frame *f;
 	char post_process[1024] = "";
 	
-	name = ast_strdupa(muxmon->chan->name);
+	STANDARD_INCREMENT_USECOUNT;
 
-	framelen = 320;
-	frame.frametype = AST_FRAME_VOICE;
-	frame.subclass = AST_FORMAT_SLINEAR;
-	frame.data = buf;
-	ast_set_flag(muxmon, MUXFLAG_RUNNING);
-	oflags = O_CREAT|O_WRONLY;
-	ast_slinfactory_init(&slinfactory[0]);
-	ast_slinfactory_init(&slinfactory[1]);
-	
+	name = ast_strdupa(mixmonitor->chan->name);
 
-
-	/* for efficiency, use a flag to bypass volume logic when it's not needed */
-	if (muxmon->readvol || muxmon->writevol) {
-		ast_set_flag(muxmon, MUXFLAG_VOLUME);
-	}
-
-	if ((ext = strchr(muxmon->filename, '.'))) {
+	oflags = O_CREAT|O_WRONLY;
+	oflags |= ast_test_flag(mixmonitor, MUXFLAG_APPEND) ? O_APPEND : O_TRUNC;
+		
+	if ((ext = strchr(mixmonitor->filename, '.'))) {
 		*(ext++) = '\0';
 	} else {
 		ext = "raw";
 	}
 
+	fs = ast_writefile(mixmonitor->filename, ext, NULL, oflags, 0, 0644);
+	if (!fs) {
+		ast_log(LOG_ERROR, "Cannot open %s.%s\n", mixmonitor->filename, ext);
+		goto out;
+	}
+
+	if (ast_test_flag(mixmonitor, MUXFLAG_APPEND))
+		ast_seekstream(fs, 0, SEEK_END);
+	
 	memset(&spy, 0, sizeof(spy));
+	ast_set_flag(&spy, CHANSPY_FORMAT_AUDIO);
+	ast_set_flag(&spy, CHANSPY_MIXAUDIO);
+	spy.type = mixmonitor_spy_type;
 	spy.status = CHANSPY_RUNNING;
+	spy.read_queue.format = AST_FORMAT_SLINEAR;
+	spy.write_queue.format = AST_FORMAT_SLINEAR;
+	if (mixmonitor->readvol) {
+		ast_set_flag(&spy, CHANSPY_READ_VOLADJUST);
+		spy.read_vol_adjustment = mixmonitor->readvol;
+	}
+	if (mixmonitor->writevol) {
+		ast_set_flag(&spy, CHANSPY_WRITE_VOLADJUST);
+		spy.write_vol_adjustment = mixmonitor->writevol;
+	}
 	ast_mutex_init(&spy.lock);
-	startmon(muxmon->chan, &spy);
-	if (ast_test_flag(muxmon, MUXFLAG_RUNNING)) {
-		if (option_verbose > 1) {
-			ast_verbose(VERBOSE_PREFIX_2 "Begin Muxmon Recording %s\n", name);
-		}
 
-		oflags |= ast_test_flag(muxmon, MUXFLAG_APPEND) ? O_APPEND : O_TRUNC;
-		
-		if (!(fs = ast_writefile(muxmon->filename, ext, NULL, oflags, 0, 0644))) {
-			ast_log(LOG_ERROR, "Cannot open %s\n", muxmon->filename);
-			spy.status = CHANSPY_DONE;
-		}  else {
+	if (startmon(mixmonitor->chan, &spy)) {
+		ast_log(LOG_WARNING, "Unable to add '%s' spy to channel '%s'\n",
+			spy.type, mixmonitor->chan->name);
+		goto out2;
+	}
 
-			if (ast_test_flag(muxmon, MUXFLAG_APPEND)) {
-				ast_seekstream(fs, 0, SEEK_END);
-			}
+	if (option_verbose > 1)
+		ast_verbose(VERBOSE_PREFIX_2 "Begin MixMonitor Recording %s\n", name);
+	
+	while (1) {
+		struct ast_frame *next;
+		int write;
+
+		ast_mutex_lock(&spy.lock);
 
-			while (ast_test_flag(muxmon, MUXFLAG_RUNNING)) {
-				samp0 = samp1 = len0 = len1 = 0;
-
-				if (ast_check_hangup(muxmon->chan) || spy.status != CHANSPY_RUNNING) {
-					ast_clear_flag(muxmon, MUXFLAG_RUNNING);
-					break;
-				}
-
-				if (ast_test_flag(muxmon, MUXFLAG_BRIDGED) && !ast_bridged_channel(muxmon->chan)) {
-					usleep(1000);
-					sched_yield();
-					continue;
-				}
-				
-				spy_queue_translate(&spy, &slinfactory[0], &slinfactory[1]);
-				
-				if (slinfactory[0].size < framelen || slinfactory[1].size < framelen) {
-					usleep(1000);
-					sched_yield();
-					continue;
-				}
-
-				if ((len0 = ast_slinfactory_read(&slinfactory[0], buf0, framelen))) {
-					samp0 = len0 / 2;
-				}
-				if((len1 = ast_slinfactory_read(&slinfactory[1], buf1, framelen))) {
-					samp1 = len1 / 2;
-				}
-				
-				if (ast_test_flag(muxmon, MUXFLAG_VOLUME)) {
-					if (samp0 && muxmon->readvol > 0) {
-						for(x=0; x < samp0 / 2; x++) {
-							buf0[x] *= muxmon->readvol;
-						}
-					} else if (samp0 && muxmon->readvol < 0) {
-						for(x=0; x < samp0 / 2; x++) {
-							buf0[x] /= muxmon->readvol;
-						}
-					}
-					if (samp1 && muxmon->writevol > 0) {
-						for(x=0; x < samp1 / 2; x++) {
-							buf1[x] *= muxmon->writevol;
-						}
-					} else if (muxmon->writevol < 0) {
-						for(x=0; x < samp1 / 2; x++) {
-							buf1[x] /= muxmon->writevol;
-						}
-					}
-				}
-				
-				maxsamp = (samp0 > samp1) ? samp0 : samp1;
-
-				if (samp0 && samp1) {
-					for(x=0; x < maxsamp; x++) {
-						if (x < samp0 && x < samp1) {
-							buf[x] = buf0[x] + buf1[x];
-						} else if (x < samp0) {
-							buf[x] = buf0[x];
-						} else if (x < samp1) {
-							buf[x] = buf1[x];
-						}
-					}
-				} else if(samp0) {
-					memcpy(buf, buf0, len0);
-					x = samp0;
-				} else if(samp1) {
-					memcpy(buf, buf1, len1);
-					x = samp1;
-				}
-
-				frame.samples = x;
-				frame.datalen = x * 2;
-				ast_writestream(fs, &frame);
+		ast_channel_spy_trigger_wait(&spy);
 		
-				usleep(1000);
-				sched_yield();
+		if (ast_check_hangup(mixmonitor->chan) || spy.status != CHANSPY_RUNNING) {
+			ast_mutex_unlock(&spy.lock);
+			break;
+		}
+		
+		while (1) {
+			if (!(f = ast_channel_spy_read_frame(&spy, SAMPLES_PER_FRAME)))
+				break;
+
+			write = (!ast_test_flag(mixmonitor, MUXFLAG_BRIDGED) ||
+				 ast_bridged_channel(mixmonitor->chan));
+
+			/* it is possible for ast_channel_spy_read_frame() to return a chain
+			   of frames if a queue flush was necessary, so process them
+			*/
+			for (; f; f = next) {
+				next = f->next;
+				if (write)
+					ast_writestream(fs, f);
+				ast_frfree(f);
 			}
 		}
-	}
 
-	if (muxmon->post_process) {
+		ast_mutex_unlock(&spy.lock);
+	}
+	
+	if (mixmonitor->post_process) {
 		char *p;
-		for(p = muxmon->post_process; *p ; p++) {
+
+		for (p = mixmonitor->post_process; *p ; p++) {
 			if (*p == '^' && *(p+1) == '{') {
 				*p = '$';
 			}
 		}
-		pbx_substitute_variables_helper(muxmon->chan, muxmon->post_process, post_process, sizeof(post_process) - 1);
-		free(muxmon->post_process);
-		muxmon->post_process = NULL;
+		pbx_substitute_variables_helper(mixmonitor->chan, mixmonitor->post_process, post_process, sizeof(post_process) - 1);
 	}
 
-	stopmon(muxmon->chan, &spy);
-	if (option_verbose > 1) {
-		ast_verbose(VERBOSE_PREFIX_2 "Finished Recording %s\n", name);
-	}
-	ast_mutex_destroy(&spy.lock);
-	
-	if(fs) {
-		ast_closestream(fs);
-	}
-	
-	ast_slinfactory_destroy(&slinfactory[0]);
-	ast_slinfactory_destroy(&slinfactory[1]);
+	stopmon(mixmonitor->chan, &spy);
 
-	if (muxmon) {
-		if (muxmon->filename) {
-			free(muxmon->filename);
-		}
-		free(muxmon);
-	}
+	if (option_verbose > 1)
+		ast_verbose(VERBOSE_PREFIX_2 "End MixMonitor Recording %s\n", name);
 
 	if (!ast_strlen_zero(post_process)) {
-		if (option_verbose > 2) {
+		if (option_verbose > 2)
 			ast_verbose(VERBOSE_PREFIX_2 "Executing [%s]\n", post_process);
-		}
 		ast_safe_system(post_process);
 	}
 
+out2:
+	ast_mutex_destroy(&spy.lock);
+
+	if (fs)
+		ast_closestream(fs);
+
+out:
+	free(mixmonitor);
+
+	STANDARD_DECREMENT_USECOUNT;
+
 	return NULL;
 }
 
-static void launch_monitor_thread(struct ast_channel *chan, char *filename, unsigned int flags, int readvol , int writevol, char *post_process) 
+static void launch_monitor_thread(struct ast_channel *chan, const char *filename, unsigned int flags,
+				  int readvol, int writevol, const char *post_process) 
 {
 	pthread_attr_t attr;
-	int result = 0;
 	pthread_t thread;
-	struct muxmon *muxmon;
+	struct mixmonitor *mixmonitor;
+	int len;
 
+	len = sizeof(*mixmonitor) + strlen(filename) + 1;
+	if (post_process && !ast_strlen_zero(post_process))
+		len += strlen(post_process) + 1;
 
-	if (!(muxmon = malloc(sizeof(struct muxmon)))) {
+	if (!(mixmonitor = calloc(1, len))) {
 		ast_log(LOG_ERROR, "Memory Error!\n");
 		return;
 	}
 
-	memset(muxmon, 0, sizeof(struct muxmon));
-	muxmon->chan = chan;
-	muxmon->filename = strdup(filename);
-	if(post_process) {
-		muxmon->post_process = strdup(post_process);
+	mixmonitor->chan = chan;
+	mixmonitor->filename = (char *) mixmonitor + sizeof(*mixmonitor);
+	strcpy(mixmonitor->filename, filename);
+	if (post_process && !ast_strlen_zero(post_process)) {
+		mixmonitor->post_process = mixmonitor->filename + strlen(filename) + 1;
+		strcpy(mixmonitor->post_process, post_process);
 	}
-	muxmon->readvol = readvol;
-	muxmon->writevol = writevol;
-	muxmon->flags = flags;
+	mixmonitor->readvol = readvol;
+	mixmonitor->writevol = writevol;
+	mixmonitor->flags = flags;
 
-	result = pthread_attr_init(&attr);
-	pthread_attr_setschedpolicy(&attr, SCHED_RR);
+	pthread_attr_init(&attr);
 	pthread_attr_setdetachstate(&attr, PTHREAD_CREATE_DETACHED);
-	result = ast_pthread_create(&thread, &attr, muxmon_thread, muxmon);
-	result = pthread_attr_destroy(&attr);
+	ast_pthread_create(&thread, &attr, mixmonitor_thread, mixmonitor);
+	pthread_attr_destroy(&attr);
 }
 
-
-static int muxmon_exec(struct ast_channel *chan, void *data)
+static int mixmonitor_exec(struct ast_channel *chan, void *data)
 {
-	int res = 0, x = 0, readvol = 0, writevol = 0;
+	int x, readvol = 0, writevol = 0;
 	struct localuser *u;
 	struct ast_flags flags = {0};
-	int argc;
-	char *options = NULL,
-		*args,
-		*argv[3],
-		*filename = NULL,
-		*post_process = NULL;
+	char *parse;
+	AST_DECLARE_APP_ARGS(args,
+		AST_APP_ARG(filename);
+		AST_APP_ARG(options);
+		AST_APP_ARG(post_process);
+	);
 	
 	if (ast_strlen_zero(data)) {
-		ast_log(LOG_WARNING, "muxmon requires an argument\n");
+		ast_log(LOG_WARNING, "MixMonitor requires an argument (filename)\n");
 		return -1;
 	}
 
 	LOCAL_USER_ADD(u);
 
-	args = ast_strdupa(data);	
-	if (!args) {
+	if (!(parse = ast_strdupa(data))) {
 		ast_log(LOG_WARNING, "Memory Error!\n");
 		LOCAL_USER_REMOVE(u);
 		return -1;
 	}
 
-	if ((argc = ast_separate_app_args(args, '|', argv, sizeof(argv) / sizeof(argv[0])))) {
-		filename = argv[0];
-		if (argc > 1) {
-			options = argv[1];
-		}
-		if (argc > 2) {
-			post_process = argv[2];
-		}
-	}
+	AST_STANDARD_APP_ARGS(args, parse);
 	
-	if (ast_strlen_zero(filename)) {
+	if (ast_strlen_zero(args.filename)) {
 		ast_log(LOG_WARNING, "Muxmon requires an argument (filename)\n");
 		LOCAL_USER_REMOVE(u);
 		return -1;
 	}
 
-	if (options) {
-		char *opts[3] = {};
-		ast_parseoptions(muxmon_opts, &flags, opts, options);
-
-		if (ast_test_flag(&flags, MUXFLAG_READVOLUME) && opts[0]) {
-			if (sscanf(opts[0], "%d", &x) != 1)
-				ast_log(LOG_NOTICE, "volume must be a number between -4 and 4\n");
-			else {
-				readvol = minmax(x, 4);
-				x = get_volfactor(readvol);
-				readvol = minmax(x, 16);
+	if (args.options) {
+		char *opts[3] = { NULL, };
+
+		ast_parseoptions(mixmonitor_opts, &flags, opts, args.options);
+
+		if (ast_test_flag(&flags, MUXFLAG_READVOLUME)) {
+			if (!opts[0] || ast_strlen_zero(opts[0])) {
+				ast_log(LOG_WARNING, "No volume level was provided for the heard volume ('v') option.\n");
+			} else if ((sscanf(opts[0], "%d", &x) != 1) || (x < -4) || (x > 4)) {
+				ast_log(LOG_NOTICE, "Heard volume must be a number between -4 and 4, not '%s'\n", opts[0]);
+			} else {
+				readvol = get_volfactor(x);
 			}
 		}
 		
-		if (ast_test_flag(&flags, MUXFLAG_WRITEVOLUME) && opts[1]) {
-			if (sscanf(opts[1], "%d", &x) != 1)
-				ast_log(LOG_NOTICE, "volume must be a number between -4 and 4\n");
-			else {
-				writevol = minmax(x, 4);
-				x = get_volfactor(writevol);
-				writevol = minmax(x, 16);
+		if (ast_test_flag(&flags, MUXFLAG_WRITEVOLUME)) {
+			if (!opts[1] || ast_strlen_zero(opts[1])) {
+				ast_log(LOG_WARNING, "No volume level was provided for the spoken volume ('V') option.\n");
+			} else if ((sscanf(opts[1], "%d", &x) != 1) || (x < -4) || (x > 4)) {
+				ast_log(LOG_NOTICE, "Spoken volume must be a number between -4 and 4, not '%s'\n", opts[1]);
+			} else {
+				writevol = get_volfactor(x);
 			}
 		}
-
-		if (ast_test_flag(&flags, MUXFLAG_VOLUME) && opts[2]) {
-			if (sscanf(opts[2], "%d", &x) != 1)
-				ast_log(LOG_NOTICE, "volume must be a number between -4 and 4\n");
-			else {
-				readvol = writevol = minmax(x, 4);
-				x = get_volfactor(readvol);
-				readvol = minmax(x, 16);
-				x = get_volfactor(writevol);
-				writevol = minmax(x, 16);
+		
+		if (ast_test_flag(&flags, MUXFLAG_VOLUME)) {
+			if (!opts[2] || ast_strlen_zero(opts[2])) {
+				ast_log(LOG_WARNING, "No volume level was provided for the combined volume ('W') option.\n");
+			} else if ((sscanf(opts[2], "%d", &x) != 1) || (x < -4) || (x > 4)) {
+				ast_log(LOG_NOTICE, "Combined volume must be a number between -4 and 4, not '%s'\n", opts[2]);
+			} else {
+				readvol = writevol = get_volfactor(x);
 			}
 		}
 	}
-	pbx_builtin_setvar_helper(chan, "MUXMON_FILENAME", filename);
-	launch_monitor_thread(chan, filename, flags.flags, readvol, writevol, post_process);
+
+	/* if not provided an absolute path, use the system-configured monitoring directory */
+	if (args.filename[0] != '/') {
+		char *build;
+
+		build = alloca(strlen(ast_config_AST_MONITOR_DIR) + strlen(args.filename) + 3);
+		sprintf(build, "%s/%s", ast_config_AST_MONITOR_DIR, args.filename);
+		args.filename = build;
+	}
+
+	pbx_builtin_setvar_helper(chan, "MIXMONITOR_FILENAME", args.filename);
+	launch_monitor_thread(chan, args.filename, flags.flags, readvol, writevol, args.post_process);
 
 	LOCAL_USER_REMOVE(u);
-	return res;
-}
 
+	return 0;
+}
 
-static int muxmon_cli(int fd, int argc, char **argv) 
+static int mixmonitor_cli(int fd, int argc, char **argv) 
 {
-	char *op, *chan_name = NULL, *args = NULL;
 	struct ast_channel *chan;
 
-	if (argc > 2) {
-		op = argv[1];
-		chan_name = argv[2];
-
-		if (argv[3]) {
-			args = argv[3];
-		}
+	if (argc < 3)
+		return RESULT_SHOWUSAGE;
 
-		if (!(chan = ast_get_channel_by_name_prefix_locked(chan_name, strlen(chan_name)))) {
-			ast_cli(fd, "Invalid Channel!\n");
-			return -1;
-		}
-		if (!strcasecmp(op, "start")) {
-			muxmon_exec(chan, args);
-		} else if (!strcasecmp(op, "stop")) {
-			struct ast_channel_spy *cptr=NULL;
-			for(cptr=chan->spiers; cptr; cptr=cptr->next) {
-				cptr->status = CHANSPY_DONE;
-			}
-		}
-		ast_mutex_unlock(&chan->lock);
-		return 0;
+	if (!(chan = ast_get_channel_by_name_prefix_locked(argv[2], strlen(argv[2])))) {
+		ast_cli(fd, "No channel matching '%s' found.\n", argv[2]);
+		return RESULT_SUCCESS;
 	}
 
-	ast_cli(fd, "Usage: muxmon <start|stop> <chan_name> <args>\n");
-	return -1;
+	if (!strcasecmp(argv[1], "start"))
+		mixmonitor_exec(chan, argv[3]);
+	else if (!strcasecmp(argv[1], "stop"))
+		ast_channel_spy_stop_by_type(chan, mixmonitor_spy_type);
+
+	ast_mutex_unlock(&chan->lock);
+
+	return RESULT_SUCCESS;
 }
 
 
-static struct ast_cli_entry cli_muxmon = {
-	{ "muxmon", NULL, NULL }, muxmon_cli, 
-	"Execute a monitor command", "muxmon <start|stop> <chan_name> <args>"};
+static struct ast_cli_entry cli_mixmonitor = {
+	{ "mixmonitor", NULL, NULL },
+	mixmonitor_cli, 
+	"Execute a MixMonitor command",
+	"mixmonitor <start|stop> <chan_name> [<args>]"
+};
 
 
 int unload_module(void)
 {
 	int res;
 
-	res = ast_cli_unregister(&cli_muxmon);
+	res = ast_cli_unregister(&cli_mixmonitor);
 	res |= ast_unregister_application(app);
 	
 	STANDARD_HANGUP_LOCALUSERS;
@@ -514,21 +430,23 @@ int load_module(void)
 {
 	int res;
 
-	res = ast_cli_register(&cli_muxmon);
-	res |= ast_register_application(app, muxmon_exec, synopsis, desc);
+	res = ast_cli_register(&cli_mixmonitor);
+	res |= ast_register_application(app, mixmonitor_exec, synopsis, desc);
 
 	return res;
 }
 
 char *description(void)
 {
-	return tdesc;
+	return (char *) tdesc;
 }
 
 int usecount(void)
 {
 	int res;
+
 	STANDARD_USECOUNT(res);
+
 	return res;
 }
 
@@ -536,4 +454,3 @@ char *key()
 {
 	return ASTERISK_GPL_KEY;
 }
-
diff --git a/apps/app_muxmon.c b/apps/app_muxmon.c
index ab8045835..c166c71aa 100755
--- a/apps/app_muxmon.c
+++ b/apps/app_muxmon.c
@@ -1,46 +1,64 @@
 /*
- * Asterisk -- A telephony toolkit for Linux.
- *
+ * Asterisk -- An open source telephony toolkit.
  *
  * Copyright (C) 2005, Anthony Minessale II
+ * Copyright (C) 2005, Digium, Inc.
+ *
+ * Mark Spencer <markster@digium.com>
+ * Kevin P. Fleming <kpfleming@digium.com>
  *
+ * Based on app_muxmon.c provided by
  * Anthony Minessale II <anthmct@yahoo.com>
  *
+ * See http://www.asterisk.org for more information about
+ * the Asterisk project. Please do not directly contact
+ * any of the maintainers of this project for assistance;
+ * the project provides a web site, mailing lists and IRC
+ * channels for your use.
+ *
  * This program is free software, distributed under the terms of
- * the GNU General Public License
+ * the GNU General Public License Version 2. See the LICENSE file
+ * at the top of the source tree.
  */
 
 /*! \file
  * \brief muxmon() - record a call natively
  */
 
-#include <asterisk/file.h>
-#include <asterisk/logger.h>
-#include <asterisk/channel.h>
-#include <asterisk/pbx.h>
-#include <asterisk/module.h>
-#include <asterisk/lock.h>
-#include <asterisk/cli.h>
-#include <asterisk/options.h>
-#include <asterisk/app.h>
-#include <asterisk/translate.h>
-#include <asterisk/slinfactory.h>
 #include <stdlib.h>
 #include <unistd.h>
 #include <string.h>
+
+#include "asterisk.h"
+
+ASTERISK_FILE_VERSION(__FILE__, "$Revision$")
+
+#include "asterisk/file.h"
+#include "asterisk/logger.h"
+#include "asterisk/channel.h"
+#include "asterisk/pbx.h"
+#include "asterisk/module.h"
+#include "asterisk/lock.h"
+#include "asterisk/cli.h"
+#include "asterisk/options.h"
+#include "asterisk/app.h"
+#include "asterisk/linkedlists.h"
+
 #define get_volfactor(x) x ? ((x > 0) ? (1 << x) : ((1 << abs(x)) * -1)) : 0
-#define minmax(x,y) x ? (x > y) ? y : ((x < (y * -1)) ? (y * -1) : x) : 0 
-
-static char *tdesc = "Native Channel Monitoring Module";
-static char *app = "MuxMon";
-static char *synopsis = "Record A Call Natively";
-static char *desc = ""
-"  MuxMon(<file>.<ext>[|<options>[|<command>]])\n\n"
-"Records The audio on the current channel to the specified file.\n\n"
-"Valid Options:\n"
-" b    - Only save audio to the file while the channel is bridged. Note: does\n"
-"        not include conferences\n"
-" a    - Append to the file instead of overwriting it.\n"
+
+static const char *tdesc = "Mixed Audio Monitoring Application";
+static const char *app = "MixMonitor";
+static const char *synopsis = "Record a call and mix the audio during the recording";
+static const char *desc = ""
+"  MixMonitor(<file>.<ext>[|<options>[|<command>]])\n\n"
+"Records the audio on the current channel to the specified file.\n"
+"If the filename is an absolute path, uses that path, otherwise\n"
+"creates the file in the configured monitoring directory from\n"
+"asterisk.conf.\n\n"
+"Valid options:\n"
+" a      - Append to the file instead of overwriting it.\n"
+" b      - Only save audio to the file while the channel is bridged.\n"
+"          Note: does not include conferences.\n"
 " v(<x>) - Adjust the heard volume by a factor of <x> (range -4 to 4)\n"	
 " V(<x>) - Adjust the spoken volume by a factor of <x> (range -4 to 4)\n"	
 " W(<x>) - Adjust the both heard and spoken volumes by a factor of <x>\n"
@@ -48,14 +66,16 @@ static char *desc = ""
 "<command> will be executed when the recording is over\n"
 "Any strings matching ^{X} will be unescaped to ${X} and \n"
 "all variables will be evaluated at that time.\n"
-"The variable MUXMON_FILENAME will contain the filename used to record.\n"
+"The variable MIXMONITOR_FILENAME will contain the filename used to record.\n"
 "";
 
 STANDARD_LOCAL_USER;
 
 LOCAL_USER_DECL;
 
-struct muxmon {
+static const char *mixmonitor_spy_type = "MixMonitor";
+
+struct mixmonitor {
 	struct ast_channel *chan;
 	char *filename;
 	char *post_process;
@@ -64,445 +84,341 @@ struct muxmon {
 	int writevol;
 };
 
-typedef enum {
-    MUXFLAG_RUNNING = (1 << 0),
+enum {
     MUXFLAG_APPEND = (1 << 1),
     MUXFLAG_BRIDGED = (1 << 2),
     MUXFLAG_VOLUME = (1 << 3),
     MUXFLAG_READVOLUME = (1 << 4),
-    MUXFLAG_WRITEVOLUME = (1 << 5)
-} muxflags;
-
+    MUXFLAG_WRITEVOLUME = (1 << 5),
+} mixmonitor_flags;
 
-AST_DECLARE_OPTIONS(muxmon_opts,{
-    ['a'] = { MUXFLAG_APPEND },
+AST_DECLARE_OPTIONS(mixmonitor_opts,{
+	['a'] = { MUXFLAG_APPEND },
 	['b'] = { MUXFLAG_BRIDGED },
 	['v'] = { MUXFLAG_READVOLUME, 1 },
 	['V'] = { MUXFLAG_WRITEVOLUME, 2 },
 	['W'] = { MUXFLAG_VOLUME, 3 },
 });
 
-
 static void stopmon(struct ast_channel *chan, struct ast_channel_spy *spy) 
 {
-	struct ast_channel_spy *cptr=NULL, *prev=NULL;
-	int count = 0;
-
-	if (chan) {
-		while(ast_mutex_trylock(&chan->lock)) {
-			if (chan->spiers == spy) {
-				chan->spiers = NULL;
-				return;
-			}
-			count++;
-			if (count > 10) {
-				return;
-			}
-			sched_yield();
-		}
-		
-		for(cptr=chan->spiers; cptr; cptr=cptr->next) {
-			if (cptr == spy) {
-				if (prev) {
-					prev->next = cptr->next;
-					cptr->next = NULL;
-				} else
-					chan->spiers = NULL;
-			}
-			prev = cptr;
-		}
+	/* If our status has changed, then the channel we're spying on is gone....
+	   DON'T TOUCH IT!!!  RUN AWAY!!! */
+	if (spy->status != CHANSPY_RUNNING)
+		return;
 
-		ast_mutex_unlock(&chan->lock);
-	}
+	if (!chan)
+		return;
+
+	ast_mutex_lock(&chan->lock);
+	ast_channel_spy_remove(chan, spy);
+	ast_mutex_unlock(&chan->lock);
 }
 
-static void startmon(struct ast_channel *chan, struct ast_channel_spy *spy) 
+static int startmon(struct ast_channel *chan, struct ast_channel_spy *spy) 
 {
-
-	struct ast_channel_spy *cptr=NULL;
 	struct ast_channel *peer;
+	int res;
 
-	if (chan) {
-		ast_mutex_lock(&chan->lock);
-		if (chan->spiers) {
-			for(cptr=chan->spiers;cptr->next;cptr=cptr->next);
-			cptr->next = spy;
-		} else {
-			chan->spiers = spy;
-		}
-		ast_mutex_unlock(&chan->lock);
+	if (!chan)
+		return -1;
+
+	ast_mutex_lock(&chan->lock);
+	res = ast_channel_spy_add(chan, spy);
+	ast_mutex_unlock(&chan->lock);
 		
-		if (ast_test_flag(chan, AST_FLAG_NBRIDGE) && (peer = ast_bridged_channel(chan))) {
-			ast_softhangup(peer, AST_SOFTHANGUP_UNBRIDGE);	
-		}
-	}
-}
+	if (!res && ast_test_flag(chan, AST_FLAG_NBRIDGE) && (peer = ast_bridged_channel(chan)))
+		ast_softhangup(peer, AST_SOFTHANGUP_UNBRIDGE);	
 
-static int spy_queue_translate(struct ast_channel_spy *spy,
-							   struct ast_slinfactory *slinfactory0,
-							   struct ast_slinfactory *slinfactory1)
-{
-	int res = 0;
-	struct ast_frame *f;
-	
-	ast_mutex_lock(&spy->lock);
-	while((f = spy->queue[0])) {
-		spy->queue[0] = f->next;
-		ast_slinfactory_feed(slinfactory0, f);
-		ast_frfree(f);
-	}
-	ast_mutex_unlock(&spy->lock);
-	ast_mutex_lock(&spy->lock);
-	while((f = spy->queue[1])) {
-		spy->queue[1] = f->next;
-		ast_slinfactory_feed(slinfactory1, f);
-		ast_frfree(f);
-	}
-	ast_mutex_unlock(&spy->lock);
 	return res;
 }
 
-static void *muxmon_thread(void *obj) 
-{
+#define SAMPLES_PER_FRAME 160
 
-	int len0 = 0, len1 = 0, samp0 = 0, samp1 = 0, framelen, maxsamp = 0, x = 0;
-	short buf0[1280], buf1[1280], buf[1280];
-	struct ast_frame frame;
-	struct muxmon *muxmon = obj;
+static void *mixmonitor_thread(void *obj) 
+{
+	struct mixmonitor *mixmonitor = obj;
 	struct ast_channel_spy spy;
 	struct ast_filestream *fs = NULL;
 	char *ext, *name;
 	unsigned int oflags;
-	struct ast_slinfactory slinfactory[2];
+	struct ast_frame *f;
 	char post_process[1024] = "";
 	
-	name = ast_strdupa(muxmon->chan->name);
+	STANDARD_INCREMENT_USECOUNT;
 
-	framelen = 320;
-	frame.frametype = AST_FRAME_VOICE;
-	frame.subclass = AST_FORMAT_SLINEAR;
-	frame.data = buf;
-	ast_set_flag(muxmon, MUXFLAG_RUNNING);
-	oflags = O_CREAT|O_WRONLY;
-	ast_slinfactory_init(&slinfactory[0]);
-	ast_slinfactory_init(&slinfactory[1]);
-	
+	name = ast_strdupa(mixmonitor->chan->name);
 
-
-	/* for efficiency, use a flag to bypass volume logic when it's not needed */
-	if (muxmon->readvol || muxmon->writevol) {
-		ast_set_flag(muxmon, MUXFLAG_VOLUME);
-	}
-
-	if ((ext = strchr(muxmon->filename, '.'))) {
+	oflags = O_CREAT|O_WRONLY;
+	oflags |= ast_test_flag(mixmonitor, MUXFLAG_APPEND) ? O_APPEND : O_TRUNC;
+		
+	if ((ext = strchr(mixmonitor->filename, '.'))) {
 		*(ext++) = '\0';
 	} else {
 		ext = "raw";
 	}
 
+	fs = ast_writefile(mixmonitor->filename, ext, NULL, oflags, 0, 0644);
+	if (!fs) {
+		ast_log(LOG_ERROR, "Cannot open %s.%s\n", mixmonitor->filename, ext);
+		goto out;
+	}
+
+	if (ast_test_flag(mixmonitor, MUXFLAG_APPEND))
+		ast_seekstream(fs, 0, SEEK_END);
+	
 	memset(&spy, 0, sizeof(spy));
+	ast_set_flag(&spy, CHANSPY_FORMAT_AUDIO);
+	ast_set_flag(&spy, CHANSPY_MIXAUDIO);
+	spy.type = mixmonitor_spy_type;
 	spy.status = CHANSPY_RUNNING;
+	spy.read_queue.format = AST_FORMAT_SLINEAR;
+	spy.write_queue.format = AST_FORMAT_SLINEAR;
+	if (mixmonitor->readvol) {
+		ast_set_flag(&spy, CHANSPY_READ_VOLADJUST);
+		spy.read_vol_adjustment = mixmonitor->readvol;
+	}
+	if (mixmonitor->writevol) {
+		ast_set_flag(&spy, CHANSPY_WRITE_VOLADJUST);
+		spy.write_vol_adjustment = mixmonitor->writevol;
+	}
 	ast_mutex_init(&spy.lock);
-	startmon(muxmon->chan, &spy);
-	if (ast_test_flag(muxmon, MUXFLAG_RUNNING)) {
-		if (option_verbose > 1) {
-			ast_verbose(VERBOSE_PREFIX_2 "Begin Muxmon Recording %s\n", name);
-		}
 
-		oflags |= ast_test_flag(muxmon, MUXFLAG_APPEND) ? O_APPEND : O_TRUNC;
-		
-		if (!(fs = ast_writefile(muxmon->filename, ext, NULL, oflags, 0, 0644))) {
-			ast_log(LOG_ERROR, "Cannot open %s\n", muxmon->filename);
-			spy.status = CHANSPY_DONE;
-		}  else {
+	if (startmon(mixmonitor->chan, &spy)) {
+		ast_log(LOG_WARNING, "Unable to add '%s' spy to channel '%s'\n",
+			spy.type, mixmonitor->chan->name);
+		goto out2;
+	}
 
-			if (ast_test_flag(muxmon, MUXFLAG_APPEND)) {
-				ast_seekstream(fs, 0, SEEK_END);
-			}
+	if (option_verbose > 1)
+		ast_verbose(VERBOSE_PREFIX_2 "Begin MixMonitor Recording %s\n", name);
+	
+	while (1) {
+		struct ast_frame *next;
+		int write;
+
+		ast_mutex_lock(&spy.lock);
 
-			while (ast_test_flag(muxmon, MUXFLAG_RUNNING)) {
-				samp0 = samp1 = len0 = len1 = 0;
-
-				if (ast_check_hangup(muxmon->chan) || spy.status != CHANSPY_RUNNING) {
-					ast_clear_flag(muxmon, MUXFLAG_RUNNING);
-					break;
-				}
-
-				if (ast_test_flag(muxmon, MUXFLAG_BRIDGED) && !ast_bridged_channel(muxmon->chan)) {
-					usleep(1000);
-					sched_yield();
-					continue;
-				}
-				
-				spy_queue_translate(&spy, &slinfactory[0], &slinfactory[1]);
-				
-				if (slinfactory[0].size < framelen || slinfactory[1].size < framelen) {
-					usleep(1000);
-					sched_yield();
-					continue;
-				}
-
-				if ((len0 = ast_slinfactory_read(&slinfactory[0], buf0, framelen))) {
-					samp0 = len0 / 2;
-				}
-				if((len1 = ast_slinfactory_read(&slinfactory[1], buf1, framelen))) {
-					samp1 = len1 / 2;
-				}
-				
-				if (ast_test_flag(muxmon, MUXFLAG_VOLUME)) {
-					if (samp0 && muxmon->readvol > 0) {
-						for(x=0; x < samp0 / 2; x++) {
-							buf0[x] *= muxmon->readvol;
-						}
-					} else if (samp0 && muxmon->readvol < 0) {
-						for(x=0; x < samp0 / 2; x++) {
-							buf0[x] /= muxmon->readvol;
-						}
-					}
-					if (samp1 && muxmon->writevol > 0) {
-						for(x=0; x < samp1 / 2; x++) {
-							buf1[x] *= muxmon->writevol;
-						}
-					} else if (muxmon->writevol < 0) {
-						for(x=0; x < samp1 / 2; x++) {
-							buf1[x] /= muxmon->writevol;
-						}
-					}
-				}
-				
-				maxsamp = (samp0 > samp1) ? samp0 : samp1;
-
-				if (samp0 && samp1) {
-					for(x=0; x < maxsamp; x++) {
-						if (x < samp0 && x < samp1) {
-							buf[x] = buf0[x] + buf1[x];
-						} else if (x < samp0) {
-							buf[x] = buf0[x];
-						} else if (x < samp1) {
-							buf[x] = buf1[x];
-						}
-					}
-				} else if(samp0) {
-					memcpy(buf, buf0, len0);
-					x = samp0;
-				} else if(samp1) {
-					memcpy(buf, buf1, len1);
-					x = samp1;
-				}
-
-				frame.samples = x;
-				frame.datalen = x * 2;
-				ast_writestream(fs, &frame);
+		ast_channel_spy_trigger_wait(&spy);
 		
-				usleep(1000);
-				sched_yield();
+		if (ast_check_hangup(mixmonitor->chan) || spy.status != CHANSPY_RUNNING) {
+			ast_mutex_unlock(&spy.lock);
+			break;
+		}
+		
+		while (1) {
+			if (!(f = ast_channel_spy_read_frame(&spy, SAMPLES_PER_FRAME)))
+				break;
+
+			write = (!ast_test_flag(mixmonitor, MUXFLAG_BRIDGED) ||
+				 ast_bridged_channel(mixmonitor->chan));
+
+			/* it is possible for ast_channel_spy_read_frame() to return a chain
+			   of frames if a queue flush was necessary, so process them
+			*/
+			for (; f; f = next) {
+				next = f->next;
+				if (write)
+					ast_writestream(fs, f);
+				ast_frfree(f);
 			}
 		}
-	}
 
-	if (muxmon->post_process) {
+		ast_mutex_unlock(&spy.lock);
+	}
+	
+	if (mixmonitor->post_process) {
 		char *p;
-		for(p = muxmon->post_process; *p ; p++) {
+
+		for (p = mixmonitor->post_process; *p ; p++) {
 			if (*p == '^' && *(p+1) == '{') {
 				*p = '$';
 			}
 		}
-		pbx_substitute_variables_helper(muxmon->chan, muxmon->post_process, post_process, sizeof(post_process) - 1);
-		free(muxmon->post_process);
-		muxmon->post_process = NULL;
+		pbx_substitute_variables_helper(mixmonitor->chan, mixmonitor->post_process, post_process, sizeof(post_process) - 1);
 	}
 
-	stopmon(muxmon->chan, &spy);
-	if (option_verbose > 1) {
-		ast_verbose(VERBOSE_PREFIX_2 "Finished Recording %s\n", name);
-	}
-	ast_mutex_destroy(&spy.lock);
-	
-	if(fs) {
-		ast_closestream(fs);
-	}
-	
-	ast_slinfactory_destroy(&slinfactory[0]);
-	ast_slinfactory_destroy(&slinfactory[1]);
+	stopmon(mixmonitor->chan, &spy);
 
-	if (muxmon) {
-		if (muxmon->filename) {
-			free(muxmon->filename);
-		}
-		free(muxmon);
-	}
+	if (option_verbose > 1)
+		ast_verbose(VERBOSE_PREFIX_2 "End MixMonitor Recording %s\n", name);
 
 	if (!ast_strlen_zero(post_process)) {
-		if (option_verbose > 2) {
+		if (option_verbose > 2)
 			ast_verbose(VERBOSE_PREFIX_2 "Executing [%s]\n", post_process);
-		}
 		ast_safe_system(post_process);
 	}
 
+out2:
+	ast_mutex_destroy(&spy.lock);
+
+	if (fs)
+		ast_closestream(fs);
+
+out:
+	free(mixmonitor);
+
+	STANDARD_DECREMENT_USECOUNT;
+
 	return NULL;
 }
 
-static void launch_monitor_thread(struct ast_channel *chan, char *filename, unsigned int flags, int readvol , int writevol, char *post_process) 
+static void launch_monitor_thread(struct ast_channel *chan, const char *filename, unsigned int flags,
+				  int readvol, int writevol, const char *post_process) 
 {
 	pthread_attr_t attr;
-	int result = 0;
 	pthread_t thread;
-	struct muxmon *muxmon;
+	struct mixmonitor *mixmonitor;
+	int len;
 
+	len = sizeof(*mixmonitor) + strlen(filename) + 1;
+	if (post_process && !ast_strlen_zero(post_process))
+		len += strlen(post_process) + 1;
 
-	if (!(muxmon = malloc(sizeof(struct muxmon)))) {
+	if (!(mixmonitor = calloc(1, len))) {
 		ast_log(LOG_ERROR, "Memory Error!\n");
 		return;
 	}
 
-	memset(muxmon, 0, sizeof(struct muxmon));
-	muxmon->chan = chan;
-	muxmon->filename = strdup(filename);
-	if(post_process) {
-		muxmon->post_process = strdup(post_process);
+	mixmonitor->chan = chan;
+	mixmonitor->filename = (char *) mixmonitor + sizeof(*mixmonitor);
+	strcpy(mixmonitor->filename, filename);
+	if (post_process && !ast_strlen_zero(post_process)) {
+		mixmonitor->post_process = mixmonitor->filename + strlen(filename) + 1;
+		strcpy(mixmonitor->post_process, post_process);
 	}
-	muxmon->readvol = readvol;
-	muxmon->writevol = writevol;
-	muxmon->flags = flags;
+	mixmonitor->readvol = readvol;
+	mixmonitor->writevol = writevol;
+	mixmonitor->flags = flags;
 
-	result = pthread_attr_init(&attr);
-	pthread_attr_setschedpolicy(&attr, SCHED_RR);
+	pthread_attr_init(&attr);
 	pthread_attr_setdetachstate(&attr, PTHREAD_CREATE_DETACHED);
-	result = ast_pthread_create(&thread, &attr, muxmon_thread, muxmon);
-	result = pthread_attr_destroy(&attr);
+	ast_pthread_create(&thread, &attr, mixmonitor_thread, mixmonitor);
+	pthread_attr_destroy(&attr);
 }
 
-
-static int muxmon_exec(struct ast_channel *chan, void *data)
+static int mixmonitor_exec(struct ast_channel *chan, void *data)
 {
-	int res = 0, x = 0, readvol = 0, writevol = 0;
+	int x, readvol = 0, writevol = 0;
 	struct localuser *u;
 	struct ast_flags flags = {0};
-	int argc;
-	char *options = NULL,
-		*args,
-		*argv[3],
-		*filename = NULL,
-		*post_process = NULL;
+	char *parse;
+	AST_DECLARE_APP_ARGS(args,
+		AST_APP_ARG(filename);
+		AST_APP_ARG(options);
+		AST_APP_ARG(post_process);
+	);
 	
 	if (ast_strlen_zero(data)) {
-		ast_log(LOG_WARNING, "muxmon requires an argument\n");
+		ast_log(LOG_WARNING, "MixMonitor requires an argument (filename)\n");
 		return -1;
 	}
 
 	LOCAL_USER_ADD(u);
 
-	args = ast_strdupa(data);	
-	if (!args) {
+	if (!(parse = ast_strdupa(data))) {
 		ast_log(LOG_WARNING, "Memory Error!\n");
 		LOCAL_USER_REMOVE(u);
 		return -1;
 	}
 
-	if ((argc = ast_separate_app_args(args, '|', argv, sizeof(argv) / sizeof(argv[0])))) {
-		filename = argv[0];
-		if (argc > 1) {
-			options = argv[1];
-		}
-		if (argc > 2) {
-			post_process = argv[2];
-		}
-	}
+	AST_STANDARD_APP_ARGS(args, parse);
 	
-	if (ast_strlen_zero(filename)) {
+	if (ast_strlen_zero(args.filename)) {
 		ast_log(LOG_WARNING, "Muxmon requires an argument (filename)\n");
 		LOCAL_USER_REMOVE(u);
 		return -1;
 	}
 
-	if (options) {
-		char *opts[3] = {};
-		ast_parseoptions(muxmon_opts, &flags, opts, options);
-
-		if (ast_test_flag(&flags, MUXFLAG_READVOLUME) && opts[0]) {
-			if (sscanf(opts[0], "%d", &x) != 1)
-				ast_log(LOG_NOTICE, "volume must be a number between -4 and 4\n");
-			else {
-				readvol = minmax(x, 4);
-				x = get_volfactor(readvol);
-				readvol = minmax(x, 16);
+	if (args.options) {
+		char *opts[3] = { NULL, };
+
+		ast_parseoptions(mixmonitor_opts, &flags, opts, args.options);
+
+		if (ast_test_flag(&flags, MUXFLAG_READVOLUME)) {
+			if (!opts[0] || ast_strlen_zero(opts[0])) {
+				ast_log(LOG_WARNING, "No volume level was provided for the heard volume ('v') option.\n");
+			} else if ((sscanf(opts[0], "%d", &x) != 1) || (x < -4) || (x > 4)) {
+				ast_log(LOG_NOTICE, "Heard volume must be a number between -4 and 4, not '%s'\n", opts[0]);
+			} else {
+				readvol = get_volfactor(x);
 			}
 		}
 		
-		if (ast_test_flag(&flags, MUXFLAG_WRITEVOLUME) && opts[1]) {
-			if (sscanf(opts[1], "%d", &x) != 1)
-				ast_log(LOG_NOTICE, "volume must be a number between -4 and 4\n");
-			else {
-				writevol = minmax(x, 4);
-				x = get_volfactor(writevol);
-				writevol = minmax(x, 16);
+		if (ast_test_flag(&flags, MUXFLAG_WRITEVOLUME)) {
+			if (!opts[1] || ast_strlen_zero(opts[1])) {
+				ast_log(LOG_WARNING, "No volume level was provided for the spoken volume ('V') option.\n");
+			} else if ((sscanf(opts[1], "%d", &x) != 1) || (x < -4) || (x > 4)) {
+				ast_log(LOG_NOTICE, "Spoken volume must be a number between -4 and 4, not '%s'\n", opts[1]);
+			} else {
+				writevol = get_volfactor(x);
 			}
 		}
-
-		if (ast_test_flag(&flags, MUXFLAG_VOLUME) && opts[2]) {
-			if (sscanf(opts[2], "%d", &x) != 1)
-				ast_log(LOG_NOTICE, "volume must be a number between -4 and 4\n");
-			else {
-				readvol = writevol = minmax(x, 4);
-				x = get_volfactor(readvol);
-				readvol = minmax(x, 16);
-				x = get_volfactor(writevol);
-				writevol = minmax(x, 16);
+		
+		if (ast_test_flag(&flags, MUXFLAG_VOLUME)) {
+			if (!opts[2] || ast_strlen_zero(opts[2])) {
+				ast_log(LOG_WARNING, "No volume level was provided for the combined volume ('W') option.\n");
+			} else if ((sscanf(opts[2], "%d", &x) != 1) || (x < -4) || (x > 4)) {
+				ast_log(LOG_NOTICE, "Combined volume must be a number between -4 and 4, not '%s'\n", opts[2]);
+			} else {
+				readvol = writevol = get_volfactor(x);
 			}
 		}
 	}
-	pbx_builtin_setvar_helper(chan, "MUXMON_FILENAME", filename);
-	launch_monitor_thread(chan, filename, flags.flags, readvol, writevol, post_process);
+
+	/* if not provided an absolute path, use the system-configured monitoring directory */
+	if (args.filename[0] != '/') {
+		char *build;
+
+		build = alloca(strlen(ast_config_AST_MONITOR_DIR) + strlen(args.filename) + 3);
+		sprintf(build, "%s/%s", ast_config_AST_MONITOR_DIR, args.filename);
+		args.filename = build;
+	}
+
+	pbx_builtin_setvar_helper(chan, "MIXMONITOR_FILENAME", args.filename);
+	launch_monitor_thread(chan, args.filename, flags.flags, readvol, writevol, args.post_process);
 
 	LOCAL_USER_REMOVE(u);
-	return res;
-}
 
+	return 0;
+}
 
-static int muxmon_cli(int fd, int argc, char **argv) 
+static int mixmonitor_cli(int fd, int argc, char **argv) 
 {
-	char *op, *chan_name = NULL, *args = NULL;
 	struct ast_channel *chan;
 
-	if (argc > 2) {
-		op = argv[1];
-		chan_name = argv[2];
-
-		if (argv[3]) {
-			args = argv[3];
-		}
+	if (argc < 3)
+		return RESULT_SHOWUSAGE;
 
-		if (!(chan = ast_get_channel_by_name_prefix_locked(chan_name, strlen(chan_name)))) {
-			ast_cli(fd, "Invalid Channel!\n");
-			return -1;
-		}
-		if (!strcasecmp(op, "start")) {
-			muxmon_exec(chan, args);
-		} else if (!strcasecmp(op, "stop")) {
-			struct ast_channel_spy *cptr=NULL;
-			for(cptr=chan->spiers; cptr; cptr=cptr->next) {
-				cptr->status = CHANSPY_DONE;
-			}
-		}
-		ast_mutex_unlock(&chan->lock);
-		return 0;
+	if (!(chan = ast_get_channel_by_name_prefix_locked(argv[2], strlen(argv[2])))) {
+		ast_cli(fd, "No channel matching '%s' found.\n", argv[2]);
+		return RESULT_SUCCESS;
 	}
 
-	ast_cli(fd, "Usage: muxmon <start|stop> <chan_name> <args>\n");
-	return -1;
+	if (!strcasecmp(argv[1], "start"))
+		mixmonitor_exec(chan, argv[3]);
+	else if (!strcasecmp(argv[1], "stop"))
+		ast_channel_spy_stop_by_type(chan, mixmonitor_spy_type);
+
+	ast_mutex_unlock(&chan->lock);
+
+	return RESULT_SUCCESS;
 }
 
 
-static struct ast_cli_entry cli_muxmon = {
-	{ "muxmon", NULL, NULL }, muxmon_cli, 
-	"Execute a monitor command", "muxmon <start|stop> <chan_name> <args>"};
+static struct ast_cli_entry cli_mixmonitor = {
+	{ "mixmonitor", NULL, NULL },
+	mixmonitor_cli, 
+	"Execute a MixMonitor command",
+	"mixmonitor <start|stop> <chan_name> [<args>]"
+};
 
 
 int unload_module(void)
 {
 	int res;
 
-	res = ast_cli_unregister(&cli_muxmon);
+	res = ast_cli_unregister(&cli_mixmonitor);
 	res |= ast_unregister_application(app);
 	
 	STANDARD_HANGUP_LOCALUSERS;
@@ -514,21 +430,23 @@ int load_module(void)
 {
 	int res;
 
-	res = ast_cli_register(&cli_muxmon);
-	res |= ast_register_application(app, muxmon_exec, synopsis, desc);
+	res = ast_cli_register(&cli_mixmonitor);
+	res |= ast_register_application(app, mixmonitor_exec, synopsis, desc);
 
 	return res;
 }
 
 char *description(void)
 {
-	return tdesc;
+	return (char *) tdesc;
 }
 
 int usecount(void)
 {
 	int res;
+
 	STANDARD_USECOUNT(res);
+
 	return res;
 }
 
@@ -536,4 +454,3 @@ char *key()
 {
 	return ASTERISK_GPL_KEY;
 }
-