After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 586464 - [xingmux] Unit test failures on sparc and arm/eabi
[xingmux] Unit test failures on sparc and arm/eabi
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-ugly
git master
Other Linux
: Normal critical
: 0.10.14
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2009-06-20 12:50 UTC by Sebastian Dröge (slomo)
Modified: 2010-01-11 13:07 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Fix xingmux on arm (1.85 KB, patch)
2010-01-11 12:47 UTC, Arnaud Patard
committed Details | Review

Description Sebastian Dröge (slomo) 2009-06-20 12:50:36 UTC
Hi,
the unit test of xingmux fails on sparc and arm/eabi since at least 0.10.7 (I don't have older build logs with the test suite).

arm/eabi:

Running suite(s): xingmux
0%: Checks: 1, Failures: 1, Errors: 0
elements/xingmux.c:112:F:general:test_xing_remux:0: Assertion 'memcmp (test_xing, GST_BUFFER_DATA (outbuffer), 25) == 0' failed
FAIL: elements/xingmux


sparc:

Running suite(s): xingmux
0%: Checks: 1, Failures: 0, Errors: 1
elements/xingmux.c:95:E:general:test_xing_remux:0: (after this point) Received signal 10 (Bus error)
FAIL: elements/xingmux
Comment 1 Arnaud Patard 2010-01-11 12:47:39 UTC
Created attachment 151165 [details] [review]
Fix xingmux on arm
Comment 2 Arnaud Patard 2010-01-11 12:48:06 UTC
I've wrote a patch to fix the arm case. Maybe it'll solve this bug on sparc too but as I've no sparc at hand, we'll have to wait someone test it (or
wait a new upload in debian).
Please, if you commit my patch,  make sure that it's my @mandriva.com address is used :)
Comment 3 Sebastian Dröge (slomo) 2010-01-11 13:07:03 UTC
Thanks for the patch. As SPARC has the same alignment requirements, this will most likely fix it there too.

commit 9094dc1bc6566d3afd7a98ac408208c1ee10ab62
Author: Arnaud Patard <apatard@mandriva.com>
Date:   Mon Jan 11 12:23:35 2010 +0100

    xingmux: Fix unaligned memory access
    
    ARM/SPARC need 32bit alignment but xingmux accesses possibly
    unaligned memory, which leads to SIGBUS.
    
    Fixes bug #586464.