Re: [osmo-pcu 0.2.896-0a8f] testsuite: 4 failed

On Fri, Mar 24, 2017 at 10:53:53AM +0100, Arnaud ZANETTI wrote: Arnaud, many thanks for submitting your report! I took a look, and it actually is an amazingly stupid bug in the test itself. In fact 0xc0 is the correct expected result, which our decoding routines also actually return, but the author of the test has put two osmo_hexdump() calls into the same log line. Since osmo_hexdump() uses an internal static buffer, that can't work: one of the two hexdumps wins, so that both dumped strings are the same. m( On our "usual" systems, the expected hexdump wins and it prints them both as the expected value. On the pi, the actually decoded hexdump wins, and it prints both "expected" and "decoded" data as what the decoding routine returns. In other words, the test never before printed expected vs. decoded results, and the author has never properly checked the stdout / stderr results. The test does check expected results, but not the remaining bits after the decoded data. In my/our review, we oversaw the double osmo_hexdump() and a bunch of other issues. Looking into this triggered a whole series of fixes, more explanations in the commit logs: https://gerrit.osmocom.org/#/q/topic:bitcomp_test The last one also uncovers a test marked "invalid" which seems to be valid, or fails to be detected as invalid. I don't have a fix for that, would be good if someone else could look into that. ~N -- - Neels Hofmeyr <nhofmeyr(a)sysmocom.de> http://www.sysmocom.de/ ======================================================================= * sysmocom - systems for mobile communications GmbH * Alt-Moabit 93 * 10559 Berlin, Germany * Sitz / Registered office: Berlin, HRB 134158 B * Geschäftsführer / Managing Directors: Harald Welte