Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 321745 - media-libs/gst-plugins-ugly-0.10.13 segfaults in ASF demuxer
Summary: media-libs/gst-plugins-ugly-0.10.13 segfaults in ASF demuxer
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High minor (vote)
Assignee: GStreamer package maintainers
URL: https://bugzilla.gnome.org/show_bug.c...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-05-28 00:13 UTC by bob5972
Modified: 2010-12-27 11:39 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description bob5972 2010-05-28 00:13:01 UTC
Some wma files can cause a null pointer exception and segfault in the asfdemux of gst-plugins-ugly.  This has been fixed in the later versions of gst-plugins-ugly, but 0.10.13, the latest version in Gentoo still has it.  I hit this all the time and it causes amarok or totem to crash.  I have a patch that fixes the issue.

From cd361dfaeb79200024caf7dc88379d1fc9fb2136 Mon Sep 17 00:00:00 2001
From: Michael Banack <bob5972@gmail.com>
Date: Thu, 27 May 2010 16:49:20 -0700
Subject: [PATCH] Fixed NULL Pointer Exception in gstasfdmux.c

---
 gst/asfdemux/gstasfdemux.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/gst/asfdemux/gstasfdemux.c b/gst/asfdemux/gstasfdemux.c
index 39a9f2e..cb176ee 100644
--- a/gst/asfdemux/gstasfdemux.c
+++ b/gst/asfdemux/gstasfdemux.c
@@ -2202,13 +2202,14 @@ gst_asf_demux_get_gst_tag_from_tag_name (const gchar * name_utf8)
     "WM/Year", GST_TAG_DATE}
     /* { "WM/Composer", GST_TAG_COMPOSER } */
   };
-  gsize out = strlen (name_utf8);
   guint i;
 
   if (name_utf8 == NULL) {
     GST_WARNING ("Failed to convert name to UTF8, skipping");
     return NULL;
   }
+  
+  gsize out = strlen (name_utf8);
 
   for (i = 0; i < G_N_ELEMENTS (tags); ++i) {
     if (strncmp (tags[i].asf_name, name_utf8, out) == 0) {
-- 
1.7.1
Comment 1 Michael Weber (RETIRED) gentoo-dev 2010-06-07 22:55:09 UTC
Hello Bob,

it's unlikely that the maintainer will change the stabled version, but hopefullyt, he'll take it as version bump request/argument.

Thanks fir the report+patch, Michael

p.s. Can you name a URL to a file that produces this error? Preferrably a creative common/... one?
Comment 2 Mart Raudsepp gentoo-dev 2010-06-07 23:22:17 UTC
Removing blocker. We can't just block stabilization of two remaining arches of the whole gstreamer set on some corner case null pointer exception in a patent encumbered audio codec. It does appear to be a regression specific to 0.10.13 though, according to my git annotate checks.
Bumps of everything should be coming later this week if all goes as planned...
Comment 3 bob5972 2010-06-08 00:10:51 UTC
I don't have an example I can link to.  The files I have that break are some huge web radio files in Japanese.  I can't tell if they're missing tags it's expecting, corrupt, or just using a weird character set.  But in any case I believe the new version fixes it.

Here's a link to the upstream bug for reference.
https://bugzilla.gnome.org/show_bug.cgi?id=602280

Thanks all!
Comment 4 Mart Raudsepp gentoo-dev 2010-12-27 11:39:44 UTC
gst-plugins-ugly-0.10.16 in ~arch with the referenced fix then included upstream, as I understand it. Thanks