home *** CD-ROM | disk | FTP | other *** search
/ NetNews Usenet Archive 1992 #27 / NN_1992_27.iso / spool / comp / infosyst / gopher / 1145 next >
Encoding:
Text File  |  1992-11-15  |  7.7 KB  |  288 lines
  1. Path: sparky!uunet!pmafire!mica.inel.gov!ux1!news.byu.edu!eff!sol.ctr.columbia.edu!usc!rpi!batcomputer!munnari.oz.au!mel.dit.csiro.au!mineng.dmpe.CSIRO.AU!dmssyd.syd.dms.CSIRO.AU!metro!grivel!alsvid!mark
  2. From: mark@alsvid.une.edu.au (Mark Andrew Garrett)
  3. Newsgroups: comp.infosystems.gopher
  4. Subject: BUG in gopher1.1b1
  5. Message-ID: <Bxr5zA.Bx1@alsvid.une.edu.au>
  6. Date: 15 Nov 92 10:18:45 GMT
  7. Organization: University of New England - Northern Rivers (Lismore)
  8. Lines: 278
  9.  
  10. I've been try gopher1.1b1 + wais-8-b5 out on ultrix4.2 gcc2.3.1 (and gcc2.2.2)
  11.  
  12. I've added an extra secure mode Securemode == VERY
  13. which disallows the mail option as well for the client.
  14.  
  15. I've also been having crash problems with memory aborts.
  16. I've found at least on bug in the STRstring.c but not the one
  17. giving me crashes in side of STRinit()
  18.  
  19.     Here are my diffs for gopher and wais
  20. Any comments gratefully recieved.
  21.  
  22.         Cheers
  23.             Mark :)
  24.  
  25.  
  26. diff -cr gopher1.1b1-DIST/doc/gopher.1 gopher1.1b1/doc/gopher.1
  27. *** gopher1.1b1-DIST/doc/gopher.1    Sat Aug  1 03:34:16 1992
  28. --- gopher1.1b1/doc/gopher.1    Sun Nov 15 19:44:30 1992
  29. ***************
  30. *** 3,9 ****
  31.   gopher \- connect to gopher document server
  32.   .SH SYNOPSIS
  33.   .B gopher
  34. ! .RI [ -sb ]
  35.   .RI [ -t
  36.   .IR title]
  37.   .RI [ -p 
  38. --- 3,9 ----
  39.   gopher \- connect to gopher document server
  40.   .SH SYNOPSIS
  41.   .B gopher
  42. ! .RI [ -sSb ]
  43.   .RI [ -t
  44.   .IR title]
  45.   .RI [ -p 
  46. ***************
  47. *** 23,28 ****
  48. --- 23,31 ----
  49.   .PP
  50.   -s means "secure mode" which means that you can't save or print the
  51.   files that you browse.
  52. + .PP
  53. + -S means "very secure mode" which means that you can't save,  print or
  54. + mail the files that you browse.
  55.   .PP
  56.   -b starts the client on the bookmark page.
  57.   
  58. diff -cr gopher1.1b1-DIST/gopher/globals.h gopher1.1b1/gopher/globals.h
  59. *** gopher1.1b1-DIST/gopher/globals.h    Wed Oct 14 18:07:16 1992
  60. --- gopher1.1b1/gopher/globals.h    Sun Nov 15 19:40:12 1992
  61. ***************
  62. *** 79,81 ****
  63. --- 79,83 ----
  64.   int  Load_Index_or_Dir();
  65.   void GetOneOption(/* */);
  66.   void check_sock(/* int, char* */);
  67. + 
  68. + #define VERY (-1)
  69. Only in gopher1.1b1/gopher: globals.h-DIST
  70. Only in gopher1.1b1/gopher: globals.o
  71. Only in gopher1.1b1/gopher: gopher
  72.  
  73. diff -cr gopher1.1b1-DIST/gopher/gopher.c gopher1.1b1/gopher/gopher.c
  74. *** gopher1.1b1-DIST/gopher/gopher.c    Thu Nov  5 11:33:02 1992
  75. --- gopher1.1b1/gopher/gopher.c    Sun Nov 15 19:45:56 1992
  76. ***************
  77. *** 809,819 ****
  78.   
  79.        sTmp[0] = '\0';
  80.   
  81. !      while ((c = getopt(argc, argv, "Dsbp:t:")) != -1)
  82.         switch (c) {
  83.         case 's':
  84.              SecureMode = TRUE;
  85.              break;
  86.         case 'p':
  87.              GSsetPath(RootGophers[0], optarg);
  88.              GSsetPath(RootGophers[1], optarg);
  89. --- 809,822 ----
  90.   
  91.        sTmp[0] = '\0';
  92.   
  93. !      while ((c = getopt(argc, argv, "DsSbp:t:")) != -1)
  94.         switch (c) {
  95.         case 's':
  96.              SecureMode = TRUE;
  97.              break;
  98. +       case 'S':
  99. +            SecureMode = VERY;
  100. +            break;
  101.         case 'p':
  102.              GSsetPath(RootGophers[0], optarg);
  103.              GSsetPath(RootGophers[1], optarg);
  104. ***************
  105. *** 834,840 ****
  106.   
  107.   
  108.        if (errflag) {
  109. !       fprintf(stderr, "Usage: %s [-sb] [-p path] [-t title] [hostname port]+\n", argv[0]);
  110.         exit(-1);
  111.        }
  112.   
  113. --- 837,843 ----
  114.   
  115.   
  116.        if (errflag) {
  117. !       fprintf(stderr, "Usage: %s [-sSb] [-p path] [-t title] [hostname port]+\n", argv[0]);
  118.         exit(-1);
  119.        }
  120.   
  121. diff -cr gopher1.1b1-DIST/gopher/ourutils.c gopher1.1b1/gopher/ourutils.c
  122. *** gopher1.1b1-DIST/gopher/ourutils.c    Sat Oct 31 07:48:38 1992
  123. --- gopher1.1b1/gopher/ourutils.c    Sun Nov 15 19:31:25 1992
  124. ***************
  125. *** 131,138 ****
  126.   
  127.        if (!SecureMode)
  128.         printf(", <m> to mail, <s> to save, or <p> to print:");
  129. !      else
  130.         printf(", <m> to mail:");
  131.   
  132.        fflush(stdout);
  133.        noecho();
  134. --- 131,142 ----
  135.   
  136.        if (!SecureMode)
  137.         printf(", <m> to mail, <s> to save, or <p> to print:");
  138. !      else {
  139. !     if (SecureMode == VERY)
  140. !       printf(":");
  141. !     else
  142.         printf(", <m> to mail:");
  143. +      }
  144.   
  145.        fflush(stdout);
  146.        noecho();
  147. ***************
  148. *** 153,158 ****
  149. --- 157,163 ----
  150.              case ' ':
  151.               break;
  152.              case 'm':
  153. +           if (SecureMode != VERY)
  154.               mail_file(Filename, Realname);
  155.               break;
  156.   
  157. diff -cr gopher1.1b1-DIST/object/STRstring.c gopher1.1b1/object/STRstring.c
  158. *** gopher1.1b1-DIST/object/STRstring.c    Sat Oct 31 07:18:46 1992
  159. --- gopher1.1b1/object/STRstring.c    Sun Nov 15 20:48:32 1992
  160. ***************
  161. *** 18,24 ****
  162.        if (in == NULL)
  163.         return(temp);
  164.   
  165. !      len = strlen(in);
  166.   
  167.        temp->data = (char *) malloc(len * sizeof(char*));
  168.        strcpy(temp->data, in);
  169. --- 18,24 ----
  170.        if (in == NULL)
  171.         return(temp);
  172.   
  173. !      len = strlen(in) + 1; /* THIS WAS A BUG!allow for null termination.... */
  174.   
  175.        temp->data = (char *) malloc(len * sizeof(char*));
  176.        strcpy(temp->data, in);
  177. diff -cr gopher1.1b1-DIST/object/compatible.c gopher1.1b1/object/compatible.c
  178. *** gopher1.1b1-DIST/object/compatible.c    Sat Nov  7 05:21:39 1992
  179. --- gopher1.1b1/object/compatible.c    Thu Nov 12 04:54:26 1992
  180. ***************
  181. *** 9,19 ****
  182.   
  183.   /*** For machines that don't have strstr ***/
  184.   
  185. ! #if defined(NOSTRSTR) || defined(mips) || defined(sequent) || defined(n16)
  186.   
  187.   char *
  188.   strstr(host_name, cp)
  189. !   char host_name[256];
  190.     char *cp;
  191.   {
  192.        int i, j;
  193. --- 9,19 ----
  194.   
  195.   /*** For machines that don't have strstr ***/
  196.   
  197. ! #if defined(NOSTRSTR) || (defined(mips) && !defined(ultrix)) || defined(sequent) || defined(n16)
  198. + /* this was also a problem in 1.03 DEC mips have strstr and this wont compile */
  199.   char *
  200.   strstr(host_name, cp)
  201. ! char host_name[256];
  202.     char *cp;
  203.   {
  204.        int i, j;
  205.  
  206.  
  207. And the changes for WAIS-8-B5 .... diffs
  208.  
  209.  
  210. diff -cr wais-8-b5-DIST/ir/irretrvl.c wais-8-b5/ir/irretrvl.c
  211. *** wais-8-b5-DIST/ir/irretrvl.c    Mon May 11 07:43:57 1992
  212. --- wais-8-b5/ir/irretrvl.c    Sat Nov 14 21:00:41 1992
  213. ***************
  214. *** 76,88 ****
  215.   
  216.    */
  217.   
  218. ! #include "irretrvl.h"
  219.   #include "irfiles.h" /* for filename_table_ext */
  220.   #include <string.h>
  221. ! #include "futil.h"
  222.   #include <ctype.h>  /* for isspace */
  223. ! #include "irext.h"
  224. ! #include "irdirent.h"
  225.   #include <sys/stat.h>
  226.   
  227.   #ifdef Mach
  228. --- 76,89 ----
  229.   
  230.    */
  231.   
  232. ! #include "irretrvl.h" /* */
  233.   #include "irfiles.h" /* for filename_table_ext */
  234.   #include <string.h>
  235. ! #include "futil.h" /* */
  236.   #include <ctype.h>  /* for isspace */
  237. ! #include "irext.h" /* */
  238. ! #include "irdirent.h" /* */
  239. ! #include <sys/types.h> /* must include for ultrix gcc for stat.h */
  240.   #include <sys/stat.h>
  241.   
  242.   #ifdef Mach
  243. diff -cr wais-8-b5-DIST/ui/util.h wais-8-b5/ui/util.h
  244. *** wais-8-b5-DIST/ui/util.h    Wed Mar 18 08:29:40 1992
  245. --- wais-8-b5/ui/util.h    Sat Nov 14 21:39:28 1992
  246. ***************
  247. *** 17,24 ****
  248.   #define _H_UTIL
  249.   
  250.   #include <cdialect.h>
  251.   
  252. ! int charlistlength _AP((char** list));
  253. ! void listlength _AP((List list));
  254.   void find_value _AP((char* source, char* key, char* value, int value_size));
  255.   #endif
  256. --- 17,25 ----
  257.   #define _H_UTIL
  258.   
  259.   #include <cdialect.h>
  260. + #include <list.h>    /* MG attempt to keep gcc happy about listlength */
  261.   
  262. ! int charlistlength _AP((char ** list));
  263. ! /* void listlength _AP(( List list)); Now way could i get this to compile
  264. so its comment it out, everything works ok with our this profile */
  265.   void find_value _AP((char* source, char* key, char* value, int value_size));
  266.   #endif
  267. diff -cr wais-8-b5-DIST/x/qcommands.c wais-8-b5/x/qcommands.c
  268. *** wais-8-b5-DIST/x/qcommands.c    Fri May  8 07:51:26 1992
  269. --- wais-8-b5/x/qcommands.c    Sat Aug 22 18:30:12 1992
  270. ***************
  271. *** 386,392 ****
  272.   void
  273.   AddResponseToQuestion(w, closure, call_data)
  274.   Widget w;
  275. ! caddr_t closure, call_data;
  276.   {
  277.     int numdocs, document_number, i;
  278.     Question q = the_Question->q;
  279. --- 386,393 ----
  280.   void
  281.   AddResponseToQuestion(w, closure, call_data)
  282.   Widget w;
  283. ! /* caddr_t closure, call_data; */
  284. ! XtPointer closure, call_data; 
  285.   {
  286.     int numdocs, document_number, i;
  287.     Question q = the_Question->q;
  288.