eden-emu/opus
19
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Limit mode creation to supported modes.

Browse source 
We did no real error checking to see if a mode is supported when it
 is created.
This patch implements checks for Jean-Marc's rules:
1) A mode must have frames at least 1ms in length (no more than
    1000 per second).
2) A mode must have shorts of at most 3.33 ms (at least 300 per
    second).
It also adds error checking to dump_modes so we report the error
 instead of crashing when we fail to create a mode.
This commit is contained in:
Timothy B. Terriberry 2011-02-01 15:36:59 -08:00 • committed by Jean-Marc Valin
parent 7e983194a3
commit aa6fec669e
2 changed files with 36 additions and 15 deletions
Download patch file Download diff file Expand all files Collapse all files

7
libcelt/dump_modes.c
View file

@ -34,6 +34,7 @@
#include "config.h" #include "config.h"
#endif #endif
#include <stdlib.h>
#include <stdio.h> #include <stdio.h>
#include "modes.h" #include "modes.h"
#include "celt.h" #include "celt.h"
@ -308,6 +309,12 @@ int main(int argc, char **argv)
Fs = atoi(argv[2*i+1]); Fs = atoi(argv[2*i+1]);
frame = atoi(argv[2*i+2]); frame = atoi(argv[2*i+2]);
m[i] = celt_mode_create(Fs, frame, NULL); m[i] = celt_mode_create(Fs, frame, NULL);
if (m[i]==NULL)
{
fprintf(stderr,"Error creating mode with Fs=%s, frame_size=%s\n",
argv[2*i+1],argv[2*i+2]);
return EXIT_FAILURE;
}
} }
file = fopen(BASENAME ".c", "w"); file = fopen(BASENAME ".c", "w");
dump_modes(file, m, nb); dump_modes(file, m, nb);

42
libcelt/modes.c
View file

@ -331,6 +331,34 @@ CELTMode *celt_mode_create(celt_int32 Fs, int frame_size, int *error)
*error = CELT_BAD_ARG; *error = CELT_BAD_ARG;
return NULL; return NULL;
} }
/* Frames of less than 1ms are not supported. */
if ((celt_int32)frame_size*1000 < Fs)
{
if (error)
*error = CELT_INVALID_MODE;
return NULL;
}
if ((celt_int32)frame_size*75 >= Fs && (frame_size%16)==0)
{
LM = 3;
} else if ((celt_int32)frame_size*150 >= Fs && (frame_size%8)==0)
{
LM = 2;
} else if ((celt_int32)frame_size*300 >= Fs && (frame_size%4)==0)
{
LM = 1;
} else if ((celt_int32)frame_size*300 <= Fs)
{
LM = 0;
}
/* Shorts longer than 3.3ms are not supported. */
else
{
if (error)
*error = CELT_INVALID_MODE;
return NULL;
}
mode = celt_alloc(sizeof(CELTMode)); mode = celt_alloc(sizeof(CELTMode));
if (mode==NULL) if (mode==NULL)
@ -366,20 +394,6 @@ CELTMode *celt_mode_create(celt_int32 Fs, int frame_size, int *error)
mode->preemph[3] = QCONST16(1.f, 13); mode->preemph[3] = QCONST16(1.f, 13);
} }
if ((celt_int32)frame_size*75 >= Fs && (frame_size%16)==0)
{
LM = 3;
} else if ((celt_int32)frame_size*150 >= Fs && (frame_size%8)==0)
{
LM = 2;
} else if ((celt_int32)frame_size*300 >= Fs && (frame_size%4)==0)
{
LM = 1;
} else
{
LM = 0;
}
mode->maxLM = LM; mode->maxLM = LM;
mode->nbShortMdcts = 1<<LM; mode->nbShortMdcts = 1<<LM;
mode->shortMdctSize = frame_size/mode->nbShortMdcts; mode->shortMdctSize = frame_size/mode->nbShortMdcts;