Re: [Tracker] tracker-extract-playlist

[Martyn Russell <martyn lanedo com>]

On 11/10/12 07:31, Putinei Ionut wrote:
> Hello all,

Hello,

> I am experiencing some problems with tracker-extract-playlist plugin.
> There is callback connected to signal "entry-parsed" of and object of
> type "TotemPlParser".
> This is how signal is connected:
> g_signal_connect (G_OBJECT (pl), "entry-parsed",
>                   G_CALLBACK (entry_parsed), &data);
>
> this is the prototype of callback method :
> static void
> entry_parsed (TotemPlParser *parser,
>                const gchar   *to_uri,
>                GHashTable    *to_metadata,
>                gpointer       user_data)
>
>
> My question is where does the GHashTable come from?
> because sometimes that member is  NULL and tracker-extract process
> fails. (no null checks on "to_metadata"  variable)
> GLib-Critical **: g_hash_table_lookup: assertion `hash_table != NULL'
> failed.
>
> Hope i am not knocking on the wrong door and this to be issue because of
> totem-pl-parse library.

I don't know if they've changed something in the library, but the 
documentation has no hints either about if NULL is possible. So I've 
added a check:

http://git.gnome.org/browse/tracker/commit/?id=fb9ebf07e5179b8e56717a27cfd9d8cf38666508

I also added a log message if there are > 1k entries - which is when we 
ignore the playlist for performance reasons.

http://git.gnome.org/browse/tracker/commit/?id=02783022e70b0fa28777bfc5381960c5a73f12a8

Thanks for noticing!

--
Regards,
Martyn

Founder and CEO of Lanedo GmbH.