X-Git-Url: https://git.dogcows.com/gitweb?a=blobdiff_plain;ds=inline;f=src%2Fcreate.c;h=6808fba926869e09f7d07726ddd292e640fe8964;hb=ebeb3d4f3058b7bbb92e7ea797cf938ec03ee2ed;hp=b3e4d73e9d172fb7c1dd4fe71addaef359db181f;hpb=fe4ee38fce628e37fb3d5f562d0ca21840760b66;p=chaz%2Ftar

diff --git a/src/create.c b/src/create.c
index b3e4d73..6808fba 100644
--- a/src/create.c
+++ b/src/create.c
@@ -588,8 +588,6 @@ start_header (const char *name, struct tar_stat_info *st)
   union block *header;
 
   name = safer_name_suffix (name, 0);
-  if (name[0] == '.' && name[1] == 0) /*FIXME!!!*/
-    return NULL;
   assign_string (&st->file_name, name);
 
   header = write_header_name (st);
@@ -1087,6 +1085,15 @@ compare_links (void const *entry1, void const *entry2)
   return ((link1->dev ^ link2->dev) | (link1->ino ^ link2->ino)) == 0;
 }
 
+/* Copy at most LEN bytes from SRC to DST. Terminate with NUL unless
+   SRC is LEN characters long */
+static void
+tar_copy_str (char *dst, const char *src, size_t len)
+{
+  dst[len-1] = 0;
+  strncpy (dst, src, len);
+}
+
 /* Table of all non-directories that we've written so far.  Any time
    we see another, we check the table and avoid dumping the data
    again if we've done it once already.  */
@@ -1217,15 +1224,6 @@ dump_file (char *p, int top_level, dev_t parent_device)
 	  block_ordinal = current_block_ordinal ();
 	  current_stat_info.stat.st_size = 0;	/* force 0 size on dir */
 
-	  /* FIXME: If people could really read standard archives, this
-	     should be:
-
-	     header
-	       = start_header (standard_option ? p : namebuf, &current_stat_info);
-
-	     but since they'd interpret DIRTYPE blocks as regular
-	     files, we'd better put the / on the name.  */
-
 	  header = start_header (namebuf, &current_stat_info);
 	  if (!header)
 	    return;
@@ -1358,17 +1356,15 @@ dump_file (char *p, int top_level, dev_t parent_device)
 	      
 	      block_ordinal = current_block_ordinal ();
 	      assign_string (&current_stat_info.link_name, link_name);
-	      if (NAME_FIELD_SIZE <= strlen (link_name))
+	      if (NAME_FIELD_SIZE < strlen (link_name))
 		write_long_link (&current_stat_info);
 
 	      current_stat_info.stat.st_size = 0;
 	      header = start_header (p, &current_stat_info);
 	      if (!header)
 		return;
-	      strncpy (header->header.linkname, link_name, NAME_FIELD_SIZE);
-
-	      /* Force null termination.  */
-	      header->header.linkname[NAME_FIELD_SIZE - 1] = 0;
+	      tar_copy_str (header->header.linkname, link_name,
+			    NAME_FIELD_SIZE);
 
 	      header->header.typeflag = LNKTYPE;
 	      finish_header (header, block_ordinal);
@@ -1487,8 +1483,8 @@ dump_file (char *p, int top_level, dev_t parent_device)
 
 	  sizeleft = current_stat_info.stat.st_size;
 
-	  /* Don't bother opening empty, world readable files.  Also do not open
-	     files when archive is meant for /dev/null.  */
+	  /* Don't bother opening empty, world readable files.  Also do not
+	     open files when archive is meant for /dev/null.  */
 
 	  if (dev_null_output
 	      || (sizeleft == 0
@@ -1528,8 +1524,7 @@ dump_file (char *p, int top_level, dev_t parent_device)
 	    isextended = header->oldgnu_header.isextended;
 	  else
 	    isextended = 0;
-	  if (isextended)
-	    abort();
+
 	  save_typeflag = header->header.typeflag;
 	  finish_header (header, block_ordinal);
 	  if (isextended)
@@ -1699,7 +1694,7 @@ dump_file (char *p, int top_level, dev_t parent_device)
 	    }
 	  buffer[size] = '\0';
 	  assign_string (&current_stat_info.link_name, buffer);
-	  if (size >= NAME_FIELD_SIZE)
+	  if (size > NAME_FIELD_SIZE)
 	    write_long_link (&current_stat_info);
 
 	  block_ordinal = current_block_ordinal ();
@@ -1707,8 +1702,7 @@ dump_file (char *p, int top_level, dev_t parent_device)
 	  header = start_header (p, &current_stat_info);
 	  if (!header)
 	    return;
-	  strncpy (header->header.linkname, buffer, NAME_FIELD_SIZE);
-	  header->header.linkname[NAME_FIELD_SIZE - 1] = '\0';
+	  tar_copy_str (header->header.linkname, buffer, NAME_FIELD_SIZE);
 	  header->header.typeflag = SYMTYPE;
 	  finish_header (header, block_ordinal);
 	  /* nothing more to do to it */