On 16 September 2013 08:19, Glenn Fowler <[email protected]> wrote:
>
> if you had built with nmake and had -g in CCFLAGS the src would have
> compiled with -D_BLD_DEBUG and each malloc would be tagged with the
> originating file and line number -- which would have lead you sh/init.c
>
> if you are skipping nmake or hiding some options in a cc wrapper
> (I'm guitly of this too) add -D_BLD_DEBUG when you compile with -g
> (but don't use _BLD_DEBUG for production/timing)
>
> the bug is indeed signal related but probably not for the reason you think
> its an under-allocation related to the new siginfo code
> here's a fix
> homework assignment is to explain why adding 8 bytes works and 7 doesn't
> (I'm not familiar with the svar_init() code or SH_SIG internal usage
>
> --
>     static void siginfo_init(Shell_t *shp)
>     {
>         struct Svars    *sp;
>         size_t          dsize = sizeof(siginfo_t)+SIGNAME_MAX+8;
>         svar_init(shp,SH_SIG,shtab_siginfo,dsize);
>         sp = (struct Svars*)SH_SIG->nvfun->next;
>         sp->dsize = dsize;
>     }
> --

Glenn, could you send a diff to the list, please?

Ced
-- 
Cedric Blancher <[email protected]>
Institute Pasteur
_______________________________________________
ast-developers mailing list
[email protected]
http://lists.research.att.com/mailman/listinfo/ast-developers

Reply via email to