Write two EOF blocks for tar rather than one

PR: 28776
Obtained from: Oliver Rossmueller


git-svn-id: https://svn.apache.org/repos/asf/ant/core/trunk@277339 13f79535-47bb-0310-9956-ffa450edef68

CONTRIBUTORS

@@ -155,6 +155,7 @@ Nick Pellow
 Nicola Ken Barozzi
 Nico Seessle
 Nigel Magnay
+Oliver Rossmueller
 Patrick C. Beard
 Patrick Chanezon
 Patrick G. Heck (Gus Heck)

WHATSNEW

@@ -123,6 +123,9 @@ Changes that could break older environments:
 * The subant task used the canonical version of a file path. This
   has been changed to use the absolute path. Bugzilla 30438.
 
+* Tar now writes two EOF blocks rather than one.
+  Bugzilla report 28776
+
 Other changes:
 --------------
 

src/main/org/apache/tools/tar/TarOutputStream.java

@@ -96,9 +96,12 @@ public class TarOutputStream extends FilterOutputStream {
 
     /**
      * Ends the TAR archive without closing the underlying OutputStream.
-     * The result is that the EOF record of nulls is written.
+     * The result is that the two EOF records of nulls are written.
      */
     public void finish() throws IOException {
+        // See Bugzilla 28776 for a discussion on this
+        // http://issues.apache.org/bugzilla/show_bug.cgi?id=28776
+        this.writeEOFRecord();
         this.writeEOFRecord();
     }