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.0 / 7.0.183 < prev    next >
Encoding:
Internet Message Format  |  2007-01-13  |  4.0 KB
  1. To: vim-dev@vim.org
  2. Subject: patch 7.0.183
  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 7.0.183
  11. Problem:    Crash in ":let" when redirecting to a variable that's being
  12.         displayed. (Thomas Link)
  13. Solution:   When redirecting to a variable only do the assignment when
  14.         stopping redirection to avoid that setting the variable causes a
  15.         freed string to be accessed.
  16. Files:        src/eval.c
  17.  
  18.  
  19. *** ../vim-7.0.182/src/eval.c    Tue Dec  5 10:33:57 2006
  20. --- src/eval.c    Sun Jan 14 14:20:49 2007
  21. ***************
  22. *** 898,903 ****
  23. --- 898,904 ----
  24.   }
  25.   
  26.   static lval_T    *redir_lval = NULL;
  27. + static garray_T redir_ga;    /* only valid when redir_lval is not NULL */
  28.   static char_u    *redir_endp = NULL;
  29.   static char_u    *redir_varname = NULL;
  30.   
  31. ***************
  32. *** 932,937 ****
  33. --- 933,941 ----
  34.       return FAIL;
  35.       }
  36.   
  37. +     /* The output is stored in growarray "redir_ga" until redirection ends. */
  38. +     ga_init2(&redir_ga, (int)sizeof(char), 500);
  39. + 
  40.       /* Parse the variable name (can be a dict or list entry). */
  41.       redir_endp = get_lval(redir_varname, NULL, redir_lval, FALSE, FALSE, FALSE,
  42.                                    FNE_CHECK_START);
  43. ***************
  44. *** 974,1015 ****
  45.   }
  46.   
  47.   /*
  48. !  * Append "value[len]" to the variable set by var_redir_start().
  49.    */
  50.       void
  51. ! var_redir_str(value, len)
  52.       char_u    *value;
  53. !     int        len;
  54.   {
  55. !     char_u    *val;
  56. !     typval_T    tv;
  57. !     int        save_emsg;
  58. !     int        err;
  59.   
  60.       if (redir_lval == NULL)
  61.       return;
  62.   
  63. !     if (len == -1)
  64. !     /* Append the entire string */
  65. !     val = vim_strsave(value);
  66. !     else
  67. !     /* Append only the specified number of characters */
  68. !     val = vim_strnsave(value, len);
  69. !     if (val == NULL)
  70. !     return;
  71. ! 
  72. !     tv.v_type = VAR_STRING;
  73. !     tv.vval.v_string = val;
  74.   
  75. !     save_emsg = did_emsg;
  76. !     did_emsg = FALSE;
  77. !     set_var_lval(redir_lval, redir_endp, &tv, FALSE, (char_u *)".");
  78. !     err = did_emsg;
  79. !     did_emsg |= save_emsg;
  80. !     if (err)
  81.       var_redir_stop();
  82. - 
  83. -     vim_free(tv.vval.v_string);
  84.   }
  85.   
  86.   /*
  87. --- 978,1013 ----
  88.   }
  89.   
  90.   /*
  91. !  * Append "value[value_len]" to the variable set by var_redir_start().
  92. !  * The actual appending is postponed until redirection ends, because the value
  93. !  * appended may in fact be the string we write to, changing it may cause freed
  94. !  * memory to be used:
  95. !  *   :redir => foo
  96. !  *   :let foo
  97. !  *   :redir END
  98.    */
  99.       void
  100. ! var_redir_str(value, value_len)
  101.       char_u    *value;
  102. !     int        value_len;
  103.   {
  104. !     size_t    len;
  105.   
  106.       if (redir_lval == NULL)
  107.       return;
  108.   
  109. !     if (value_len == -1)
  110. !     len = STRLEN(value);    /* Append the entire string */
  111. !     else
  112. !     len = value_len;    /* Append only "value_len" characters */
  113.   
  114. !     if (ga_grow(&redir_ga, (int)len) == OK)
  115. !     {
  116. !     mch_memmove((char *)redir_ga.ga_data + redir_ga.ga_len, value, len);
  117. !     redir_ga.ga_len += len;
  118. !     }
  119. !     else
  120.       var_redir_stop();
  121.   }
  122.   
  123.   /*
  124. ***************
  125. *** 1018,1025 ****
  126. --- 1016,1034 ----
  127.       void
  128.   var_redir_stop()
  129.   {
  130. +     typval_T    tv;
  131. + 
  132.       if (redir_lval != NULL)
  133.       {
  134. +     /* Append the trailing NUL. */
  135. +     ga_append(&redir_ga, NUL);
  136. + 
  137. +     /* Assign the text to the variable. */
  138. +     tv.v_type = VAR_STRING;
  139. +     tv.vval.v_string = redir_ga.ga_data;
  140. +     set_var_lval(redir_lval, redir_endp, &tv, FALSE, (char_u *)".");
  141. +     vim_free(tv.vval.v_string);
  142. + 
  143.       clear_lval(redir_lval);
  144.       vim_free(redir_lval);
  145.       redir_lval = NULL;
  146. *** ../vim-7.0.182/src/version.c    Tue Jan  9 20:29:55 2007
  147. --- src/version.c    Sun Jan 14 15:23:23 2007
  148. ***************
  149. *** 668,669 ****
  150. --- 668,671 ----
  151.   {   /* Add new patch number below this line */
  152. + /**/
  153. +     183,
  154.   /**/
  155.  
  156. -- 
  157. How To Keep A Healthy Level Of Insanity:
  158. 16. Have your coworkers address you by your wrestling name, Rock Hard Kim.
  159.  
  160.  /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
  161. ///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
  162. \\\        download, build and distribute -- http://www.A-A-P.org        ///
  163.  \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///
  164.