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.2 / 7.2.423 < prev    next >
Encoding:
Internet Message Format  |  2010-05-13  |  4.4 KB
  1. To: vim-dev@vim.org
  2. Subject: Patch 7.2.423
  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.2.423
  11. Problem:    Crash when assigning s: to variable. (Yukihiro Nakadaira)
  12. Solution:   Make ga_scripts contain pointer to scriptvar_T instead of
  13.         scriptvar_T itself. (Dominique Pelle)
  14. Files:        src/eval.c
  15.  
  16.  
  17. *** ../vim-7.2.422/src/eval.c    2010-03-17 19:53:44.000000000 +0100
  18. --- src/eval.c    2010-05-14 12:02:16.000000000 +0200
  19. ***************
  20. *** 145,153 ****
  21.       dict_T    sv_dict;
  22.   } scriptvar_T;
  23.   
  24. ! static garray_T        ga_scripts = {0, 0, sizeof(scriptvar_T), 4, NULL};
  25. ! #define SCRIPT_SV(id) (((scriptvar_T *)ga_scripts.ga_data)[(id) - 1])
  26. ! #define SCRIPT_VARS(id) (SCRIPT_SV(id).sv_dict.dv_hashtab)
  27.   
  28.   static int echo_attr = 0;   /* attributes used for ":echo" */
  29.   
  30. --- 145,153 ----
  31.       dict_T    sv_dict;
  32.   } scriptvar_T;
  33.   
  34. ! static garray_T        ga_scripts = {0, 0, sizeof(scriptvar_T *), 4, NULL};
  35. ! #define SCRIPT_SV(id) (((scriptvar_T **)ga_scripts.ga_data)[(id) - 1])
  36. ! #define SCRIPT_VARS(id) (SCRIPT_SV(id)->sv_dict.dv_hashtab)
  37.   
  38.   static int echo_attr = 0;   /* attributes used for ":echo" */
  39.   
  40. ***************
  41. *** 866,875 ****
  42.       hash_init(&vimvarht);  /* garbage_collect() will access it */
  43.       hash_clear(&compat_hashtab);
  44.   
  45. -     /* script-local variables */
  46. -     for (i = 1; i <= ga_scripts.ga_len; ++i)
  47. -     vars_clear(&SCRIPT_VARS(i));
  48. -     ga_clear(&ga_scripts);
  49.       free_scriptnames();
  50.   
  51.       /* global variables */
  52. --- 866,871 ----
  53. ***************
  54. *** 878,883 ****
  55. --- 874,887 ----
  56.       /* autoloaded script names */
  57.       ga_clear_strings(&ga_loaded);
  58.   
  59. +     /* script-local variables */
  60. +     for (i = 1; i <= ga_scripts.ga_len; ++i)
  61. +     {
  62. +     vars_clear(&SCRIPT_VARS(i));
  63. +     vim_free(SCRIPT_SV(i));
  64. +     }
  65. +     ga_clear(&ga_scripts);
  66. + 
  67.       /* unreferenced lists and dicts */
  68.       (void)garbage_collect();
  69.   
  70. ***************
  71. *** 18803,18809 ****
  72.       /* Must be something like "s:", otherwise "ht" would be NULL. */
  73.       switch (varname[-2])
  74.       {
  75. !         case 's': return &SCRIPT_SV(current_SID).sv_var;
  76.           case 'g': return &globvars_var;
  77.           case 'v': return &vimvars_var;
  78.           case 'b': return &curbuf->b_bufvar;
  79. --- 18807,18813 ----
  80.       /* Must be something like "s:", otherwise "ht" would be NULL. */
  81.       switch (varname[-2])
  82.       {
  83. !         case 's': return &SCRIPT_SV(current_SID)->sv_var;
  84.           case 'g': return &globvars_var;
  85.           case 'v': return &vimvars_var;
  86.           case 'b': return &curbuf->b_bufvar;
  87. ***************
  88. *** 18928,18940 ****
  89.           ht = &SCRIPT_VARS(i);
  90.           if (ht->ht_mask == HT_INIT_SIZE - 1)
  91.           ht->ht_array = ht->ht_smallarray;
  92. !         sv = &SCRIPT_SV(i);
  93.           sv->sv_var.di_tv.vval.v_dict = &sv->sv_dict;
  94.       }
  95.   
  96.       while (ga_scripts.ga_len < id)
  97.       {
  98. !         sv = &SCRIPT_SV(ga_scripts.ga_len + 1);
  99.           init_var_dict(&sv->sv_dict, &sv->sv_var);
  100.           ++ga_scripts.ga_len;
  101.       }
  102. --- 18932,18945 ----
  103.           ht = &SCRIPT_VARS(i);
  104.           if (ht->ht_mask == HT_INIT_SIZE - 1)
  105.           ht->ht_array = ht->ht_smallarray;
  106. !         sv = SCRIPT_SV(i);
  107.           sv->sv_var.di_tv.vval.v_dict = &sv->sv_dict;
  108.       }
  109.   
  110.       while (ga_scripts.ga_len < id)
  111.       {
  112. !         sv = SCRIPT_SV(ga_scripts.ga_len + 1) = 
  113. !         (scriptvar_T *)alloc_clear(sizeof(scriptvar_T));
  114.           init_var_dict(&sv->sv_dict, &sv->sv_var);
  115.           ++ga_scripts.ga_len;
  116.       }
  117. ***************
  118. *** 21931,21937 ****
  119.       if (find_viminfo_parameter('!') == NULL)
  120.       return;
  121.   
  122. !     fprintf(fp, _("\n# global variables:\n"));
  123.   
  124.       todo = (int)globvarht.ht_used;
  125.       for (hi = globvarht.ht_array; todo > 0; ++hi)
  126. --- 21936,21942 ----
  127.       if (find_viminfo_parameter('!') == NULL)
  128.       return;
  129.   
  130. !     fputs(_("\n# global variables:\n"), fp);
  131.   
  132.       todo = (int)globvarht.ht_used;
  133.       for (hi = globvarht.ht_array; todo > 0; ++hi)
  134. *** ../vim-7.2.422/src/version.c    2010-05-13 17:46:53.000000000 +0200
  135. --- src/version.c    2010-05-14 12:13:19.000000000 +0200
  136. ***************
  137. *** 683,684 ****
  138. --- 683,686 ----
  139.   {   /* Add new patch number below this line */
  140. + /**/
  141. +     423,
  142.   /**/
  143.  
  144. -- 
  145. He who laughs last, thinks slowest.
  146.  
  147.  /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
  148. ///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
  149. \\\        download, build and distribute -- http://www.A-A-P.org        ///
  150.  \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///
  151.