]> git.eshelyaron.com Git - emacs.git/commitdiff
* xdisp.c: Integer overflow fix.
authorPaul Eggert <eggert@cs.ucla.edu>
Sun, 4 Sep 2011 18:48:35 +0000 (11:48 -0700)
committerPaul Eggert <eggert@cs.ucla.edu>
Sun, 4 Sep 2011 18:48:35 +0000 (11:48 -0700)
(try_window_id): Check Emacs fixnum range before converting to 'int'.

src/ChangeLog
src/xdisp.c

index d2f956fe66f761182023d68711a51f922ffcd2bf..e7dac0204a07d7a79fd86c5659087cba12120d90 100644 (file)
@@ -1,5 +1,8 @@
 2011-09-04  Paul Eggert  <eggert@cs.ucla.edu>
 
+       * xdisp.c: Integer overflow fix.
+       (try_window_id): Check Emacs fixnum range before converting to 'int'.
+
        * window.c: Integer overflow fixes.
        (window_scroll_line_based, Frecenter):
        Check that an Emacs fixnum is in range before assigning it to 'int'.
index 1716cc821880ff2b0401afa60ab8832cb8f8127a..f11362c1ae6d3b6e2e4a13b5505934b97f5f2b85 100644 (file)
@@ -16919,8 +16919,8 @@ try_window_id (struct window *w)
   {
     int this_scroll_margin, cursor_height;
 
-    this_scroll_margin = max (0, scroll_margin);
-    this_scroll_margin = min (this_scroll_margin, WINDOW_TOTAL_LINES (w) / 4);
+    this_scroll_margin =
+      max (0, min (scroll_margin, WINDOW_TOTAL_LINES (w) / 4));
     this_scroll_margin *= FRAME_LINE_HEIGHT (it.f);
     cursor_height = MATRIX_ROW (w->desired_matrix, w->cursor.vpos)->height;