[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Linphone-developers] Problem with an infinite loop in bcg729 codec

From: Johan Pascal
Subject: Re: [Linphone-developers] Problem with an infinite loop in bcg729 codec
Date: Mon, 23 Nov 2020 11:36:39 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.12.0

Hi Frederic,

sorry for the delay. I just released the version 1.1.1 of bcg729 including fixes for the errors given by ASAN (and the infinite loop one). Most of them were harmless but not all! Thanks for your help.

The test patterns evolved with the fix, so to really get the update and test it you must delete the pattern directory in your tree.



On 18/09/2020 16:07, BOITEUX, FREDERIC wrote:
	Hi Johan,

Thanks for your quick reply !

About the test suite, you’re correct, I didn’t had the latest pattern archive (I had an obsolete URL which didn’t work anymore and found somewhere an outdated archive). With the latest sources and patterns, all tests went fine 😊.

About ASAN errors, I still got errors, I share with you the whole options I use : I build the test suite with the following command :

(mkdir build_testASAN && cd build_testASAN && cmake -D ENABLE_TESTS=YES -DCMAKE_SKIP_INSTALL_RPATH=ON  -D CMAKE_C_FLAGS:STRING="${CMAKE_C_FLAGS} -fsanitize=address -fno-omit-frame-pointer  -fno-common -fsanitize=undefined -fsanitize=shift -fsanitize=integer-divide-by-zero -fsanitize=unreachable -fsanitize=vla-bound -fsanitize=signed-integer-overflow -fsanitize=bounds-strict -fsanitize=object-size -fsanitize=nonnull-attribute -fsanitize=returns-nonnull-attribute -fsanitize=bool -fsanitize=enum" .. && make)
[I'm running on Debian 10 Buster]

To be able to fully compile test suite, I had to fix some trivial warnings in some test programs, you'll find them attached to this e-mail, feel free to include them in your source tree.

 I run tests with :
(cd build_testASAN/test && ./testCampaignAll 2>&1|tee testCampaignAll.log)

The ASAN system produces a lot of logs, but I still get errors, like :

Test fixedCodebookSearch bloc
… lot of ASAN log…
…/bcg729.git/src/fixedCodebookSearch.c:211:28: runtime error: left shift of negative value -1
…/bcg729.git/src/fixedCodebookSearch.c:212:28: runtime error: left shift of negative value -1
…/bcg729.git/src/fixedCodebookSearch.c:213:28: runtime error: left shift of negative value -1
…/bcg729.git/src/fixedCodebookSearch.c:210:28: runtime error: left shift of negative value -1
  ... Pass

And when counting these errors  for the whole test suite, I get :
$ grep error testCampaignAll.log| wc -l

I'm not sure about the gravity/impact of these messages…

About the endless loop, many thanks for the fix, I've also tested it successfully, it's perfect 😊 !

	With regards,
	Frédéric Boiteux - Odigo IVR

This message contains information that may be privileged or confidential and is the property of the Capgemini Group. It is intended only for the person to whom it is addressed. If you are not the intended recipient, you are not authorized to read, print, retain, copy, disseminate, distribute, or use this message or any part thereof. If you receive this message in error, please notify the sender immediately and delete all copies of this message.

Linphone-developers mailing list

reply via email to

[Prev in Thread] Current Thread [Next in Thread]