home *** CD-ROM | disk | FTP | other *** search
/ NetNews Usenet Archive 1992 #20 / NN_1992_20.iso / spool / gnu / utils / bug / 1530 < prev    next >
Encoding:
Text File  |  1992-09-10  |  2.3 KB  |  75 lines
  1. Newsgroups: gnu.utils.bug
  2. Path: sparky!uunet!usc!zaphod.mps.ohio-state.edu!cis.ohio-state.edu!prepress.com!bob
  3. From: bob@prepress.com (Robert Crowe)
  4. Subject: problem with tar-1.11 and -v option.
  5. Message-ID: <m0mScc3-00048AC@indian.prepress.com>
  6. Sender: gnulists@ai.mit.edu
  7. Organization: GNUs Not Usenet
  8. Distribution: gnu
  9. Date: Thu, 10 Sep 1992 00:42:00 GMT
  10. Approved: bug-gnu-utils@prep.ai.mit.edu
  11. Lines: 62
  12.  
  13. I just finished installing tar-1.11 on our SGI's (4D/25,Indigos;
  14. IRIX-4.01), and I ran into a problem.  When specifying the -v option
  15. with the -c option, the program core dumps. I traced the problem to
  16. the fact that current_file_name never gets set when going through the 
  17. create code.  My fix was ass follows:
  18.  
  19. ======= Diff Starts here ===========
  20. *** create.c    Wed Sep  9 17:21:37 1992
  21. --- create.c.~2~    Wed Sep  9 17:13:31 1992
  22. ***************
  23. *** 1273,1279 ****
  24.           head = header;
  25.           /* hstat is already set up */
  26.           head_standard = f_standard;
  27. !         current_file_name = header->header.arch_name; /* Added by bob. */
  28.           print_header();
  29.       }
  30.   
  31. --- 1273,1279 ----
  32.           head = header;
  33.           /* hstat is already set up */
  34.           head_standard = f_standard;
  35. !         current_file_name = name; /* Added by bob. */
  36.           print_header();
  37.       }
  38.   
  39. ============ Diff ends here =============
  40.  
  41. There may be a better place to set current_file_name; If so please
  42. post it and I will gladly apply that instead.
  43.  
  44. The actual place where it was dumping core was in quote_copy_string,
  45. called from ~ line 536 in list.c:
  46.  
  47.         if (f_verbose <= 1) {
  48.                 /* Just the fax, mam. */
  49.                 char *name;
  50.  
  51.                 name=quote_copy_string(current_file_name);  <This is the Call>
  52.                 if(name==0)
  53.                         name=current_file_name;
  54.  
  55.  
  56. Other than that, everything seems to work fine. Granted I have not
  57. tested extensively yet. 
  58.  
  59. Thanks, the new features seem like they will come in real handy.
  60.  
  61. Bob.
  62.  
  63.  
  64. P.S.  The makefile wants to build an info file ( which I love to have... ),
  65.       But the .texi file seems to be missing in the distribution I
  66.       grabbed (tar-1.11.tar.Z). Is it available elsewhere?
  67.  
  68.  
  69. ============================================================
  70. Bob Crowe                      Voice:   (619) 931-2695
  71. Pre-Press Technologies.        Email:    bob@prepress.com
  72. 2443 Impala dr.               FAX:     (619) 931-2698
  73. Carlsbad, Ca 92008.
  74.  
  75.