[svn-commits] seanbright: branch 1.6.1 r1085 - /branches/1.6.1/formats/format_mp3.c

SVN commits to the Digium repositories svn-commits at lists.digium.com
Wed Jan 6 09:38:36 CST 2010


Author: seanbright
Date: Wed Jan  6 09:38:26 2010
New Revision: 1085

URL: http://svnview.digium.com/svn/asterisk-addons?view=rev&rev=1085
Log:
Fix reading samples from format_mp3 after ast_seekstream/ast_tellstream.

There is a bug when using ast_seekstream/ast_tellstream with format_mp3 in that
the file read position is not reset before attempting to read samples.  So when
we seek to determine the maximum size of the file (as in res_agi's STREAM FILE)
we weren't then resetting the file pointer so that we could properly read
samples.  This patch addresses that (in a similar manner to format_wav.c).

(closes issue #15224)
Reported by: rbd
Patches:
      20091230_addons_1.4_issue15224.diff uploaded by seanbright (license 71)
Tested by: rbd, seanbright

Review: https://reviewboard.asterisk.org/r/453

Modified:
    branches/1.6.1/formats/format_mp3.c

Modified: branches/1.6.1/formats/format_mp3.c
URL: http://svnview.digium.com/svn/asterisk-addons/branches/1.6.1/formats/format_mp3.c?view=diff&rev=1085&r1=1084&r2=1085
==============================================================================
--- branches/1.6.1/formats/format_mp3.c (original)
+++ branches/1.6.1/formats/format_mp3.c Wed Jan  6 09:38:26 2010
@@ -260,7 +260,7 @@
 	}
 
 	p->seek = offset;
-	return p->seek;
+	return fseek(s->f, offset, SEEK_SET);
 	
 }
 




More information about the svn-commits mailing list