From fdb44349e938860651cedf4fe0ec7161305a06bd Mon Sep 17 00:00:00 2001
From: Joshua Colp <jcolp@digium.com>
Date: Wed, 27 Feb 2008 20:56:23 +0000
Subject: [PATCH] Don't loop around infinitely trying to spy on our own
 channel, and don't forget to free/detach the datastore upon hangup of the
 spy.

git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@104787 65c4cc65-6c06-0410-ace0-fbb531ad65f3
---
 apps/app_chanspy.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/apps/app_chanspy.c b/apps/app_chanspy.c
index e25389520e..a1e54ca440 100644
--- a/apps/app_chanspy.c
+++ b/apps/app_chanspy.c
@@ -462,6 +462,10 @@ redo:
 	if (!strncmp(this->name, "Zap/pseudo", 10)) {
 		ast_channel_unlock(this);
 		goto redo;
+	} else if (this == chan) {
+		last = this;
+		ast_channel_unlock(this);
+		goto redo;
 	}
 
 	return setup_chanspy_ds(this, chanspy_ds);
@@ -544,11 +548,6 @@ static int common_exec(struct ast_channel *chan, const struct ast_flags *flags,
 				break;
 			}
 
-			if (peer == chan) {
-				ast_channel_unlock(peer);
-				continue;
-			}
-
 			if (ast_test_flag(flags, OPTION_BRIDGED) && !ast_bridged_channel(peer)) {
 				ast_channel_unlock(peer);
 				continue;
@@ -612,6 +611,7 @@ static int common_exec(struct ast_channel *chan, const struct ast_flags *flags,
 			num_spyed_upon++;	
 
 			if (res == -1) {
+				chanspy_ds_free(peer_chanspy_ds);
 				break;
 			} else if (res > 1 && spec) {
 				struct ast_channel *next;