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.238 < prev    next >
Encoding:
Internet Message Format  |  2009-07-21  |  3.8 KB
  1. To: vim-dev@vim.org
  2. Subject: Patch 7.2.238
  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.238
  11. Problem:    Leaking memory when setting term to "builtin_dumb".
  12. Solution:   Free memory when resetting term option t_Co.
  13. Files:        src/option.c, src/proto/option.pro, src/term.c
  14.  
  15.  
  16. *** ../vim-7.2.237/src/option.c    2009-06-16 17:50:56.000000000 +0200
  17. --- src/option.c    2009-07-22 12:49:19.000000000 +0200
  18. ***************
  19. *** 403,410 ****
  20.   #define P_NUM        0x02    /* the option is numeric */
  21.   #define P_STRING    0x04    /* the option is a string */
  22.   #define P_ALLOCED    0x08    /* the string option is in allocated memory,
  23. !                     must use vim_free() when assigning new
  24. !                     value. Not set if default is the same. */
  25.   #define P_EXPAND    0x10    /* environment expansion.  NOTE: P_EXPAND can
  26.                      never be used for local or hidden options! */
  27.   #define P_NODEFAULT    0x40    /* don't set to default value */
  28. --- 403,411 ----
  29.   #define P_NUM        0x02    /* the option is numeric */
  30.   #define P_STRING    0x04    /* the option is a string */
  31.   #define P_ALLOCED    0x08    /* the string option is in allocated memory,
  32. !                    must use free_string_option() when
  33. !                    assigning new value. Not set if default is
  34. !                    the same. */
  35.   #define P_EXPAND    0x10    /* environment expansion.  NOTE: P_EXPAND can
  36.                      never be used for local or hidden options! */
  37.   #define P_NODEFAULT    0x40    /* don't set to default value */
  38. ***************
  39. *** 8927,8932 ****
  40. --- 8928,8955 ----
  41.   }
  42.   
  43.   /*
  44. +  * Free the string for one term option, if it was allocated.
  45. +  * Set the string to empty_option and clear allocated flag.
  46. +  * "var" points to the option value.
  47. +  */
  48. +     void
  49. + free_one_termoption(var)
  50. +     char_u *var;
  51. + {
  52. +     struct vimoption   *p;
  53. + 
  54. +     for (p = &options[0]; p->fullname != NULL; p++)
  55. +     if (p->var == var)
  56. +     {
  57. +         if (p->flags & P_ALLOCED)
  58. +         free_string_option(*(char_u **)(p->var));
  59. +         *(char_u **)(p->var) = empty_option;
  60. +         p->flags &= ~P_ALLOCED;
  61. +         break;
  62. +     }
  63. + }
  64. + 
  65. + /*
  66.    * Set the terminal option defaults to the current value.
  67.    * Used after setting the terminal name.
  68.    */
  69. *** ../vim-7.2.237/src/proto/option.pro    2009-02-21 20:27:00.000000000 +0100
  70. --- src/proto/option.pro    2009-07-22 12:52:31.000000000 +0200
  71. ***************
  72. *** 29,34 ****
  73. --- 29,35 ----
  74.   int makefoldset __ARGS((FILE *fd));
  75.   void clear_termoptions __ARGS((void));
  76.   void free_termoptions __ARGS((void));
  77. + void free_one_termoption __ARGS((char_u *var));
  78.   void set_term_defaults __ARGS((void));
  79.   void comp_col __ARGS((void));
  80.   char_u *get_equalprg __ARGS((void));
  81. *** ../vim-7.2.237/src/term.c    2009-06-16 14:31:56.000000000 +0200
  82. --- src/term.c    2009-07-22 13:19:59.000000000 +0200
  83. ***************
  84. *** 2881,2887 ****
  85.   
  86.       /* if 'Sb' and 'AB' are not defined, reset "Co" */
  87.       if (*T_CSB == NUL && *T_CAB == NUL)
  88. !         T_CCO = empty_option;
  89.   
  90.       /* Set 'weirdinvert' according to value of 't_xs' */
  91.       p_wiv = (*T_XS != NUL);
  92. --- 2881,2887 ----
  93.   
  94.       /* if 'Sb' and 'AB' are not defined, reset "Co" */
  95.       if (*T_CSB == NUL && *T_CAB == NUL)
  96. !         free_one_termoption(T_CCO);
  97.   
  98.       /* Set 'weirdinvert' according to value of 't_xs' */
  99.       p_wiv = (*T_XS != NUL);
  100. *** ../vim-7.2.237/src/version.c    2009-07-22 13:27:50.000000000 +0200
  101. --- src/version.c    2009-07-22 14:25:44.000000000 +0200
  102. ***************
  103. *** 678,679 ****
  104. --- 678,681 ----
  105.   {   /* Add new patch number below this line */
  106. + /**/
  107. +     238,
  108.   /**/
  109.  
  110. -- 
  111. hundred-and-one symptoms of being an internet addict:
  112. 95. Only communication in your household is through email.
  113.  
  114.  /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
  115. ///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
  116. \\\        download, build and distribute -- http://www.A-A-P.org        ///
  117.  \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///
  118.