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.060 < prev    next >
Encoding:
Internet Message Format  |  2012-11-20  |  5.2 KB
  1. To: vim_dev@googlegroups.com
  2. Subject: Patch 7.3.060
  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.060
  11. Problem:    Netbeans: crash when socket is disconnected unexpectedly.
  12. Solution:   Don't cleanup when a read fails, put a message in the queue and
  13.         disconnect later. (Xavier de Gaye)
  14. Files:        src/netbeans.c
  15.  
  16.  
  17. *** ../vim-7.3.059/src/netbeans.c    2010-11-16 15:04:51.000000000 +0100
  18. --- src/netbeans.c    2010-11-16 15:48:36.000000000 +0100
  19. ***************
  20. *** 135,148 ****
  21.   static int needupdate = 0;
  22.   static int inAtomic = 0;
  23.   
  24.       static void
  25. ! netbeans_close(void)
  26.   {
  27. -     if (!NETBEANS_OPEN)
  28. -     return;
  29. - 
  30. -     netbeans_send_disconnect();
  31. - 
  32.   #ifdef FEAT_GUI_X11
  33.       if (inputHandler != (XtInputId)NULL)
  34.       {
  35. --- 135,146 ----
  36.   static int needupdate = 0;
  37.   static int inAtomic = 0;
  38.   
  39. + /*
  40. +  * Close the socket and remove the input handlers.
  41. +  */
  42.       static void
  43. ! nb_close_socket(void)
  44.   {
  45.   #ifdef FEAT_GUI_X11
  46.       if (inputHandler != (XtInputId)NULL)
  47.       {
  48. ***************
  49. *** 167,179 ****
  50.   # endif
  51.   #endif
  52.   
  53.   #ifdef FEAT_BEVAL
  54.       bevalServers &= ~BEVAL_NETBEANS;
  55.   #endif
  56.   
  57. -     sock_close(nbsock);
  58. -     nbsock = -1;
  59. - 
  60.       needupdate = 0;
  61.       inAtomic = 0;
  62.       nb_free();
  63. --- 165,191 ----
  64.   # endif
  65.   #endif
  66.   
  67. +     sock_close(nbsock);
  68. +     nbsock = -1;
  69. + }
  70. + 
  71. + /*
  72. +  * Close the connection and cleanup.
  73. +  * May be called when nb_close_socket() was called earlier.
  74. +  */
  75. +     static void
  76. + netbeans_close(void)
  77. + {
  78. +     if (NETBEANS_OPEN)
  79. +     {
  80. +     netbeans_send_disconnect();
  81. +     nb_close_socket();
  82. +     }
  83. + 
  84.   #ifdef FEAT_BEVAL
  85.       bevalServers &= ~BEVAL_NETBEANS;
  86.   #endif
  87.   
  88.       needupdate = 0;
  89.       inAtomic = 0;
  90.       nb_free();
  91. ***************
  92. *** 632,640 ****
  93.       char_u    *p;
  94.       queue_T    *node;
  95.   
  96. -     if (!NETBEANS_OPEN)
  97. -     return;
  98. - 
  99.       while (head.next != NULL && head.next != &head)
  100.       {
  101.       node = head.next;
  102. --- 644,649 ----
  103. ***************
  104. *** 720,725 ****
  105. --- 729,736 ----
  106.   }
  107.   #endif
  108.   
  109. + #define DETACH_MSG "DETACH\n"
  110. + 
  111.       void
  112.   netbeans_read()
  113.   {
  114. ***************
  115. *** 780,801 ****
  116.           break;    /* did read everything that's available */
  117.       }
  118.   
  119.       if (readlen <= 0)
  120.       {
  121. !     /* read error or didn't read anything */
  122. !     netbeans_close();
  123. !     nbdebug(("messageFromNetbeans: Error in read() from socket\n"));
  124.       if (len < 0)
  125.       {
  126.           nbdebug(("read from Netbeans socket\n"));
  127.           PERROR(_("read from Netbeans socket"));
  128.       }
  129. -     return; /* don't try to parse it */
  130.       }
  131.   
  132.   #if defined(NB_HAS_GUI) && defined(FEAT_GUI_GTK)
  133.       if (NB_HAS_GUI && gtk_main_level() > 0)
  134. !         gtk_main_quit();
  135.   #endif
  136.   }
  137.   
  138. --- 791,822 ----
  139.           break;    /* did read everything that's available */
  140.       }
  141.   
  142. +     /* Reading a socket disconnection (readlen == 0), or a socket error. */
  143.       if (readlen <= 0)
  144.       {
  145. !     /* Queue a "DETACH" netbeans message in the command queue in order to
  146. !      * terminate the netbeans session later. Do not end the session here
  147. !      * directly as we may be running in the context of a call to
  148. !      * netbeans_parse_messages():
  149. !      *    netbeans_parse_messages
  150. !      *        -> autocmd triggered while processing the netbeans cmd
  151. !      *        -> ui_breakcheck
  152. !      *            -> gui event loop or select loop
  153. !      *            -> netbeans_read()
  154. !      */
  155. !     save((char_u *)DETACH_MSG, strlen(DETACH_MSG));
  156. !     nb_close_socket();
  157. ! 
  158.       if (len < 0)
  159.       {
  160.           nbdebug(("read from Netbeans socket\n"));
  161.           PERROR(_("read from Netbeans socket"));
  162.       }
  163.       }
  164.   
  165.   #if defined(NB_HAS_GUI) && defined(FEAT_GUI_GTK)
  166.       if (NB_HAS_GUI && gtk_main_level() > 0)
  167. !     gtk_main_quit();
  168.   #endif
  169.   }
  170.   
  171. ***************
  172. *** 1164,1169 ****
  173. --- 1185,1194 ----
  174.   
  175.       nbdebug(("REP %d: <none>\n", cmdno));
  176.   
  177. +     /* Avoid printing an annoying error message. */
  178. +     if (!NETBEANS_OPEN)
  179. +     return;
  180. + 
  181.       sprintf(reply, "%d\n", cmdno);
  182.       nb_send(reply, "nb_reply_nil");
  183.   }
  184. ***************
  185. *** 2753,2763 ****
  186.   {
  187.   #ifdef FEAT_GUI
  188.   # if !defined(FEAT_GUI_X11) && !defined(FEAT_GUI_GTK)  \
  189. !                 && !defined(FEAT_GUI_W32)
  190.       if (gui.in_use)
  191.       {
  192. !         EMSG(_("E838: netbeans is not supported with this GUI"));
  193. !         return;
  194.       }
  195.   # endif
  196.   #endif
  197. --- 2778,2788 ----
  198.   {
  199.   #ifdef FEAT_GUI
  200.   # if !defined(FEAT_GUI_X11) && !defined(FEAT_GUI_GTK)  \
  201. !         && !defined(FEAT_GUI_W32)
  202.       if (gui.in_use)
  203.       {
  204. !     EMSG(_("E838: netbeans is not supported with this GUI"));
  205. !     return;
  206.       }
  207.   # endif
  208.   #endif
  209. *** ../vim-7.3.059/src/version.c    2010-11-16 15:04:51.000000000 +0100
  210. --- src/version.c    2010-11-16 15:22:39.000000000 +0100
  211. ***************
  212. *** 716,717 ****
  213. --- 716,719 ----
  214.   {   /* Add new patch number below this line */
  215. + /**/
  216. +     60,
  217.   /**/
  218.  
  219. -- 
  220.    Another bucket of what can only be described as human ordure hits ARTHUR.
  221. ARTHUR: ... Right!  (to the KNIGHTS) That settles it!
  222.                  "Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD
  223.  
  224.  /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
  225. ///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
  226. \\\  an exciting new programming language -- http://www.Zimbu.org        ///
  227.  \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///
  228.