Re: [Tracker] yet another patch for segfault
- From: "Samuel Cormier-Iijima" <sciyoshi gmail com>
- To: "Jamie McCracken" <jamiemcc blueyonder co uk>
- Cc: Tracker List <tracker-list gnome org>
- Subject: Re: [Tracker] yet another patch for segfault
- Date: Wed, 13 Sep 2006 19:30:27 -0400
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/
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]