X-Git-Url: http://repo.macrolet.net/gitweb/?a=blobdiff_plain;f=src%2Fruntime%2Finterr.c;h=a5f8650c5ecd9109b5079bb9e91d8716be293844;hb=3bd7a97d1b11a2b0aee086ef211cae807f3dfc35;hp=1fd412c706507226646b1b94998c6ca05d806c26;hpb=62272dd0e7254b19a484fa1a2050ba15504d3ece;p=sbcl.git diff --git a/src/runtime/interr.c b/src/runtime/interr.c index 1fd412c..a5f8650 100644 --- a/src/runtime/interr.c +++ b/src/runtime/interr.c @@ -152,6 +152,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; }