home *** CD-ROM | disk | FTP | other *** search
/ vim.ftp.fu-berlin.de / 2015-02-03.vim.ftp.fu-berlin.de.tar / vim.ftp.fu-berlin.de / patches / 6.1.462 < prev    next >
Encoding:
Internet Message Format  |  2003-04-12  |  10.0 KB
  1. To: vim-dev@vim.org
  2. Subject: Patch 6.1.462
  3. Fcc: outbox
  4. From: Bram Moolenaar <Bram@moolenaar.net>
  5. Mime-Version: 1.0
  6. Content-Type: text/plain; charset=ISO-8859-1
  7. Content-Transfer-Encoding: 8bit
  8. ------------
  9.  
  10. Patch 6.1.462
  11. Problem:    When autocommands wipe out a buffer, a crash may happen. (Hari
  12.         Krishna Dara)
  13. Solution:   Don't decrement the window count of a buffer before calling the
  14.         autocommands for it.  When re-using the current buffer, watch out
  15.         for autocommands changing the current buffer.
  16. Files:        src/buffer.c, src/ex_cmds.c, src/proto/buffer.pro
  17.  
  18.  
  19. *** ../vim61.461/src/buffer.c    Sun Apr  6 15:22:34 2003
  20. --- src/buffer.c    Thu Apr 10 21:48:43 2003
  21. ***************
  22. *** 308,324 ****
  23.       unload_buf = TRUE;
  24.   #endif
  25.   
  26. -     /* decrease the link count from windows (unless not in any window) */
  27. -     if (buf->b_nwindows > 0)
  28. -     --buf->b_nwindows;
  29. - 
  30.       if (win != NULL)
  31.       {
  32.       /* Set b_last_cursor when closing the last window for the buffer.
  33.        * Remember the last cursor position and window options of the buffer.
  34.        * This used to be only for the current window, but then options like
  35.        * 'foldmethod' may be lost with a ":only" command. */
  36. !     if (buf->b_nwindows == 0)
  37.           set_last_cursor(win);
  38.       buflist_setfpos(buf, win,
  39.               win->w_cursor.lnum == 1 ? 0 : win->w_cursor.lnum,
  40. --- 308,320 ----
  41.       unload_buf = TRUE;
  42.   #endif
  43.   
  44.       if (win != NULL)
  45.       {
  46.       /* Set b_last_cursor when closing the last window for the buffer.
  47.        * Remember the last cursor position and window options of the buffer.
  48.        * This used to be only for the current window, but then options like
  49.        * 'foldmethod' may be lost with a ":only" command. */
  50. !     if (buf->b_nwindows == 1)
  51.           set_last_cursor(win);
  52.       buflist_setfpos(buf, win,
  53.               win->w_cursor.lnum == 1 ? 0 : win->w_cursor.lnum,
  54. ***************
  55. *** 327,333 ****
  56.   
  57.   #ifdef FEAT_AUTOCMD
  58.       /* When the buffer is no longer in a window, trigger BufWinLeave */
  59. !     if (buf->b_nwindows == 0 && nwindows > 0)
  60.       {
  61.       apply_autocmds(EVENT_BUFWINLEAVE, buf->b_fname, buf->b_fname,
  62.                                     FALSE, buf);
  63. --- 323,329 ----
  64.   
  65.   #ifdef FEAT_AUTOCMD
  66.       /* When the buffer is no longer in a window, trigger BufWinLeave */
  67. !     if (buf->b_nwindows == 1)
  68.       {
  69.       apply_autocmds(EVENT_BUFWINLEAVE, buf->b_fname, buf->b_fname,
  70.                                     FALSE, buf);
  71. ***************
  72. *** 344,351 ****
  73. --- 340,352 ----
  74.           return;
  75.       }
  76.       }
  77. +     nwindows = buf->b_nwindows;
  78.   #endif
  79.   
  80. +     /* decrease the link count from windows (unless not in any window) */
  81. +     if (buf->b_nwindows > 0)
  82. +     --buf->b_nwindows;
  83. + 
  84.       /* Return when a window is displaying the buffer or when it's not
  85.        * unloaded. */
  86.       if (buf->b_nwindows > 0 || !unload_buf)
  87. ***************
  88. *** 364,385 ****
  89.        * Also calls the "BufDelete" autocommands when del_buf is TRUE.
  90.        */
  91.   #ifdef FEAT_AUTOCMD
  92.       is_curbuf = (buf == curbuf);
  93.   #endif
  94. !     buf_freeall(buf, del_buf);
  95.   #ifdef FEAT_AUTOCMD
  96. !     if (wipe_buf && buf_valid(buf))
  97. !     apply_autocmds(EVENT_BUFWIPEOUT, buf->b_fname, buf->b_fname,
  98. !                                   FALSE, buf);
  99.       /*
  100. -      * Autocommands may have deleted the buffer.
  101.        * It's possible that autocommands change curbuf to the one being deleted.
  102.        * This might cause the previous curbuf to be deleted unexpectedly.  But
  103.        * in some cases it's OK to delete the curbuf, because a new one is
  104.        * obtained anyway.  Therefore only return if curbuf changed to the
  105.        * deleted buffer.
  106.        */
  107. !     if (!buf_valid(buf) || (buf == curbuf && !is_curbuf))
  108.       return;
  109.   #endif
  110.   
  111. --- 365,396 ----
  112.        * Also calls the "BufDelete" autocommands when del_buf is TRUE.
  113.        */
  114.   #ifdef FEAT_AUTOCMD
  115. +     /* Remember if we are closing the current buffer.  Restore the number of
  116. +      * windows, so that autocommands in buf_freeall() don't get confused. */
  117.       is_curbuf = (buf == curbuf);
  118. +     buf->b_nwindows = nwindows;
  119.   #endif
  120. ! 
  121. !     buf_freeall(buf, del_buf, wipe_buf);
  122. ! 
  123.   #ifdef FEAT_AUTOCMD
  124. !     /* Autocommands may have deleted the buffer. */
  125. !     if (!buf_valid(buf))
  126. !     return;
  127. ! 
  128. !     /* Autocommands may have opened or closed windows for this buffer.
  129. !      * Decrement the count for the close we do here. */
  130. !     if (buf->b_nwindows > 0)
  131. !     --buf->b_nwindows;
  132. ! 
  133.       /*
  134.        * It's possible that autocommands change curbuf to the one being deleted.
  135.        * This might cause the previous curbuf to be deleted unexpectedly.  But
  136.        * in some cases it's OK to delete the curbuf, because a new one is
  137.        * obtained anyway.  Therefore only return if curbuf changed to the
  138.        * deleted buffer.
  139.        */
  140. !     if (buf == curbuf && !is_curbuf)
  141.       return;
  142.   #endif
  143.   
  144. ***************
  145. *** 467,475 ****
  146.    */
  147.   /*ARGSUSED*/
  148.       void
  149. ! buf_freeall(buf, del_buf)
  150.       buf_T    *buf;
  151.       int        del_buf;    /* buffer is going to be deleted */
  152.   {
  153.   #ifdef FEAT_AUTOCMD
  154.       int        is_curbuf = (buf == curbuf);
  155. --- 478,487 ----
  156.    */
  157.   /*ARGSUSED*/
  158.       void
  159. ! buf_freeall(buf, del_buf, wipe_buf)
  160.       buf_T    *buf;
  161.       int        del_buf;    /* buffer is going to be deleted */
  162. +     int        wipe_buf;    /* buffer is going to be wiped out */
  163.   {
  164.   #ifdef FEAT_AUTOCMD
  165.       int        is_curbuf = (buf == curbuf);
  166. ***************
  167. *** 483,488 ****
  168. --- 495,508 ----
  169.       if (!buf_valid(buf))        /* autocommands may delete the buffer */
  170.           return;
  171.       }
  172. +     if (wipe_buf)
  173. +     {
  174. +     apply_autocmds(EVENT_BUFWIPEOUT, buf->b_fname, buf->b_fname,
  175. +                                   FALSE, buf);
  176. +     if (!buf_valid(buf))        /* autocommands may delete the buffer */
  177. +         return;
  178. +     }
  179. + 
  180.       /*
  181.        * It's possible that autocommands change curbuf to the one being deleted.
  182.        * This might cause curbuf to be deleted unexpectedly.  But in some cases
  183. ***************
  184. *** 1320,1325 ****
  185. --- 1340,1346 ----
  186.        *
  187.        * This is the ONLY place where a new buffer structure is allocated!
  188.        */
  189. +     buf = NULL;
  190.       if ((flags & BLN_CURBUF)
  191.           && curbuf != NULL
  192.           && curbuf->b_ffname == NULL
  193. ***************
  194. *** 1328,1345 ****
  195.       {
  196.       buf = curbuf;
  197.   #ifdef FEAT_AUTOCMD
  198. !     /* It's like this buffer is deleted. */
  199.       if (curbuf->b_p_bl)
  200.           apply_autocmds(EVENT_BUFDELETE, NULL, NULL, FALSE, curbuf);
  201. !     apply_autocmds(EVENT_BUFWIPEOUT, NULL, NULL, FALSE, curbuf);
  202.   #endif
  203.   #ifdef FEAT_QUICKFIX
  204. !     /* Make sure 'bufhidden' and 'buftype' are empty */
  205. !     clear_string_option(&buf->b_p_bh);
  206. !     clear_string_option(&buf->b_p_bt);
  207.   #endif
  208.       }
  209. !     else
  210.       {
  211.       buf = (buf_T *)alloc_clear((unsigned)sizeof(buf_T));
  212.       if (buf == NULL)
  213. --- 1349,1373 ----
  214.       {
  215.       buf = curbuf;
  216.   #ifdef FEAT_AUTOCMD
  217. !     /* It's like this buffer is deleted.  Watch out for autocommands that
  218. !      * change curbuf!  If that happens, allocate a new buffer anyway. */
  219.       if (curbuf->b_p_bl)
  220.           apply_autocmds(EVENT_BUFDELETE, NULL, NULL, FALSE, curbuf);
  221. !     if (buf == curbuf)
  222. !         apply_autocmds(EVENT_BUFWIPEOUT, NULL, NULL, FALSE, curbuf);
  223.   #endif
  224.   #ifdef FEAT_QUICKFIX
  225. ! # ifdef FEAT_AUTOCMD
  226. !     if (buf == curbuf)
  227. ! # endif
  228. !     {
  229. !         /* Make sure 'bufhidden' and 'buftype' are empty */
  230. !         clear_string_option(&buf->b_p_bh);
  231. !         clear_string_option(&buf->b_p_bt);
  232. !     }
  233.   #endif
  234.       }
  235. !     if (buf != curbuf || curbuf == NULL)
  236.       {
  237.       buf = (buf_T *)alloc_clear((unsigned)sizeof(buf_T));
  238.       if (buf == NULL)
  239. ***************
  240. *** 1372,1378 ****
  241.   
  242.       if (buf == curbuf)
  243.       {
  244. !     buf_freeall(buf, FALSE); /* free all things allocated for this buffer */
  245.       if (buf != curbuf)     /* autocommands deleted the buffer! */
  246.           return NULL;
  247.       /* buf->b_nwindows = 0; why was this here? */
  248. --- 1400,1407 ----
  249.   
  250.       if (buf == curbuf)
  251.       {
  252. !     /* free all things allocated for this buffer */
  253. !     buf_freeall(buf, FALSE, FALSE);
  254.       if (buf != curbuf)     /* autocommands deleted the buffer! */
  255.           return NULL;
  256.       /* buf->b_nwindows = 0; why was this here? */
  257. *** ../vim61.461/src/ex_cmds.c    Sun Apr 13 20:05:37 2003
  258. --- src/ex_cmds.c    Fri Apr 11 10:50:00 2003
  259. ***************
  260. *** 2833,2839 ****
  261.       else
  262.           new_name = NULL;
  263.   #endif
  264. !     buf_freeall(curbuf, FALSE);    /* free all things for buffer */
  265.   #ifdef FEAT_AUTOCMD
  266.       /* If autocommands deleted the buffer we were going to re-edit, give
  267.        * up and jump to the end. */
  268. --- 2833,2839 ----
  269.       else
  270.           new_name = NULL;
  271.   #endif
  272. !     buf_freeall(curbuf, FALSE, FALSE);   /* free all things for buffer */
  273.   #ifdef FEAT_AUTOCMD
  274.       /* If autocommands deleted the buffer we were going to re-edit, give
  275.        * up and jump to the end. */
  276. *** ../vim61.461/src/proto/buffer.pro    Sun Mar  9 17:49:38 2003
  277. --- src/proto/buffer.pro    Thu Apr 10 21:19:22 2003
  278. ***************
  279. *** 3,9 ****
  280.   int buf_valid __ARGS((buf_T *buf));
  281.   void close_buffer __ARGS((win_T *win, buf_T *buf, int action));
  282.   void buf_clear_file __ARGS((buf_T *buf));
  283. ! void buf_freeall __ARGS((buf_T *buf, int del_buf));
  284.   void goto_buffer __ARGS((exarg_T *eap, int start, int dir, int count));
  285.   void handle_swap_exists __ARGS((buf_T *old_curbuf));
  286.   char_u *do_bufdel __ARGS((int command, char_u *arg, int addr_count, int start_bnr, int end_bnr, int forceit));
  287. --- 3,9 ----
  288.   int buf_valid __ARGS((buf_T *buf));
  289.   void close_buffer __ARGS((win_T *win, buf_T *buf, int action));
  290.   void buf_clear_file __ARGS((buf_T *buf));
  291. ! void buf_freeall __ARGS((buf_T *buf, int del_buf, int wipe_buf));
  292.   void goto_buffer __ARGS((exarg_T *eap, int start, int dir, int count));
  293.   void handle_swap_exists __ARGS((buf_T *old_curbuf));
  294.   char_u *do_bufdel __ARGS((int command, char_u *arg, int addr_count, int start_bnr, int end_bnr, int forceit));
  295. *** ../vim61.461/src/version.c    Sun Apr 13 20:37:44 2003
  296. --- src/version.c    Sun Apr 13 20:39:40 2003
  297. ***************
  298. *** 613,614 ****
  299. --- 613,616 ----
  300.   {   /* Add new patch number below this line */
  301. + /**/
  302. +     462,
  303.   /**/
  304.  
  305. -- 
  306. OLD WOMAN: King of the WHO?
  307. ARTHUR:    The Britons.
  308. OLD WOMAN: Who are the Britons?
  309.                  "Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD
  310.  
  311.  /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
  312. ///          Creator of Vim - Vi IMproved -- http://www.Vim.org          \\\
  313. \\\              Project leader for A-A-P -- http://www.A-A-P.org        ///
  314.  \\\     Help AIDS victims, buy at Amazon -- http://ICCF.nl/click1.html ///
  315.