Hey there, inside source file 'pjmedia/src/test/mips_test.c' a variable initialization for 'status' in function wsola_discard_get_frame() is missing. This causes the returning of uninitialized data in some cases. As a consequence the test will fail. The following patch fixes the bug: diff --git a/pjmedia/src/test/mips_test.c b/pjmedia/src/test/mips_test.c index fc71d4f..ac1c6b6 100644 --- a/pjmedia/src/test/mips_test.c +++ b/pjmedia/src/test/mips_test.c @@ -1172,7 +1172,7 @@ static pj_status_t wsola_discard_get_frame(struct pjmedia_port *this_port, pjmedia_frame *frame) { struct wsola_discard_port *wp = (struct wsola_discard_port*)this_port; - pj_status_t status; + pj_status_t status = PJ_SUCCESS; while (pjmedia_circ_buf_get_len(wp->circbuf) < PJMEDIA_PIA_SPF(&wp->base.info) * (CIRC_BUF_FRAME_CNT-1)) Regards, Michael

Hi Michael, Thanks again for the report. I committed your patch in r5607 ( https://trac.pjsip.org/repos/changeset/5607). Best regards, Ming On Wed, Jun 14, 2017 at 6:09 PM, Michael Scheiffler <michael@scheiffler.at> wrote: > Hey there, > > inside source file 'pjmedia/src/test/mips_test.c' a variable > initialization for 'status' in function wsola_discard_get_frame() is > missing. > This causes the returning of uninitialized data in some cases. As a > consequence the test will fail. > > The following patch fixes the bug: > > diff --git a/pjmedia/src/test/mips_test.c b/pjmedia/src/test/mips_test.c > index fc71d4f..ac1c6b6 100644 > --- a/pjmedia/src/test/mips_test.c > +++ b/pjmedia/src/test/mips_test.c > @@ -1172,7 +1172,7 @@ static pj_status_t > wsola_discard_get_frame(struct pjmedia_port *this_port, > pjmedia_frame *frame) > { > struct wsola_discard_port *wp = (struct wsola_discard_port*)this_port; > - pj_status_t status; > + pj_status_t status = PJ_SUCCESS; > > while (pjmedia_circ_buf_get_len(wp->circbuf) < > PJMEDIA_PIA_SPF(&wp->base.info) * (CIRC_BUF_FRAME_CNT-1)) > > > Regards, > Michael > > _______________________________________________ > Visit our blog: http://blog.pjsip.org > > pjsip mailing list > pjsip@lists.pjsip.org > http://lists.pjsip.org/mailman/listinfo/pjsip_lists.pjsip.org >