home *** CD-ROM | disk | FTP | other *** search
/ Usenet 1994 October / usenetsourcesnewsgroupsinfomagicoctober1994disk2.iso / misc / volume15 / pcmail2 / patch03 next >
Encoding:
Text File  |  1990-12-16  |  7.5 KB  |  285 lines
  1. Newsgroups: comp.sources.misc
  2. X-UNIX-From: mcsun!info.win.tue.nl!news
  3. subject: v15i098: pc-mail release 2 patch 3
  4. from: wswietse@svbs01.bs.win.tue.nl (Wietse Venema)
  5. Sender: allbery@uunet.UU.NET (Brandon S. Allbery - comp.sources.misc)
  6.  
  7. Posting-number: Volume 15, Issue 98
  8. Submitted-by: wswietse@svbs01.bs.win.tue.nl (Wietse Venema)
  9. Archive-name: pcmail2/patch03
  10.  
  11. This is the third patch for the pc-mail release 2 package that appeared
  12. in comp.sources.misc january 1990. 
  13.  
  14. Sources and earlier patches can be obtained via anonymous ftp from the
  15. comp.sources.misc archives at uunet.uu.net, or from the author's
  16. machine at ftp.win.tue.nl (internet address 131.155.2.8). The latter
  17. host also has a directory tree with up-to-date MS-DOS binaries.
  18.  
  19. The following four problems are taken care of by this patch:
  20.  
  21. 1 - Allow UUCP control messages (S, H, etc.) to span more than one packet.
  22.     The problem would manifest itself as garbage appearing at the
  23.     beginning of incoming e-mail messages, but only if the g protocol
  24.     was being used. Thanks to Jim OBrien (JEO01@albnydh2.bitnet) for
  25.     providing the information that led to the resolution of the problem.
  26.  
  27. 2 - The sscanf(3) implementations with Microsoft C and Turbo C behave 
  28.     different from the UNIX versions that I tested the programs with. 
  29.     This caused the displayed numbers of (new, out, in or sent) messages 
  30.     to be wrong. The program no longer uses sscanf(3) to parse file names. 
  31.  
  32. 3 - Turbo C thrashes memory when it is freed. The program no longer
  33.     tries to access members of data structures that have been freed.
  34.     Thanks to Richard Brittain (richard@calvin.spp.cornell.edu) for
  35.     pointing out the problem and its solution.
  36.  
  37. 4 - After the protocol negotiation has completed, the program now allows 
  38.     the remote host some time to change its tty mode settings. The problem 
  39.     would sometimes result in the loss of the first packet sent by the PC, 
  40.     and could cause 'g' protocol initialization failures. Thanks to Richard
  41.     Brittain (richard@calvin.spp.cornell.edu) for reporting the problem.
  42.  
  43. The patches are in the form of context diffs: for each block of changed
  44. text, the old one appears first, followed by the corrected one.  The
  45. nature of the changes is indicated in the first column:  replacements
  46. are indicated with a `!' character, additions with  `+', and deletions
  47. with `-'. If you don't have the patch program it should not be too hard
  48. to apply the corrections by hand.
  49.  
  50. *** protomsg.c-    Mon Jan 22 13:04:33 1990
  51. --- protomsg.c    Sat Oct 13 19:33:48 1990
  52. ***************
  53. *** 45,53 ****
  54.   /* CREATION DATE
  55.   /*      Thu Mar 26 16:14:57 GMT+1:00 1987
  56.   /* LAST MODIFICATION
  57. ! /*    90/01/22 13:02:28
  58.   /* VERSION/RELEASE
  59. ! /*    2.1
  60.   /*--*/
  61.   
  62.   #include "defs.h"
  63. --- 45,53 ----
  64.   /* CREATION DATE
  65.   /*      Thu Mar 26 16:14:57 GMT+1:00 1987
  66.   /* LAST MODIFICATION
  67. ! /*    90/10/13 19:33:46
  68.   /* VERSION/RELEASE
  69. ! /*    2.3
  70.   /*--*/
  71.   
  72.   #include "defs.h"
  73. ***************
  74. *** 90,99 ****
  75.   public char *hear()
  76.   {
  77.       register int nread;
  78.   
  79. !     if ((nread = CALL(Read)(ttfd,msgin,MSGBUF)) < 0)
  80. !     trap(E_LOST,"FAILED (link lost)");
  81. !     msgin[nread] = '\0';
  82.       debug(4)("got \"%s\"\n",msgin);
  83.       return(msgin);
  84.   }
  85. --- 90,108 ----
  86.   public char *hear()
  87.   {
  88.       register int nread;
  89. +     register int bufpos = 0;
  90.   
  91. !     /* Allow a message to span multiple packets ... */
  92. ! 
  93. !     for (;;) {
  94. !     if ((nread = CALL(Read)(ttfd,msgin+bufpos,MSGBUF-bufpos)) <= 0)
  95. !         trap(E_LOST,"FAILED (link lost)");
  96. !     if ((bufpos += nread) >= MSGBUF)
  97. !         trap(E_LOST,"FAILED (message buffer overflow)");
  98. !     msgin[bufpos] = '\0';
  99. !     if (strlen(msgin) < bufpos)
  100. !         break;
  101. !     }
  102.       debug(4)("got \"%s\"\n",msgin);
  103.       return(msgin);
  104.   }
  105. *** newseqno.c-    Mon Jan 29 15:50:48 1990
  106. --- newseqno.c    Sat Oct 13 18:47:33 1990
  107. ***************
  108. *** 30,40 ****
  109.   /* CREATION DATE
  110.   /*      Sat Mar 28 18:10:53 GMT+1:00 1987
  111.   /* LAST MODIFICATION
  112. ! /*    90/01/29 15:50:46
  113.   /* VERSION/RELEASE
  114. ! /*    2.2
  115.   /*--*/
  116.   
  117.   #include "defs.h"
  118.   #include "path.h"
  119.   #include "ndir.h"
  120. --- 30,42 ----
  121.   /* CREATION DATE
  122.   /*      Sat Mar 28 18:10:53 GMT+1:00 1987
  123.   /* LAST MODIFICATION
  124. ! /*    90/10/13 18:47:32
  125.   /* VERSION/RELEASE
  126. ! /*    2.4
  127.   /*--*/
  128.   
  129. + #include <ctype.h>
  130. + 
  131.   #include "defs.h"
  132.   #include "path.h"
  133.   #include "ndir.h"
  134. ***************
  135. *** 72,85 ****
  136.   public unsigned seqno(s)
  137.   char   *s;
  138.   {
  139. !     unsigned seq;
  140. !     char    junk = 0;
  141.   
  142. !     /* MicroSoft C sscanf() does not terminate if assignment to seq fails */
  143.   
  144. !     if (strlen(s) == NAMELEN && sscanf(s + 1, "%u%c", &seq, &junk) == 1
  145. !     && junk == 0)
  146. !     return (seq);
  147.       else
  148.       return (0);
  149.   }
  150. --- 74,104 ----
  151.   public unsigned seqno(s)
  152.   char   *s;
  153.   {
  154. !     long    atol();
  155.   
  156. !     /*
  157. !      * Originally, we used sscanf() to parse the string:
  158. !      * 
  159. !      * sscanf(s, "%u%c", &seq, &junk) == 1
  160. !      * 
  161. !      * but that caused problems with Microsoft C (%c conversion is done even
  162. !      * if the %u conversion fails) and Turbo C (assigment to junk even if no
  163. !      * %c conversion is performed). Remedy: do not try to parse the string with
  164. !      * sscanf().
  165. !      */
  166.   
  167. !     if (strlen(s) == NAMELEN && isalpha(*s) && alldig(s+1))
  168. !     return ((unsigned int) atol(s+1));
  169.       else
  170.       return (0);
  171. + }
  172. + 
  173. + /* alldig - does string contain digits only? */
  174. + 
  175. + hidden int alldig(s)
  176. + register char *s;
  177. + {
  178. +     while (*s && isascii(*s) && isdigit(*s))
  179. +     s++;
  180. +     return (*s == 0);
  181.   }
  182. *** pager.c-    Tue Apr 17 11:41:39 1990
  183. --- pager.c    Tue Apr 17 11:45:53 1990
  184. ***************
  185. *** 142,150 ****
  186.   /* CREATION DATE
  187.   /*      Fri Apr  3 22:06:00 GMT+1:00 1987
  188.   /* LAST MODIFICATION
  189. ! /*    90/01/22 13:02:21
  190.   /* VERSION/RELEASE
  191. ! /*    2.1
  192.   /*--*/
  193.   
  194.   #include <stdio.h>
  195. --- 142,150 ----
  196.   /* CREATION DATE
  197.   /*      Fri Apr  3 22:06:00 GMT+1:00 1987
  198.   /* LAST MODIFICATION
  199. ! /*    90/04/17 11:45:48
  200.   /* VERSION/RELEASE
  201. ! /*    2.2
  202.   /*--*/
  203.   
  204.   #include <stdio.h>
  205. ***************
  206. *** 171,183 ****
  207.   /* close_pager - release memory in a pager file */
  208.   
  209.   public void close_pager(p)
  210. ! register File *p;
  211.   {
  212.       register Line *q;
  213.   
  214.       if (p) {
  215. !     for (q = p->head; q; q = q->next)    /* release lines */
  216.           free((char *) q);
  217.       if (curfile == p)            /* unset current file */
  218.           curfile = 0;
  219.       free((char *) p);            /* release header */
  220. --- 171,186 ----
  221.   /* close_pager - release memory in a pager file */
  222.   
  223.   public void close_pager(p)
  224. ! File   *p;
  225.   {
  226.       register Line *q;
  227. +     register Line *next;
  228.   
  229.       if (p) {
  230. !     for (q = p->head; q; q = next) {    /* release lines */
  231. !         next = q->next;            /* avoid using freed block */
  232.           free((char *) q);
  233. +     }
  234.       if (curfile == p)            /* unset current file */
  235.           curfile = 0;
  236.       free((char *) p);            /* release header */
  237. *** startup.c-    Mon Jan 22 13:04:38 1990
  238. --- startup.c    Sat Oct 13 19:01:39 1990
  239. ***************
  240. *** 40,48 ****
  241.   /* CREATION DATE
  242.   /*      Fri Mar 27 13:43:00 GMT+1:00 1987
  243.   /* LAST MODIFICATION
  244. ! /*    90/01/22 13:02:41
  245.   /* VERSION/RELEASE
  246. ! /*    2.1
  247.   /*--*/
  248.   
  249.   #include <stdio.h>
  250. --- 40,48 ----
  251.   /* CREATION DATE
  252.   /*      Fri Mar 27 13:43:00 GMT+1:00 1987
  253.   /* LAST MODIFICATION
  254. ! /*    90/10/13 19:01:38
  255.   /* VERSION/RELEASE
  256. ! /*    2.2
  257.   /*--*/
  258.   
  259.   #include <stdio.h>
  260. ***************
  261. *** 129,134 ****
  262. --- 129,138 ----
  263.   
  264.       log("OK (startup)");
  265.       systrap = savetrap;                /* get here if expect wins */
  266. + 
  267. +     /* Give remote host some time to switch tty modes... */
  268. + 
  269. +     sleep(1);
  270.       return (0);
  271.   }
  272.   
  273. ***************
  274. *** 136,143 ****
  275.   
  276.   public  endproto()
  277.   {
  278. -     int    *savetrap = systrap;            /* save exception handler */
  279. -     jmp_buf mytrap;
  280.       int     status;
  281.   
  282.       if (Close)                    /* check there is one */
  283. --- 140,145 ----
  284.  
  285.