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.3 / 7.3.1038 < prev    next >
Encoding:
Internet Message Format  |  2013-05-28  |  4.4 KB
  1. To: vim_dev@googlegroups.com
  2. Subject: Patch 7.3.1038
  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.3.1038
  11. Problem:    Crash when using Cscope.
  12. Solution:   Avoid negative argument to vim_strncpy(). (Narendran
  13.         Gopalakrishnan)
  14. Files:        src/if_cscope.c
  15.  
  16.  
  17. *** ../vim-7.3.1037/src/if_cscope.c    2013-05-06 04:21:35.000000000 +0200
  18. --- src/if_cscope.c    2013-05-29 19:12:55.000000000 +0200
  19. ***************
  20. *** 2460,2472 ****
  21.   /*
  22.    * PRIVATE: cs_resolve_file
  23.    *
  24. !  * construct the full pathname to a file found in the cscope database.
  25.    * (Prepends ppath, if there is one and if it's not already prepended,
  26.    * otherwise just uses the name found.)
  27.    *
  28. !  * we need to prepend the prefix because on some cscope's (e.g., the one that
  29.    * ships with Solaris 2.6), the output never has the prefix prepended.
  30. !  * contrast this with my development system (Digital Unix), which does.
  31.    */
  32.       static char *
  33.   cs_resolve_file(i, name)
  34. --- 2460,2472 ----
  35.   /*
  36.    * PRIVATE: cs_resolve_file
  37.    *
  38. !  * Construct the full pathname to a file found in the cscope database.
  39.    * (Prepends ppath, if there is one and if it's not already prepended,
  40.    * otherwise just uses the name found.)
  41.    *
  42. !  * We need to prepend the prefix because on some cscope's (e.g., the one that
  43.    * ships with Solaris 2.6), the output never has the prefix prepended.
  44. !  * Contrast this with my development system (Digital Unix), which does.
  45.    */
  46.       static char *
  47.   cs_resolve_file(i, name)
  48. ***************
  49. *** 2493,2506 ****
  50.       if (csdir != NULL)
  51.       {
  52.           vim_strncpy(csdir, (char_u *)csinfo[i].fname,
  53. !             gettail((char_u *)csinfo[i].fname) - 1 - (char_u *)csinfo[i].fname);
  54.           len += (int)STRLEN(csdir);
  55.       }
  56.       }
  57.   
  58. -     if ((fullname = (char *)alloc(len)) == NULL)
  59. -     return NULL;
  60. - 
  61.       /* Note/example: this won't work if the cscope output already starts
  62.        * "../.." and the prefix path is also "../..".  if something like this
  63.        * happens, you are screwed up and need to fix how you're using cscope. */
  64. --- 2493,2504 ----
  65.       if (csdir != NULL)
  66.       {
  67.           vim_strncpy(csdir, (char_u *)csinfo[i].fname,
  68. !                                gettail((char_u *)csinfo[i].fname)
  69. !                          - (char_u *)csinfo[i].fname);
  70.           len += (int)STRLEN(csdir);
  71.       }
  72.       }
  73.   
  74.       /* Note/example: this won't work if the cscope output already starts
  75.        * "../.." and the prefix path is also "../..".  if something like this
  76.        * happens, you are screwed up and need to fix how you're using cscope. */
  77. ***************
  78. *** 2511,2526 ****
  79.           && name[0] != '\\' && name[1] != ':'
  80.   #endif
  81.          )
  82. !     (void)sprintf(fullname, "%s/%s", csinfo[i].ppath, name);
  83. !     else if (csdir != NULL && csinfo[i].fname != NULL && STRLEN(csdir) > 0)
  84.       {
  85.       /* Check for csdir to be non empty to avoid empty path concatenated to
  86. !      * cscope output. TODO: avoid the unnecessary alloc/free of fullname. */
  87. !     vim_free(fullname);
  88.       fullname = (char *)concat_fnames(csdir, (char_u *)name, TRUE);
  89.       }
  90.       else
  91. !     (void)sprintf(fullname, "%s", name);
  92.   
  93.       vim_free(csdir);
  94.       return fullname;
  95. --- 2509,2528 ----
  96.           && name[0] != '\\' && name[1] != ':'
  97.   #endif
  98.          )
  99. !     {
  100. !     if ((fullname = (char *)alloc(len)) != NULL)
  101. !         (void)sprintf(fullname, "%s/%s", csinfo[i].ppath, name);
  102. !     }
  103. !     else if (csdir != NULL && csinfo[i].fname != NULL && *csdir != NUL)
  104.       {
  105.       /* Check for csdir to be non empty to avoid empty path concatenated to
  106. !      * cscope output. */
  107.       fullname = (char *)concat_fnames(csdir, (char_u *)name, TRUE);
  108.       }
  109.       else
  110. !     {
  111. !     fullname = (char *)vim_strsave((char_u *)name);
  112. !     }
  113.   
  114.       vim_free(csdir);
  115.       return fullname;
  116. *** ../vim-7.3.1037/src/version.c    2013-05-29 18:45:07.000000000 +0200
  117. --- src/version.c    2013-05-29 19:17:16.000000000 +0200
  118. ***************
  119. *** 730,731 ****
  120. --- 730,733 ----
  121.   {   /* Add new patch number below this line */
  122. + /**/
  123. +     1038,
  124.   /**/
  125.  
  126. -- 
  127. hundred-and-one symptoms of being an internet addict:
  128. 12. You turn off your modem and get this awful empty feeling, like you just
  129.     pulled the plug on a loved one.
  130.  
  131.  /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
  132. ///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
  133. \\\  an exciting new programming language -- http://www.Zimbu.org        ///
  134.  \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///
  135.