Re: [E-devel] E SVN: discomfitor trunk/e/src/modules/everything-files

[Gustavo S. B. <bar...@pr...>]

On Wednesday, October 13, 2010, Enlightenment SVN
<no-...@en...> wrote:
> Log:
>   fix annoying random segv
>
> Author:       discomfitor
> Date:         2010-10-13 19:57:35 -0700 (Wed, 13 Oct 2010)
> New Revision: 53387
>
> Modified:
>   trunk/e/src/modules/everything-files/e_mod_main.c
>
> Modified: trunk/e/src/modules/everything-files/e_mod_main.c
> ===================================================================
> --- trunk/e/src/modules/everything-files/e_mod_main.c   2010-10-14 02:37:37 UTC (rev 53386)
> +++ trunk/e/src/modules/everything-files/e_mod_main.c   2010-10-14 02:57:35 UTC (rev 53387)
> @@ -177,6 +177,8 @@
>     char buf[4096];
>     Eina_Bool is_dir;
>
> +   if (!d->directory)
> +     return;
>     d->dirp = opendir(d->directory);
>     if (!d->dirp) return;
>
> @@ -342,6 +344,7 @@
>       }
>
>     free(d->directory);
> +   d->directory = NULL;
>     E_FREE(d);

This makes NO sense, don't do this

>
>     p->thread = NULL;
> @@ -492,6 +495,7 @@
>           }
>
>         free(d->directory);
> +        d->directory = NULL;
>         E_FREE(d);
>         p->thread = NULL;
>       }
>
>
> ------------------------------------------------------------------------------
> Beautiful is writing same markup. Internet Explorer 9 supports
> standards for HTML5, CSS3, SVG 1.1,  ECMAScript5, and DOM L2 & L3.
> Spend less time writing and  rewriting code and more time creating great
> experiences on the web. Be a part of the beta today.
> http://p.sf.net/sfu/beautyoftheweb
> _______________________________________________
> enlightenment-svn mailing list
> enl...@li...
> https://lists.sourceforge.net/lists/listinfo/enlightenment-svn
>

-- 
Gustavo Sverzut Barbieri
http://profusion.mobi embedded systems
--------------------------------------
MSN: bar...@gm...
Skype: gsbarbieri
Mobile: +55 (19) 9225-2202