[Emacs-diffs] emacs-24 r117471: * eval.c (internal_lisp_condition_case): Don't overrun the stack

[Paul Eggert]

------------------------------------------------------------
revno: 117471
revision-id: address@hidden
parent: address@hidden
committer: Paul Eggert <address@hidden>
branch nick: emacs-24
timestamp: Mon 2014-09-01 23:29:01 -0700
message:
  * eval.c (internal_lisp_condition_case): Don't overrun the stack
  
  when configured --with-wide-int on typical 32-bit platforms.
modified:
  src/ChangeLog                  changelog-20091113204419-o5vbwnq5f7feedwu-1438
  src/eval.c                     eval.c-20091113204419-o5vbwnq5f7feedwu-237

=== modified file 'src/ChangeLog'
--- a/src/ChangeLog     2014-08-31 15:53:27 +0000
+++ b/src/ChangeLog     2014-09-02 06:29:01 +0000
@@ -1,3 +1,8 @@
+2014-09-02  Paul Eggert  <address@hidden>
+
+       * eval.c (internal_lisp_condition_case): Don't overrun the stack
+       when configured --with-wide-int on typical 32-bit platforms.
+
 2014-08-31  Eli Zaretskii  <address@hidden>
 
        * xdisp.c (display_and_set_cursor): Call erase_phys_cursor also

=== modified file 'src/eval.c'
--- a/src/eval.c        2014-02-10 09:48:17 +0000
+++ b/src/eval.c        2014-09-02 06:29:01 +0000
@@ -1280,7 +1280,7 @@
   { /* The first clause is the one that should be checked first, so it should
        be added to handlerlist last.  So we build in `clauses' a table that
        contains `handlers' but in reverse order.  */
-    Lisp_Object *clauses = alloca (clausenb * sizeof (Lisp_Object *));
+    Lisp_Object *clauses = alloca (clausenb * sizeof *clauses);
     Lisp_Object *volatile clauses_volatile = clauses;
     int i = clausenb;
     for (val = handlers; CONSP (val); val = XCDR (val))