Is there a backtrace for this segfault ?? I'm asking because the illume2 
keyboard module has very similar code, and I've never seen it segfault here.

Thanks,
dh

On 12/22/2010 05:22 PM, Enlightenment SVN wrote:
> Log:
> this will avoid a segv, but the entire function is broken and should be 
> looked at by someone who knows the code
>
>
> Author:       discomfitor
> Date:         2010-12-22 14:22:59 -0800 (Wed, 22 Dec 2010)
> New Revision: 55698
> Trac:         http://trac.enlightenment.org/e/changeset/55698
>
> Modified:
>    trunk/e/src/modules/illume/e_kbd_buf.c
>
> Modified: trunk/e/src/modules/illume/e_kbd_buf.c
> ===================================================================
> --- trunk/e/src/modules/illume/e_kbd_buf.c    2010-12-22 11:23:30 UTC (rev 
> 55697)
> +++ trunk/e/src/modules/illume/e_kbd_buf.c    2010-12-22 22:22:59 UTC (rev 
> 55698)
> @@ -164,9 +164,10 @@
>            if ((actual_len + strlen(str) + 1)>  actual_size)
>              {
>                 actual_size += 64;
> -               actual = realloc(actual, actual_size);
> +               actual = malloc(actual_size);
> +                  EINA_SAFETY_ON_NULL_RETURN(actual);
> +                  strcpy(actual, str);
>              }
> -          strcpy(actual + actual_len, str);
>            actual_len += strlen(str);
>         }
>        }
>

------------------------------------------------------------------------------
Learn how Oracle Real Application Clusters (RAC) One Node allows customers
to consolidate database storage, standardize their database environment, and, 
should the need arise, upgrade to a full multi-node Oracle RAC database 
without downtime or disruption
http://p.sf.net/sfu/oracle-sfdevnl
_______________________________________________
enlightenment-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to