Hi, On Thu, 5 Aug 2010 22:14:15 +0200 Daniel Marjamäki <[email protected]> wrote:
> I believe this will fix a memory leak in src/search.c. But please
> double-check!
>
> At lines 1417-1422 the search_text is allocated:
> if (enc != NULL && g_utf8_validate(utf8_search_text,
> utf8_text_len, NULL)) {
> search_text = g_convert(utf8_search_text,
> utf8_text_len, enc, "UTF-8", NULL, NULL, NULL);
> }
> if (search_text == NULL)
> search_text = g_strdup(utf8_search_text);
>
>
> The only usage of search_text between the lines 1417-1456 is:
> argv_prefix[i++] = g_strdup(search_text);
>
>
> At line 1456 there is this code:
> if (argv == NULL) /* no files */
> {
> g_strfreev(argv);
> return FALSE;
> }
>
>
> I assume that a "g_free(search_text);" is needed before the return.
I've just committed a patch based on your findings with svn r5145.
However, maybe we should revisit the function as it appears to me at
the moment that search_text could be replaced completely by using
utf8_search_text.
Cheers,
Frank
--
http://frank.uvena.de/en/
pgpcO0hDjACWn.pgp
Description: PGP signature
_______________________________________________ Geany-devel mailing list [email protected] http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel
