[Libreoffice-commits] .: package/source

Libreoffice Gerrit user logerrit at kemper.freedesktop.org
Mon Sep 24 00:27:43 PDT 2012


 package/source/zipapi/ZipFile.cxx |    4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

New commits:
commit 5db7ac239278634c39cbb15f0173db0524b5dcd6
Author: Fridrich Å trba <fridrich.strba at bluewin.ch>
Date:   Mon Sep 24 09:27:15 2012 +0200

    fdo#49819, fdo#54609: Do not consider timestamp differences as corruption
    
    MSO 2010 can produce a files with different timestamps in the
    central directory entry and local file header
    
    Change-Id: Ic6fc0b2fad96eb30babdd7e6ef4a0175936da4c5

diff --git a/package/source/zipapi/ZipFile.cxx b/package/source/zipapi/ZipFile.cxx
index b655d37..83984d6 100644
--- a/package/source/zipapi/ZipFile.cxx
+++ b/package/source/zipapi/ZipFile.cxx
@@ -693,9 +693,11 @@ sal_Bool ZipFile::readLOC( ZipEntry &rEntry )
         // Do *not* compare nMethod / nHow, older versions with
         // encrypted streams write mismatching DEFLATE/STORE pairs
         // there.
+        // Do *not* compare timestamps, since MSO 2010 can produce documents
+        // with timestamp difference in the central directory entry and local
+        // file header.
         bBroken = rEntry.nVersion != nVersion
                         || (rEntry.nFlag & ~6L) != (nFlag & ~6L)
-                        || rEntry.nTime != nTime
                         || rEntry.nPathLen != nPathLen
                         || !rEntry.sPath.equals( sLOCPath );
     }


More information about the Libreoffice-commits mailing list