5.7.023

   1 To: vim-dev@vim.org
   2 Subject: Patch 5.7.023
   3 Fcc: outbox
   4 From: Bram Moolenaar <Bram@moolenaar.net>
   5 ------------
   6
   7 Patch 5.7.023
   8 Problem:    Crash when a WinLeave autocommand deletes the buffer in the other
   9             window.
  10 Solution:   Check that after executing the WinLeave autocommands there still
  11             is a window to be closed.  Also update the test that was supposed
  12             to check for this problem.
  13 Files:      src/window.c, testdir/test13.in, testdir/test13.ok
  14
  15
  16 *** ../vim-5.7.22/src/window.c  Tue Jan 25 10:47:16 2000
  17 --- src/window.c        Thu Jan 18 13:41:47 2001
  18 ***************
  19 *** 900,910 ****
  20         {
  21             other_buffer = TRUE;
  22             apply_autocmds(EVENT_BUFLEAVE, NULL, NULL, FALSE, curbuf);
  23 !           if (!win_valid(win))
  24                 return;
  25         }
  26         apply_autocmds(EVENT_WINLEAVE, NULL, NULL, FALSE, curbuf);
  27 !       if (!win_valid(win))
  28             return;
  29       }
  30   #endif
  31 --- 900,910 ----
  32         {
  33             other_buffer = TRUE;
  34             apply_autocmds(EVENT_BUFLEAVE, NULL, NULL, FALSE, curbuf);
  35 !           if (!win_valid(win) || firstwin == lastwin)
  36                 return;
  37         }
  38         apply_autocmds(EVENT_WINLEAVE, NULL, NULL, FALSE, curbuf);
  39 !       if (!win_valid(win) || firstwin == lastwin)
  40             return;
  41       }
  42   #endif
  43 ***************
  44 *** 930,936 ****
  45    */
  46       close_buffer(win, win->w_buffer, free_buf, FALSE);
  47       /* autocommands may have closed the window already */
  48 !     if (!win_valid(win))
  49         return;
  50
  51       win_new_height(wp, wp->w_height + win->w_height + win->w_status_height);
  52 --- 930,936 ----
  53    */
  54       close_buffer(win, win->w_buffer, free_buf, FALSE);
  55       /* autocommands may have closed the window already */
  56 !     if (!win_valid(win) || firstwin == lastwin)
  57         return;
  58
  59       win_new_height(wp, wp->w_height + win->w_height + win->w_status_height);
  60 *** ../vim-5.7.22/src/testdir/test13.in Tue Jul 20 13:31:27 1999
  61 --- src/testdir/test13.in       Thu Jan 18 12:59:41 2001
  62 ***************
  63 *** 24,33 ****
  64   otestje3\e
  65   :w
  66   \17\17
  67 ! :au WinLeave testje bdel
  68   \17\17
  69   :w! test.out
  70 ! :au WinLeave testje bdel testje3
  71   :close
  72   :w >>test.out
  73   :e Xtestje1
  74 --- 24,33 ----
  75   otestje3\e
  76   :w
  77   \17\17
  78 ! :au WinLeave Xtestje2 bdel
  79   \17\17
  80   :w! test.out
  81 ! :au WinLeave Xtestje1 bdel Xtestje3
  82   :close
  83   :w >>test.out
  84   :e Xtestje1
  85 *** ../vim-5.7.22/src/testdir/test13.ok Tue Jul 20 13:35:40 1999
  86 --- src/testdir/test13.ok       Thu Jan 18 13:56:54 2001
  87 ***************
  88 *** 5,11 ****
  89         contents
  90   end of testfile
  91   start of testfile
  92 ! testje2
  93         contents
  94         contents
  95         contents
  96 --- 5,11 ----
  97         contents
  98   end of testfile
  99   start of testfile
 100 ! testje1
 101         contents
 102         contents
 103         contents
 104 *** ../vim-5.7.22/src/version.c Mon Jan  8 11:50:35 2001
 105 --- src/version.c       Thu Jan 18 13:54:38 2001
 106 ***************
 107 *** 439,440 ****
 108 --- 439,442 ----
 109   {   /* Add new patch number below this line */
 110 + /**/
 111 +     23,
 112   /**/
 113
 114 --
 115 ERIC IDLE PLAYED: THE DEAD COLLECTOR, MR BINT (A VILLAGE NE'ER-DO -WELL VERY
 116                   KEEN ON BURNING WITCHES), SIR ROBIN, THE GUARD WHO DOESN'T
 117                   HICOUGH BUT TRIES TO GET THINGS STRAIGHT, CONCORDE (SIR
 118                   LAUNCELOT'S TRUSTY STEED), ROGER THE SHRUBBER (A SHRUBBER),
 119                   BROTHER MAYNARD
 120                  "Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD
 121
 122  ///  Bram Moolenaar -- Bram@moolenaar.net -- http://www.moolenaar.net  \\\
 123 (((   Creator of Vim - http://www.vim.org -- ftp://ftp.vim.org/pub/vim   )))
 124  \\\  Help me helping AIDS orphans in Uganda - http://iccf-holland.org  ///