Jean-Marc Valin jean-marc.valin at usherbrooke.ca
Tue Jul 22 12:11:50 PDT 2008

> The implementation of SPEEX_PREPROCESS_GET_ECHO_STATE in preprocess.c
> seems to be broken. The value of the echo_state pointer is assigned
> to the stack variable rather than to the memory location pointed to
> by the stack variable.
> case SPEEX_PREPROCESS_GET_ECHO_STATE: ptr = (void*)st->echo_state; 
> break;
> It seems as though the code should be:
> case SPEEX_PREPROCESS_GET_ECHO_STATE: (*(SpeexEchoState*)ptr) =
> (SpeexEchoState*)st->echo_state; break;

Sorry about that. I'll fix it ASAP.

> PS - Sorry if this has already been received. I sent this yesterday
> before realizing that I hadn't subscribed to the mailing list.

No worries. I'll remove it from the mod queue. Note that if you post
when not subscribed, it just takes a few days before I approve it and
post it. The idea isn't to prevent non-members from posting, just to
control spam.


More information about the Speex-dev mailing list