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 / 6.1.145 < prev    next >
Encoding:
Internet Message Format  |  2002-08-02  |  6.7 KB
  1. To: vim-dev@vim.org
  2. Subject: Patch 6.1.145
  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 6.1.145
  11. Problem:    GTK: Drag&drop with more than 3 files may cause a crash. (Mickael
  12.         Marchand)
  13. Solution:   Rewrite the code that parses the received list of files to be more
  14.         robust.
  15. Files:        src/charset.c, src/gui_gtk_x11.c
  16.  
  17.  
  18. *** ../vim61.144/src/charset.c    Sat Aug  3 14:05:15 2002
  19. --- src/charset.c    Tue Jul 30 19:28:32 2002
  20. ***************
  21. *** 1558,1564 ****
  22.       return c - '0';
  23.   }
  24.   
  25. ! #if defined(FEAT_TERMRESPONSE) || defined(PROTO)
  26.   /*
  27.    * Convert two hex characters to a byte.
  28.    * Return -1 if one of the characters is not hex.
  29. --- 1558,1565 ----
  30.       return c - '0';
  31.   }
  32.   
  33. ! #if defined(FEAT_TERMRESPONSE) \
  34. !     || (defined(FEAT_GUI_GTK) && defined(FEAT_WINDOWS)) || defined(PROTO)
  35.   /*
  36.    * Convert two hex characters to a byte.
  37.    * Return -1 if one of the characters is not hex.
  38. *** ../vim61.144/src/gui_gtk_x11.c    Sat May  4 22:23:07 2002
  39. --- src/gui_gtk_x11.c    Thu Jul 18 21:20:54 2002
  40. ***************
  41. *** 1400,1408 ****
  42.       int        redo_dirs = FALSE;
  43.       int        i;
  44.       int        n;
  45. !     char    *start;
  46. !     char    *stop;
  47. !     char    *copy;
  48.       int        nfiles;
  49.       int        url = FALSE;
  50.       GdkModifierType current_modifiers;
  51. --- 1400,1408 ----
  52.       int        redo_dirs = FALSE;
  53.       int        i;
  54.       int        n;
  55. !     char_u    *start;
  56. !     char_u    *copy;
  57. !     char_u    *names = data->data;
  58.       int        nfiles;
  59.       int        url = FALSE;
  60.       GdkModifierType current_modifiers;
  61. ***************
  62. *** 1415,1495 ****
  63.       /* guard against trash */
  64.       if (data->length <= 0
  65.           || data->format != 8
  66. !         || ((char *)data->data)[data->length] != '\0')
  67.       {
  68.       gtk_drag_finish(context, FALSE, FALSE, time);
  69.       return;
  70.       }
  71.   
  72. !     /* Count how many items there may be and normalize delimiters. */
  73. !     n = 1;
  74. !     copy = strdup((char *)data->data);
  75.       for (i = 0; i < data->length; ++i)
  76.       {
  77. !     if (copy[i] == '\n')
  78. !         ++n;
  79. !     else if (copy[i] == '\r')
  80.       {
  81. !         copy[i] = '\n';
  82. !         ++n;
  83.       }
  84.       }
  85.   
  86. !     fnames = (char_u **)alloc((n + 1) * sizeof(char_u *));
  87.   
  88.       start = copy;
  89. !     stop = copy;
  90. !     nfiles = 0;
  91. !     for (i = 0; i < n; ++i)
  92.       {
  93. !     stop = strchr(start, '\n');
  94. !     if (stop != NULL)
  95. !         *stop = '\0';
  96. ! 
  97. !     if (strlen(start) == 0)
  98. !         continue;
  99. ! 
  100. !     if (strncmp(start, "http://", 7) == 0
  101. !         || strncmp(start, "ftp://", 6) == 0)
  102. !     {
  103. !         url = TRUE;
  104. !     }
  105. !     else if (strncmp(start, "file:", 5) != 0)
  106. !     {
  107. !         int j;
  108. ! 
  109. !         free(copy);
  110. !         for (j = 0; j < nfiles; ++j)
  111. !         free(fnames[j]);
  112. !         gtk_drag_finish(context, FALSE, FALSE, time);
  113. ! 
  114. !         return;
  115. !     }
  116. ! 
  117. !     if (strncmp(start, "file://localhost", 16) == 0)
  118. !     {
  119. !         fnames[nfiles] = (char_u *)strdup(start + 16);
  120. !         ++nfiles;
  121. !     }
  122.       else
  123.       {
  124. !         if (url == FALSE)
  125.           {
  126.           start += 5;
  127.           while (start[0] == '/' && start[1] == '/')
  128.               ++start;
  129.           }
  130. -         fnames[nfiles] = (char_u *)strdup(start);
  131. -         ++nfiles;
  132.       }
  133. !     start = stop + 2;
  134.       }
  135. -     free(copy);
  136.   
  137.       /* accept */
  138.       gtk_drag_finish(context, TRUE, FALSE, time);
  139.   
  140. !     /* Real files (i.e. not http and not ftp) */
  141.       if (url == FALSE)
  142.       {
  143.       if (nfiles == 1)
  144. --- 1415,1485 ----
  145.       /* guard against trash */
  146.       if (data->length <= 0
  147.           || data->format != 8
  148. !         || names[data->length] != '\0')
  149.       {
  150.       gtk_drag_finish(context, FALSE, FALSE, time);
  151.       return;
  152.       }
  153.   
  154. !     /* Count how many items there may be and separate them with a NUL.
  155. !      * Apparently the items are separated with \r\n.  This is not documented,
  156. !      * thus be careful not to go past the end.  Also allow separation with NUL
  157. !      * characters. */
  158. !     nfiles = 0;
  159. !     copy = alloc((unsigned)(data->length + 1));
  160. !     start = copy;
  161.       for (i = 0; i < data->length; ++i)
  162.       {
  163. !     if (names[i] == NUL || names[i] == '\n' || names[i] == '\r')
  164. !     {
  165. !         if (start > copy && start[-1] != NUL)
  166. !         {
  167. !         ++nfiles;
  168. !         *start++ = NUL;
  169. !         }
  170. !     }
  171. !     else if (names[i] == '%' && i + 2 < data->length
  172. !                           && hexhex2nr(names + i + 1) > 0)
  173.       {
  174. !         *start++ = hexhex2nr(names + i + 1);
  175. !         i += 2;
  176.       }
  177. +     else
  178. +         *start++ = names[i];
  179. +     }
  180. +     if (start > copy && start[-1] != NUL)
  181. +     {
  182. +     *start = NUL;   /* last item didn't have \r or \n */
  183. +     ++nfiles;
  184.       }
  185.   
  186. !     fnames = (char_u **)alloc((unsigned)(nfiles * sizeof(char_u *)));
  187.   
  188. +     url = FALSE;    /* Set when a non-file URL was found. */
  189.       start = copy;
  190. !     for (n = 0; n < nfiles; ++n)
  191.       {
  192. !     if (STRNCMP(start, "file://localhost", 16) == 0)
  193. !         start += 16;
  194.       else
  195.       {
  196. !         if (STRNCMP(start, "file:", 5) != 0)
  197. !         url = TRUE;
  198. !         else
  199.           {
  200.           start += 5;
  201.           while (start[0] == '/' && start[1] == '/')
  202.               ++start;
  203.           }
  204.       }
  205. !     fnames[n] = vim_strsave(start);
  206. !     start += STRLEN(start) + 1;
  207.       }
  208.   
  209.       /* accept */
  210.       gtk_drag_finish(context, TRUE, FALSE, time);
  211.   
  212. !     /* Special handling when all items are real files. */
  213.       if (url == FALSE)
  214.       {
  215.       if (nfiles == 1)
  216. ***************
  217. *** 1499,1505 ****
  218.           /* Handle dropping a directory on Vim. */
  219.           if (mch_chdir((char *)fnames[0]) == 0)
  220.           {
  221. !             free(fnames[0]);
  222.               fnames[0] = NULL;
  223.               redo_dirs = TRUE;
  224.           }
  225. --- 1489,1495 ----
  226.           /* Handle dropping a directory on Vim. */
  227.           if (mch_chdir((char *)fnames[0]) == 0)
  228.           {
  229. !             vim_free(fnames[0]);
  230.               fnames[0] = NULL;
  231.               redo_dirs = TRUE;
  232.           }
  233. ***************
  234. *** 1543,1548 ****
  235. --- 1533,1539 ----
  236.               }
  237.       }
  238.       }
  239. +     vim_free(copy);
  240.   
  241.       /* Handle the drop, :edit or :split to get to the file */
  242.       handle_drop(nfiles, fnames, current_modifiers & GDK_CONTROL_MASK);
  243. ***************
  244. *** 1552,1560 ****
  245.   
  246.       /* Update the screen display */
  247.       update_screen(NOT_VALID);
  248. ! #ifdef FEAT_MENU
  249.       gui_update_menus(0);
  250. ! #endif
  251.       setcursor();
  252.       out_flush();
  253.       gui_update_cursor(FALSE, FALSE);
  254. --- 1543,1551 ----
  255.   
  256.       /* Update the screen display */
  257.       update_screen(NOT_VALID);
  258. ! # ifdef FEAT_MENU
  259.       gui_update_menus(0);
  260. ! # endif
  261.       setcursor();
  262.       out_flush();
  263.       gui_update_cursor(FALSE, FALSE);
  264. *** ../vim61.144/src/version.c    Sat Aug  3 14:05:15 2002
  265. --- src/version.c    Sat Aug  3 14:09:44 2002
  266. ***************
  267. *** 608,609 ****
  268. --- 608,611 ----
  269.   {   /* Add new patch number below this line */
  270. + /**/
  271. +     145,
  272.   /**/
  273.  
  274. -- 
  275. From "know your smileys":
  276.  :'-D    Laughing so much that they're crying
  277.  
  278.  ///  Bram Moolenaar -- Bram@moolenaar.net -- http://www.moolenaar.net  \\\
  279. ///   Creator of Vim -- http://vim.sf.net -- ftp://ftp.vim.org/pub/vim   \\\
  280. \\\           Project leader for A-A-P -- http://www.a-a-p.org           ///
  281.  \\\ Lord Of The Rings helps Uganda - http://iccf-holland.org/lotr.html ///
  282.