[asterisk-commits] dvossel: branch 1.4 r188287 - /branches/1.4/main/audiohook.c

[SVN commits to the Asterisk project]

Author: dvossel
Date: Tue Apr 14 10:02:04 2009
New Revision: 188287

URL: http://svn.digium.com/svn-view/asterisk?view=rev&rev=188287
Log:
audio_audiohook_write_list() does not correctly update sample size after ast_translate.

audio_audiohook_write_list() does not take into account that the sample size may change after translation depending on if the original frame is is 8khz or 16khz.  While no 16kz codecs are supported in 1.4 at the moment, this will save headaches in the future if they ever are.  the sample size is now updated after translating to reflect this possibility.  Thanks to jcolp and mmichelson for helping me work this out.

(issue AST-197)


Modified:
    branches/1.4/main/audiohook.c

Modified: branches/1.4/main/audiohook.c
URL: http://svn.digium.com/svn-view/asterisk/branches/1.4/main/audiohook.c?view=diff&rev=188287&r1=188286&r2=188287
==============================================================================
--- branches/1.4/main/audiohook.c (original)
+++ branches/1.4/main/audiohook.c Tue Apr 14 10:02:04 2009
@@ -588,6 +588,7 @@
 		}
 		if (!(middle_frame = ast_translate(in_translate->trans_pvt, frame, 0)))
 			return frame;
+		samples = middle_frame->samples;
 	}
 
 	/* Queue up signed linear frame to each spy */