Bug 150026 - [GTK] Media tests crashing with gst-plugins-bad 1.4.4 and cpp 5.2
Summary: [GTK] Media tests crashing with gst-plugins-bad 1.4.4 and cpp 5.2
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Linux
: P2 Normal
Assignee: Nobody
URL:
Keywords: Gtk
Depends on:
Blocks:
 
Reported: 2015-10-12 04:57 PDT by Mario Sanchez Prada
Modified: 2015-10-12 05:37 PDT (History)
8 users (show)

See Also:


Attachments
Patch proposal (4.23 KB, patch)
2015-10-12 05:06 PDT, Mario Sanchez Prada
pnormand: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mario Sanchez Prada 2015-10-12 04:57:52 PDT
As it was found out while working on bug 149749, I had quite some trouble running the layout tests on a recently installed development machine with "standard" Debian Testing (no packages from other sources), where I would get ~230 tests crashing due to some kind of problem in GStreamer, as you can check here:

  Description: https://bugs.webkit.org/show_bug.cgi?id=149749#c6  
  Results here: https://drive.google.com/file/d/0B6Gdj3EoWfFLRFNSejRIMlViY28/view?usp=sharing

After some investigation today, I finally found the problem (and why it was so hard to find in the first place): 

The problem is that a newer versions of cpp (I have 5.2.1, while the bot has 4.9.2) produces a different output while processing the headers from libfaad2, causing gst-plugins-bad to be compiled without the FAAD2_MINOR_VERSION definition in config.h, which causes crashes and stack corruption on runtime.

This problem has been reported already for Fedora 22 in the RH's bugzilla:
https://bugzilla.redhat.com/show_bug.cgi?id=1219320

Fortunately, this has been already fixed upstream in gst-plugins-bad:
http://cgit.freedesktop.org/gstreamer/gst-plugins-bad/commit/?id=1f738ca5b8711ca5532a326cd646312e60484863

I'm 100% sure that's the same issue I was seeing because I could double check myself that the output for f21 and f22 as mentioned in https://bugzilla.redhat.com/show_bug.cgi?id=1219320#c21 is exactly the same one that I'm getting in the bots (with help from aperezdc), with cpp 4.9.2, and my local Debian Testing machine, with cpp 5.2.1.

Besides, I've applied the patch locally myself to the gst-plugins-bad from the internal jhbuild, recompiled it together with the WebKit libraries and binaries... and it worked! No more weird crashes around running the tests!

So, I'll submit a new patch to backport that patch from upstream into gst-plugins-bad 1.4.4 from the jhbuild
Comment 1 Mario Sanchez Prada 2015-10-12 05:06:35 PDT
Created attachment 262883 [details]
Patch proposal

See the proposed patch attached, please review. Thanks!
Comment 2 Mario Sanchez Prada 2015-10-12 05:37:44 PDT
Committed r190854: <http://trac.webkit.org/changeset/190854>