[asterisk-commits] qwell: trunk r108683 - in /trunk: ./ res/res_musiconhold.c

[SVN commits to the Asterisk project]

Author: qwell
Date: Fri Mar 14 09:32:55 2008
New Revision: 108683

URL: http://svn.digium.com/view/asterisk?view=rev&rev=108683
Log:
Merged revisions 108682 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r108682 | qwell | 2008-03-14 09:29:05 -0500 (Fri, 14 Mar 2008) | 4 lines

Fix a potential segfault if chan (or chan->music_state) is NULL.

Closes issue #12210, credit to edantie for pointing this out.

........

Modified:
    trunk/   (props changed)
    trunk/res/res_musiconhold.c

Propchange: trunk/
------------------------------------------------------------------------------
Binary property 'branch-1.4-merged' - no diff available.

Modified: trunk/res/res_musiconhold.c
URL: http://svn.digium.com/view/asterisk/trunk/res/res_musiconhold.c?view=diff&rev=108683&r1=108682&r2=108683
==============================================================================
--- trunk/res/res_musiconhold.c (original)
+++ trunk/res/res_musiconhold.c Fri Mar 14 09:32:55 2008
@@ -210,22 +210,25 @@
 
 static void moh_files_release(struct ast_channel *chan, void *data)
 {
-	struct moh_files_state *state = chan->music_state;
-
-	if (chan && state) {
-		if (chan->stream) {
-                        ast_closestream(chan->stream);
-                        chan->stream = NULL;
-                }
-		ast_verb(3, "Stopped music on hold on %s\n", chan->name);
-
-		if (state->origwfmt && ast_set_write_format(chan, state->origwfmt)) {
-			ast_log(LOG_WARNING, "Unable to restore channel '%s' to format '%d'\n", chan->name, state->origwfmt);
-		}
-		state->save_pos = state->pos;
-	}
-	if (ast_atomic_dec_and_test(&state->class->inuse) && state->class->delete)
-		ast_moh_destroy_one(state->class);
+	struct moh_files_state *state;
+
+	if (chan) {
+		if ((state = chan->music_state)) {
+			if (chan->stream) {
+	                        ast_closestream(chan->stream);
+	                        chan->stream = NULL;
+	                }
+			ast_verb(3, "Stopped music on hold on %s\n", chan->name);
+	
+			if (state->origwfmt && ast_set_write_format(chan, state->origwfmt)) {
+				ast_log(LOG_WARNING, "Unable to restore channel '%s' to format '%d'\n", chan->name, state->origwfmt);
+			}
+			state->save_pos = state->pos;
+
+			if (ast_atomic_dec_and_test(&state->class->inuse) && state->class->delete)
+				ast_moh_destroy_one(state->class);
+		}
+	}
 }