Cyril Lashkevich
2012-Aug-06 13:38 UTC
[opus] [PATCH] Fixed sizeof param in silk_get_TOC. Size for memset was calculated wrong.
--- silk/dec_API.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/silk/dec_API.c b/silk/dec_API.c index b5acb6b..1c356c7 100644 --- a/silk/dec_API.c +++ b/silk/dec_API.c @@ -357,7 +357,7 @@ opus_int silk_get_TOC( return -1; } - silk_memset( Silk_TOC, 0, sizeof( Silk_TOC ) ); + silk_memset( Silk_TOC, 0, sizeof( silk_TOC_struct ) ); /* For stereo, extract the flags for the mid channel */ flags = silk_RSHIFT( payload[ 0 ], 7 - nFramesPerPayload ) & ( silk_LSHIFT( 1, nFramesPerPayload + 1 ) - 1 ); -- 1.7.9.6 (Apple Git-31.1)
Gregory Maxwell
2012-Aug-06 13:56 UTC
[opus] [PATCH] Fixed sizeof param in silk_get_TOC. Size for memset was calculated wrong.
On Mon, Aug 6, 2012 at 9:38 AM, Cyril Lashkevich <notorca at gmail.com> wrote:> @@ -357,7 +357,7 @@ opus_int silk_get_TOC(Thanks. Are you using silk_get_TOC at all? The code is dead, unreachable, and doesn't work right without at least doing some pre-processing. I've wanted to remove it for a long time... but not if someone is actually using it.