home *** CD-ROM | disk | FTP | other *** search
/ NetNews Usenet Archive 1992 #27 / NN_1992_27.iso / spool / gnu / utils / bug / 2076 < prev    next >
Encoding:
Text File  |  1992-11-19  |  4.0 KB  |  120 lines
  1. Newsgroups: gnu.utils.bug
  2. Path: sparky!uunet!zaphod.mps.ohio-state.edu!pacific.mps.ohio-state.edu!cis.ohio-state.edu!immd3.INformatik.uni-erlangen.DE!dalibor
  3. From: dalibor@immd3.INformatik.uni-erlangen.DE (Stefan Dalibor)
  4. Subject: bug-report for finger-1.37
  5. Message-ID: <9211191758.AA02924@faui33.informatik.uni-erlangen.de>
  6. Sender: gnulists@ai.mit.edu
  7. Organization: GNUs Not Usenet
  8. Distribution: gnu
  9. Date: Thu, 19 Nov 1992 19:58:37 GMT
  10. Approved: bug-gnu-utils@prep.ai.mit.edu
  11. Lines: 107
  12.  
  13. Hi,
  14. we just completed installation of finger-1.37. We found 1 bug in src/fingerd.c
  15. and had some difficulties during the installation due to differences between
  16. the source and the documentation (and our misunderstanding of the latter :-).
  17. Filenames with extensions `-dist' refer to the files of the original distri-
  18. bution.
  19. (Scroll through items by looking for [1-4] at beginning of line.)
  20.  
  21. 1. Bug in src/finger.c:
  22.     The fingerd master-server crashes when started with the (undocumented?)
  23.     option '-i <secs>'.
  24.     Reason: Missing colon (for option -i) in the argument string to
  25.             getopt_long() in line 148 of fingerd.c; furthermore we think
  26.             it should be checked that the user specified an additional
  27.             argument before trying to convert it with atoi(). The following
  28.             works for us:
  29.  
  30. faui30a:~LS/finger-1.37/src(0)5 diff -c fingerd.c fingerd.c-dist 
  31. *** fingerd.c   Tue Nov 17 10:46:42 1992
  32. --- fingerd.c-dist      Tue Nov 17 10:41:53 1992
  33. ***************
  34. *** 145,151 ****
  35.     default_error_handling (argv[0]);
  36.   
  37.     /* Parse arguments. */
  38. !   while ((optc = getopt_long (argc, argv, "datfi:", longopts, NULL)) >= 0)
  39.       switch (optc)
  40.         {
  41.         case 'd':
  42. --- 145,151 ----
  43.     default_error_handling (argv[0]);
  44.   
  45.     /* Parse arguments. */
  46. !   while ((optc = getopt_long (argc, argv, "datfi", longopts, NULL)) >= 0)
  47.       switch (optc)
  48.         {
  49.         case 'd':
  50. ***************
  51. *** 167,173 ****
  52.         break;
  53.   
  54.         case 'i':
  55. !       if ((optarg == NULL) || (time_between_polls = atoi (optarg)) < 0)
  56.           time_between_polls = DEFAULT_POLL_INTERVAL;
  57.         break;
  58.   
  59. --- 167,173 ----
  60.         break;
  61.   
  62.         case 'i':
  63. !       if ((time_between_polls = atoi (optarg)) < 0)
  64.           time_between_polls = DEFAULT_POLL_INTERVAL;
  65.         break;
  66.  
  67. 2. It took us some time (actually we had to use gdb :-) to find out that
  68.    the finger client-program uses it's argv[0] as the name of the service
  69.    by (under? - sorry for my bad english!) which it tries to contact the
  70.    finger-server.
  71.    We think that should be pointed out more clearly in the documentation / in-
  72.    stallation-guideline. We could imagine more people wanting to call gnus's
  73.    finger-program as gnu-finger.
  74.    Perhaps BASENAME_IS_SERVICE should not be defined by default in config.h?
  75.  
  76. 3. And here the rest of the usual bug-report informations:
  77.    Program version: finger-1.37 from prep.ai.mit.edu
  78.    Hardware: Sun3 and Sun4
  79.    Operating System: SunOs 4.1.2
  80.    Compiler: Sun's cc and gcc version 2.2
  81.    Arguments to configure:
  82.       none on sun4
  83.       CC='cc -O' on sun3 (broken gcc, too lazy to install correct version :-)
  84.    Changes to configuration files:
  85.  
  86.     faui30a:~LS/finger-1.37(0)14 diff config.h config.h-dist 
  87.     62c62
  88.     < #define CHECK_OWNER_FINGERRC
  89.     ---
  90.     > /* #define CHECK_OWNER_FINGERRC */
  91.     66c66
  92.     < #define CHECK_RDONLY_FINGERRC
  93.     ---
  94.     > /* #define CHECK_RDONLY_FINGERRC */
  95.     89c89
  96.     < #define FINGERRC_SHELL "/bin/sh"
  97.     ---
  98.     > /* #define FINGERRC_SHELL "/bin/sh" */
  99.     95c95
  100.     < #define DEFAULT_POLL_INTERVAL 300
  101.     ---
  102.     > #define DEFAULT_POLL_INTERVAL 120
  103.     98,100c98
  104.     < #ifdef sparc
  105.     < #define HAVE_X11
  106.     < #endif
  107.     ---
  108.     > /* #define HAVE_X11 */
  109.  
  110.    Changes to source code: see 1. above
  111.    Changes to installation procedures: none besides adaption to local pathnames
  112.    Network configuration: see hardware/operating system/compiler above
  113.    Installation problems: see 2. above
  114.  
  115. 4. Looking forward to future versions of finger (and other gnu programs)!
  116.  
  117. Bye,
  118. Stefan
  119.  
  120.