Re: [Tracker] [PATCH] Potential crasher in set_language()

[Laurent Aguerreche <laurent aguerreche free fr>]

Le lundi 23 juillet 2007 Ã 00:38 +0200, Laurent Aguerreche a Ãcrit :
> Le dimanche 22 juillet 2007 Ã 23:26 +0100, jamie a Ãcrit :
> > On Mon, 2007-07-23 at 00:16 +0200, Laurent Aguerreche wrote:
> > > Le lundi 23 juillet 2007 Ã 00:13 +0200, Laurent Aguerreche a Ãcrit :
> > > > Le dimanche 22 juillet 2007 Ã 22:33 +0100, jamie a Ãcrit :
> > > > > On Sun, 2007-07-22 at 23:21 +0200, Laurent Aguerreche wrote:
> > > > > > Hello,
> > > > > >
> > > > > > a patch to fix a potential crasher in tracker_set_language() function :
> > > > > >
> > > > > > - g_free() was called whether pointer to free could be NULL;
> > > > > > - we were mixing allocated strings and on-stack strings;
> > > > > > - added tracker_get_english_lang_code() function and put back tmap
> > > > > > implementation in tracker-utils.c (avoid warnings at building time)
> > > > > >
> > > > > > + cosmetic changes : I removed some empty lines for instance.
> > > > > >
> > > > > >
> > > > > > Can I commit?
> > > > >
> > > > > ok for this one but no more cleanups pls until post 0.6 as I want to
> > > > > release tomorrow
> > > > >
> > > > > (I would prefer bugfixes to be separate from wide ranging cleanups if
> > > > > possible)
> > > >
> > > > Ok.
> > > >
> > > >
> > > > About Gaim: it recently changed its name in favour of Pidgin. Gaim uses
> > > > library Purple as a network layer. This library is also responsible for
> > > > chat logging.
> > >
> > > Sorry, it is Pidgin which uses Purple.
> > >
> > >
> > > > For people who used Gaim (like me), "logs" directory in "~/.gaim/logs"
> > > > became a symbolic link to "~/.purple/logs".
> > > > Since trackerd do not follow links, it does not see "logs" link as a
> > > > correct directory and do not index anything from this directory.
> > > > Furthermore, we need to index Purple chat logs (not Pigin directly
> > > > because Purple may be used by others programs).
> > > >
> > > > How to fix that? Will we delay that for post Tracker 0.6?
> >
> >
> > I quickly added ~/.purple/logs as a gaim conversation directory
> >
> > I need someone to test its picking then up though
>
> My Purple logs seem to be indexed...
>
>
> On another side, I took a look at tracker-preferences.c and saw a code
> with a really bad complexity, so a little patch.
> I hope that such a code is not anywhere else!

I forgot to ask: can I commit these changes?


Laurent.

> > jamie
> _______________________________________________
> tracker-list mailing list
> tracker-list gnome org
> http://mail.gnome.org/mailman/listinfo/tracker-list

Attachment: signature.asc
Description: Ceci est une partie de message =?ISO-8859-1?Q?num=E9riquement?= =?ISO-8859-1?Q?_sign=E9e?=