X-Git-Url: http://repo.macrolet.net/gitweb/?a=blobdiff_plain;f=src%2Fruntime%2Finterr.c;h=18b3e9e11a70ca5cb611db8c9dbbde151f8dc420;hb=dc5e3163fe667e2629c7769aa8cf2e501eeeefa6;hp=1fd412c706507226646b1b94998c6ca05d806c26;hpb=62272dd0e7254b19a484fa1a2050ba15504d3ece;p=sbcl.git diff --git a/src/runtime/interr.c b/src/runtime/interr.c index 1fd412c..18b3e9e 100644 --- a/src/runtime/interr.c +++ b/src/runtime/interr.c @@ -15,6 +15,7 @@ #include #include +#include #include "arch.h" #include "signal.h" @@ -152,6 +153,16 @@ describe_internal_error(os_context_t *context) lispobj debug_print(lispobj string) { - fprintf(stderr, "%s\n", (char *)(((struct vector *)PTR(string))->data)); + /* This is a kludge. It's not actually safe - in general - to use + %primitive print on the alpha, because it skips half of the + number stack setup that should usually be done on a function call, + so the called routine (i.e. this one) ends up being able to overwrite + local variables in the caller. Rather than fix this everywhere + that %primitive print is used (it's only a debugging aid anyway) + we just put guarantee our safety by putting an unused buffer on + the stack before doing anything else here */ + char untouched[32]; /* GCC warns about not using this, but that's the point.. */ + fprintf(stderr, "%s\n", + (char *)(((struct vector *)native_pointer(string))->data),untouched); return NIL; }