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@xxxxxxxxxxxxx> 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@xxxxxxxxxxxxxxx
http://lists.pjsip.org/mailman/listinfo/pjsip_lists. pjsip.org
_______________________________________________ Visit our blog: http://blog.pjsip.org pjsip mailing list pjsip@xxxxxxxxxxxxxxx http://lists.pjsip.org/mailman/listinfo/pjsip_lists.pjsip.org
