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 309087 - artsdsink registers conflicting property "name" which causes breakage
artsdsink registers conflicting property "name" which causes breakage
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins
0.8.10
Other Linux
: Normal major
: 0.8.11
Assigned To: GStreamer Maintainers
GStreamer Maintainers
: 312725 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2005-06-26 21:45 UTC by Loïc Minier
Modified: 2005-08-07 23:01 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
fix artsd registering "name" (782 bytes, patch)
2005-06-26 21:48 UTC, Loïc Minier
none Details | Review

Description Loïc Minier 2005-06-26 21:45:41 UTC
Hi,

The artsdsink plugin registers the "name" property (presumably to select an
output device) with default value NULL.  This causes breakage when pipeline are
constructed with the artsdsink element in the 0.8.9 series of GStreamer.

(gst_*check_uniqueness complains of a duplicate name "(null)" in pipeline)

Bye,
Comment 1 Loïc Minier 2005-06-26 21:48:38 UTC
Created attachment 48381 [details] [review]
fix artsd registering "name"
Comment 2 Ronald Bultje 2005-06-27 06:01:09 UTC
Isn't it better to rename the property than to remove it?
Comment 3 Loïc Minier 2005-06-27 10:06:17 UTC
I forgot to mention I grepped on "name" to check whether it was in use, and
couldn't find any occurrence beside the registration, hence the removal.

It's probably best if you check too so that we don't introduce more breakage if
it is in use somewhere.
Comment 4 Loïc Minier 2005-06-27 10:18:37 UTC
FYI, I get segfaults in libarts, as well as another Debian user with the patch.

I suspect this is in no way connected to the patch, but related to the build
environment and that the user would have the crash with a self-rebuilt plugins
without the patch.

This is terribly visible in:
http://bugs.kde.org/show_bug.cgi?id=62603

And in the "Important bugs - forwarded to upstream software authors (7 bugs)"
section at:
http://bugs.debian.org/cgi-bin/pkgreport.cgi?src=arts

I'll confirm this with the reporter though.
Comment 5 Ronald Bultje 2005-07-01 17:28:34 UTC
Fixed differently, by renaming the property.
Comment 6 Loïc Minier 2005-07-02 18:28:42 UTC
Thanks, I synced with:
http://cvs.freedesktop.org/gstreamer/gst-plugins/ext/artsd/gstartsdsink.c?r1=1.19&r2=1.19.6.1

(Any reason this wasn't commited in HEAD?)

BTW, I thought Tim-Philipp Mueller had a nice fix a couple of days ago, at the
time the BTS was offline.
Comment 7 Ronald Bultje 2005-07-02 23:08:11 UTC
Simple, the plugin isn't ported yet. Whoever ports it will take 0.8 code and
port it.
Comment 8 Tim-Philipp Müller 2005-08-07 23:01:32 UTC
*** Bug 312725 has been marked as a duplicate of this bug. ***