> +      buf = malloc(strlen(pwd)+1);
    > +      if(!buf)
    > +        fatal ("`malloc' failed in init_buffer\n");

    This should have used xmalloc instead of calling malloc and checking
    for errors.

Actually I don't think so.  xmalloc reports errors using Fsignal, and
that is the wrong thing to do, this early in Emacs startup.

It might be ok just to use abort, since it should not be possible
to run out of memory at that point.

In xsmfns.c, it would be right to use xmalloc.


_______________________________________________
Emacs-devel mailing list
Emacs-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-devel

Reply via email to