Re: [Tracker] yet another patch for segfault

["Samuel Cormier-Iijima" <sciyoshi gmail com>]

argghh, that caused another bug... forgot about g_strduping it, now it
abrt's sometimes when trying to free the static string. just wrap the
empty string "" with g_strdup in my last patch.

Index: tracker-dbus-metadata.c
===================================================================
RCS file: /cvs/gnome/tracker/src/trackerd/tracker-dbus-metadata.c,v
retrieving revision 1.7
diff -u -r1.7 tracker-dbus-metadata.c
--- tracker-dbus-metadata.c     13 Sep 2006 22:55:39 -0000      1.7
+++ tracker-dbus-metadata.c     13 Sep 2006 23:29:12 -0000
@@ -302,7 +302,7 @@
                                               array[i] = g_strdup (row[i]);
                                       }
                               } else {
-                                       array[i] = "";
+                                       array[i] = g_strdup ("");
                               }
                       }


On 9/13/06, Jamie McCracken <jamiemcc blueyonder co uk> wrote:
> Samuel Cormier-Iijima wrote:
> > ok, when using Metadata.Get, if one of the fields doesn't have any
> > associated metadata dbus complains that we're passing NULL (arguably a
> > bug in DBus). here's a fix for the segfault
> >
>
> thx patch applied to cvs
>
> --
> Mr Jamie McCracken
> http://jamiemcc.livejournal.com/