home *** CD-ROM | disk | FTP | other *** search
/ NetNews Usenet Archive 1992 #26 / NN_1992_26.iso / spool / comp / sources / bugs / 270 < prev    next >
Encoding:
Text File  |  1992-11-10  |  4.8 KB  |  170 lines
  1. Xref: sparky comp.sources.bugs:270 comp.sys.mac.comm:10796
  2. Path: sparky!uunet!ferkel.ucsb.edu!taco!gatech!darwin.sura.net!jvnc.net!netnews.upenn.edu!news.cc.swarthmore.edu!hirai
  3. From: hirai@cc.swarthmore.edu (Eiji Hirai)
  4. Newsgroups: comp.sources.bugs,comp.sys.mac.comm
  5. Subject: Re; UNOFFICIAL PATCH to popper-1.831b
  6. Message-ID: <M0ZSB51M@cc.swarthmore.edu>
  7. Date: 10 Nov 92 20:55:32 GMT
  8. References: <Q0YSBG55@cc.swarthmore.edu>
  9. Sender: news@cc.swarthmore.edu (USENET News System)
  10. Organization: Information Services, Swarthmore College, Swarthmore, PA, USA
  11. Lines: 156
  12. Nntp-Posting-Host: gingko
  13.  
  14. I wrote:
  15. : This adds an idle timeout routine to popper 1.831b
  16.  
  17. sdorner@qualcomm.com (Steve Dorner) pointed out a potential problem with the
  18. patch.  Thanks Steve.  Here's a fixed patch.  You'll need to apply this
  19. patch to the original popper source, not the patched source.  If you need the
  20. original, it's at lilac.berkeley.edu:/pub/popper/.
  21.  
  22. --
  23. hirai@cc.swarthmore.edu (Eiji Hirai)    :     :    :   :  : :: ::: :::: :::::
  24. Unix Geek for Swarthmore College        :     :    :   :  : :: ::: :::: :::::
  25. Information Services, Swarthmore, PA, US.       Copyright 1992 by Eiji Hirai.
  26. I don't speak for Swarthmore College.                    All Rights Reserved.
  27.  
  28. Subject: idle timeout routines
  29.  
  30. Description:
  31.  
  32.     If a pop client has opened a connection to the popper server and the
  33.     client host crashes, the connection to the popper server is never
  34.     closed and the popper server will never quit.  After a large number
  35.     of client host crashes, the server host will run out of file
  36.     descriptors and memory.
  37.  
  38. Repeat-By:
  39.  
  40.     Run Eudora (1.2.2 or 1.3b102) on a Macintosh with MacTCP 1.1, then
  41.     download a large mail file.  During the download, restart the
  42.     Macintosh.  The popper server which was talking to that client will
  43.     be running on your server host until you manually kill it.  Shout,
  44.     "eek!"
  45.  
  46. Fix:
  47.     Apply this patch.  Add -DTIMEOUT to CFLAGS in Makefile.
  48.  
  49. Index: version.h
  50. Prereq: "1.831beta"
  51. *** version.h.old    Wed Apr  3 18:25:44 1991
  52. --- version.h    Mon Nov  9 20:34:28 1992
  53. ***************
  54. *** 12,15 ****
  55. --- 12,19 ----
  56.    *  Current version of this POP implementation
  57.    */
  58.   
  59. + #ifdef TIMEOUT
  60. + #define VERSION         "1.831beta+timeout"
  61. + #else
  62.   #define VERSION         "1.831beta"
  63. + #endif /* TIMEOUT */
  64. *** popper.h.old    Wed Apr  3 18:25:43 1991
  65. --- popper.h    Tue Nov 10 14:40:48 1992
  66. ***************
  67. *** 170,172 ****
  68. --- 170,176 ----
  69.   extern int  pop_xtnd();
  70.   extern int  pop_xmit();
  71.   
  72. + #ifdef TIMEOUT
  73. + # define CHECKIDLE    60
  74. + # define MAXIDLE    600
  75. + #endif /* TIMEOUT */
  76. *** popper.c.old    Wed Apr  3 18:25:42 1991
  77. --- popper.c    Tue Nov 10 15:48:11 1992
  78. ***************
  79. *** 15,20 ****
  80. --- 15,28 ----
  81.   
  82.   extern  state_table *   pop_get_command();
  83.   
  84. + #ifdef TIMEOUT
  85. + # include <signal.h>
  86. + # include <time.h>
  87. + # define p    main_p
  88. + POP    p;
  89. + time_t    lasttime;
  90. + #endif /* TIMEOUT */
  91. + 
  92.   /* 
  93.    *  popper: Handle a Post Office Protocol version 3 session
  94.    */
  95. ***************
  96. *** 22,28 ****
  97. --- 30,40 ----
  98.   int         argc;
  99.   char    **  argv;
  100.   {
  101. + #ifdef TIMEOUT
  102. +     void timeout();
  103. + #else
  104.       POP                 p;
  105. + #endif /* TIMEOUT */
  106.       state_table     *   s;
  107.       char                message[MAXLINELEN];
  108.   
  109. ***************
  110. *** 33,38 ****
  111. --- 45,56 ----
  112.       pop_msg(&p,POP_SUCCESS,
  113.           "UCB Pop server (version %s) at %s starting.",VERSION,p.myhost);
  114.   
  115. + #ifdef TIMEOUT
  116. +     lasttime = time((time_t *)0);
  117. +     signal(SIGALRM, timeout);
  118. +     alarm(CHECKIDLE);  /* turn on idle timer */
  119. + #endif /* TIMEOUT */
  120. + 
  121.       /*  State loop.  The POP server is always in a particular state in 
  122.           which a specific suite of commands can be executed.  The following 
  123.           loop reads a line from the client, gets the command, and processes 
  124. ***************
  125. *** 47,52 ****
  126. --- 65,73 ----
  127.               pop_msg(&p,POP_FAILURE,"POP server at %s signing off.",p.myhost);
  128.           }
  129.           else {
  130. + #ifdef TIMEOUT
  131. +         alarm(0); /* turn off idle timer */
  132. + #endif /* TIMEOUT */
  133.               /*  Search for the command in the command/state table */
  134.               if ((s = pop_get_command(&p,message)) == NULL) continue;
  135.   
  136. ***************
  137. *** 59,64 ****
  138. --- 80,89 ----
  139.                   p.CurrentState = s->success_state;
  140.                   pop_msg(&p,POP_SUCCESS,NULL);
  141.               }
  142. + #ifdef TIMEOUT
  143. +         lasttime = time((time_t *)0);
  144. +         alarm(CHECKIDLE);  /* turn on idle timer */
  145. + #endif /* TIMEOUT */
  146.           }       
  147.       }
  148.   
  149. ***************
  150. *** 100,102 ****
  151. --- 125,142 ----
  152.       return(0);
  153.   }
  154.   #endif STRNCASECMP
  155. + 
  156. + #ifdef TIMEOUT
  157. + void    timeout() {
  158. +     if (time((time_t *)0) - lasttime >= MAXIDLE) {
  159. +         pop_msg(&p,POP_SUCCESS,
  160. +             "Pop server at %s signing off.",p.myhost);
  161. +         pop_log(&p,POP_PRIORITY,
  162. +     "(v%s) Idle timeout from \"%s\" at %s",VERSION,p.client,p.ipaddr);
  163. +         closelog();
  164. +         exit(0);
  165. +     }
  166. +     else
  167. +         alarm(CHECKIDLE);
  168. + }
  169. + #endif /* TIMEOUT */
  170.