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 / 7.4 / 7.4.421 < prev    next >
Encoding:
Internet Message Format  |  2014-08-29  |  3.4 KB
  1. To: vim_dev@googlegroups.com
  2. Subject: Patch 7.4.421
  3. Fcc: outbox
  4. From: Bram Moolenaar <Bram@moolenaar.net>
  5. Mime-Version: 1.0
  6. Content-Type: text/plain; charset=UTF-8
  7. Content-Transfer-Encoding: 8bit
  8. ------------
  9.  
  10. Patch 7.4.421
  11. Problem:    Crash when searching for "\ze*". (Urtica Dioica)
  12. Solution:   Disallow a multi after \ze and \zs.
  13. Files:        src/regexp_nfa.c, src/testdir/test64.in, src/testdir/test64.ok
  14.  
  15.  
  16. *** ../vim-7.4.420/src/regexp_nfa.c    2014-05-13 19:37:19.489786520 +0200
  17. --- src/regexp_nfa.c    2014-08-29 11:14:12.030416520 +0200
  18. ***************
  19. *** 291,296 ****
  20. --- 291,297 ----
  21.   static int nfa_regconcat __ARGS((void));
  22.   static int nfa_regbranch __ARGS((void));
  23.   static int nfa_reg __ARGS((int paren));
  24. + static int re_mult_next __ARGS((char *what));
  25.   #ifdef DEBUG
  26.   static void nfa_set_code __ARGS((int c));
  27.   static void nfa_postfix_dump __ARGS((char_u *expr, int retval));
  28. ***************
  29. *** 1323,1332 ****
  30. --- 1324,1337 ----
  31.           {
  32.           case 's':
  33.               EMIT(NFA_ZSTART);
  34. +             if (re_mult_next("\\zs") == FAIL)
  35. +             return FAIL;
  36.               break;
  37.           case 'e':
  38.               EMIT(NFA_ZEND);
  39.               nfa_has_zend = TRUE;
  40. +             if (re_mult_next("\\ze") == FAIL)
  41. +             return FAIL;
  42.               break;
  43.   #ifdef FEAT_SYN_HL
  44.           case '1':
  45. ***************
  46. *** 2276,2281 ****
  47. --- 2281,2298 ----
  48.       return OK;
  49.   }
  50.   
  51. + /*
  52. +  * Used in a place where no * or \+ can follow.
  53. +  */
  54. +     static int
  55. + re_mult_next(what)
  56. +     char *what;
  57. + {
  58. +     if (re_multi_type(peekchr()) == MULTI_MULT)
  59. +     EMSG2_RET_FAIL(_("E888: (NFA regexp) cannot repeat %s"), what);
  60. +     return OK;
  61. + }
  62. + 
  63.   #ifdef DEBUG
  64.   static char_u code[50];
  65.   
  66. *** ../vim-7.4.420/src/testdir/test64.in    2014-05-13 16:44:25.633695709 +0200
  67. --- src/testdir/test64.in    2014-08-29 11:33:12.330419011 +0200
  68. ***************
  69. *** 459,465 ****
  70.   :  let text = t[2]
  71.   :  let matchidx = 3
  72.   :  for engine in [0, 1, 2]
  73. ! :    if engine == 2 && re == 0 || engine == 1 && re ==1
  74.   :      continue
  75.   :    endif
  76.   :    let ®expengine = engine
  77. --- 459,465 ----
  78.   :  let text = t[2]
  79.   :  let matchidx = 3
  80.   :  for engine in [0, 1, 2]
  81. ! :    if engine == 2 && re == 0 || engine == 1 && re == 1
  82.   :      continue
  83.   :    endif
  84.   :    let ®expengine = engine
  85. ***************
  86. *** 608,613 ****
  87. --- 608,624 ----
  88.   "ayb20gg/..\%$
  89.   "bybGo"apo"bp:"
  90.   :"
  91. + :" Check for detecting error
  92. + :set regexpengine=2
  93. + :for pat in [' \ze*', ' \zs*']
  94. + :  try
  95. + :    let l = matchlist('x x', pat)
  96. + :    $put ='E888 NOT detected for ' . pat
  97. + :  catch
  98. + :    $put ='E888 detected for ' . pat
  99. + :  endtry
  100. + :endfor
  101. + :"
  102.   :""""" Write the results """""""""""""
  103.   :/\%#=1^Results/,$wq! test.out
  104.   ENDTEST
  105. *** ../vim-7.4.420/src/testdir/test64.ok    2014-05-13 16:44:25.633695709 +0200
  106. --- src/testdir/test64.ok    2014-08-29 11:36:05.782419390 +0200
  107. ***************
  108. *** 1097,1099 ****
  109. --- 1097,1101 ----
  110.   Test END
  111.   EN
  112.   E
  113. + E888 detected for  \ze*
  114. + E888 detected for  \zs*
  115. *** ../vim-7.4.420/src/version.c    2014-08-29 10:04:32.226407390 +0200
  116. --- src/version.c    2014-08-29 11:37:15.794419543 +0200
  117. ***************
  118. *** 743,744 ****
  119. --- 743,746 ----
  120.   {   /* Add new patch number below this line */
  121. + /**/
  122. +     421,
  123.   /**/
  124.  
  125. -- 
  126. From "know your smileys":
  127.  :----}  You lie like Pinocchio
  128.  
  129.  /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
  130. ///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
  131. \\\  an exciting new programming language -- http://www.Zimbu.org        ///
  132.  \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///
  133.