diff options
| author | William Wilgus <me.theuser@yahoo.com> | 2019-08-12 22:57:37 -0500 |
|---|---|---|
| committer | William Wilgus <me.theuser@yahoo.com> | 2019-08-14 17:54:35 +0200 |
| commit | 0ff6a31d7dc51ff2f0a6147a0fadff2d6359ecb9 (patch) | |
| tree | a29f97d44d576825669f69f6340e9234f1d64c27 /apps/codec_thread.c | |
| parent | 8570d4063b2c63482ceb986a2735f247e736c010 (diff) | |
opus reset decoder on seek completion to prevent stack overflow
apparently we should be doing this anyway
mark4o> The packets overlap and may reuse state set by other recent packets,
so if you seek to a different position,
resetting the state helps to ensure that the subsequent
packets won't use the state set by the unrelated packets
that were processed before the seek.
remove stack bump WORKAROUND_FS13060
Change-Id: I1c14e23b1721a360b91e3e55202c1557aef0fcc6
Diffstat (limited to 'apps/codec_thread.c')
| -rw-r--r-- | apps/codec_thread.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/apps/codec_thread.c b/apps/codec_thread.c index 4eb85c8d71..807c0b6e8e 100644 --- a/apps/codec_thread.c +++ b/apps/codec_thread.c @@ -92,7 +92,8 @@ static struct event_queue codec_queue SHAREDBSS_ATTR; static struct queue_sender_list codec_queue_sender_list SHAREDBSS_ATTR; /* Workaround stack overflow in opus codec on highmem devices (see FS#13060). */ -#if !defined(CPU_COLDFIRE) && (MEMORYSIZE >= 8) && defined(IRAMSIZE) && IRAMSIZE > (32 * 1024) +/* Fixed 2019-8-14 (see FS#13131) */ +#if 0 /*!defined(CPU_COLDFIRE) && (MEMORYSIZE >= 8) && defined(IRAMSIZE) && IRAMSIZE > (32 * 1024)*/ #define WORKAROUND_FS13060 0x800 #else #define WORKAROUND_FS13060 0 |