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 308438 - Registering the same tag twice leads to assertion warnings
Registering the same tag twice leads to assertion warnings
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gstreamer (core)
0.8.9
Other Linux
: Normal normal
: 0.8.11
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2005-06-20 21:03 UTC by Loïc Minier
Modified: 2005-06-21 08:42 UTC
See Also:
GNOME target: ---
GNOME version: 2.7/2.8



Description Loïc Minier 2005-06-20 21:03:06 UTC
Version details: Debian 0.8.9-1
Distribution/Version: Debian/sid

Hi,

(process:17854): GStreamer-CRITICAL **: gst_tag_register: assertion `info ==
NULL' failed

is output by gst-compprep if you build two incompatible plugins (ie. they
register the same tag).

this is the case of the CDIO plugin which registers "discid" already registered
in cdparanoia.

Bye,
Comment 1 Tim-Philipp Müller 2005-06-20 21:34:21 UTC
Looks like a bug in gst_tag_register() IMHO. The API says "Registers a new tag
type for the use with GStreamer's type system. If a type with that name is
already registered, that one is used. The old registration may have used a
different type however. So don't rely on your supplied values. This function
takes ownership of all supplied variables.", so by implication it should be fine
to register the same tag twice (at least as long as the types are the same).

Cheers
 -Tim
Comment 2 Tim-Philipp Müller 2005-06-21 08:42:24 UTC
This should be fixed in CVS now:

 2005-06-21  Tim-Philipp Muller  <tim at centricular dot net>

        * gst/gsttag.c: (gst_tag_register):
          Don't spew out an assertion warning if a tag is
          already registered, unless the new type differs
          from the old one (fixes #308438).

Cheers
 -Tim