fixing overrelease and out-of-bounds write, fixes #13725

[wxWidgets.git] / src / common / fs_arc.cpp

diff --git a/src/common/fs_arc.cpp b/src/common/fs_arc.cpp
index 9c92942c23bb9e44560abc63ffc9931a612065fb..4b4c382cac7e1ebea9721e357f464e27997c24e4 100644 (file)
--- a/src/common/fs_arc.cpp
+++ b/src/common/fs_arc.cpp
@@ -1,5 +1,5 @@
 /////////////////////////////////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////////////////

-// Name:        fs_arc.cpp
+// Name:        src/common/fs_arc.cpp

 // Purpose:     wxArchive file system
 // Author:      Vaclav Slavik, Mike Wetherell
 // Copyright:   (c) 1999 Vaclav Slavik, (c) 2006 Mike Wetherell
 // Purpose:     wxArchive file system
 // Author:      Vaclav Slavik, Mike Wetherell
 // Copyright:   (c) 1999 Vaclav Slavik, (c) 2006 Mike Wetherell


@@ -17,12 +17,17 @@
 
 #include "wx/fs_arc.h"
 
 
 #include "wx/fs_arc.h"
 

-#ifndef WXPRECOMP
+#ifndef WX_PRECOMP

     #include "wx/intl.h"
     #include "wx/log.h"
 #endif
 
     #include "wx/intl.h"
     #include "wx/log.h"
 #endif
 

-#include "wx/archive.h"
+#if WXWIN_COMPATIBILITY_2_6 && wxUSE_ZIPSTREAM
+    #include "wx/zipstrm.h"
+#else
+    #include "wx/archive.h"
+#endif
+

 #include "wx/private/fileback.h"
 
 //---------------------------------------------------------------------------
 #include "wx/private/fileback.h"
 
 //---------------------------------------------------------------------------


@@ -130,10 +135,8 @@ wxArchiveFSEntry *wxArchiveFSCacheDataImpl::AddToCache(wxArchiveEntry *entry)
 
 void wxArchiveFSCacheDataImpl::CloseStreams()
 {
 
 void wxArchiveFSCacheDataImpl::CloseStreams()
 {

-    delete m_archive;
-    m_archive = NULL;
-    delete m_stream;
-    m_stream = NULL;
+    wxDELETE(m_archive);
+    wxDELETE(m_stream);

 }
 
 wxArchiveEntry *wxArchiveFSCacheDataImpl::Get(const wxString& name)
 }
 
 wxArchiveEntry *wxArchiveFSCacheDataImpl::Get(const wxString& name)


@@ -355,7 +358,7 @@ wxFSFile* wxArchiveFSHandler::OpenFile(
         right = rightPart.GetFullPath(wxPATH_UNIX);
     }
 
         right = rightPart.GetFullPath(wxPATH_UNIX);
     }
 

-    if (right.GetChar(0) == wxT('/')) right = right.Mid(1);
+    if (!right.empty() && right.GetChar(0) == wxT('/')) right = right.Mid(1);

 
     if (!m_cache)
         m_cache = new wxArchiveFSCache;
 
     if (!m_cache)
         m_cache = new wxArchiveFSCache;


@@ -390,20 +393,30 @@ wxFSFile* wxArchiveFSHandler::OpenFile(
     }
 
     wxArchiveInputStream *s = factory->NewStream(leftStream);
     }
 
     wxArchiveInputStream *s = factory->NewStream(leftStream);

+    if ( !s )
+        return NULL;
+

     s->OpenEntry(*entry);
 
     s->OpenEntry(*entry);
 

-    if (s && s->IsOk())
-        return new wxFSFile(s,
-                            key + right,
-                            GetMimeTypeFromExt(location),
-                            GetAnchor(location)
+    if (!s->IsOk())
+    {
+        delete s;
+        return NULL;
+    }
+
+#if WXWIN_COMPATIBILITY_2_6 && wxUSE_ZIPSTREAM
+    if (wxDynamicCast(factory, wxZipClassFactory))
+        ((wxZipInputStream*)s)->m_allowSeeking = true;
+#endif // WXWIN_COMPATIBILITY_2_6
+
+    return new wxFSFile(s,
+                        key + right,
+                        wxEmptyString,
+                        GetAnchor(location)

 #if wxUSE_DATETIME
 #if wxUSE_DATETIME

-                            , entry->GetDateTime()
+                        , entry->GetDateTime()

 #endif // wxUSE_DATETIME
 #endif // wxUSE_DATETIME

-                            );
-
-    delete s;
-    return NULL;
+                        );

 }
 
 wxString wxArchiveFSHandler::FindFirst(const wxString& spec, int flags)
 }
 
 wxString wxArchiveFSHandler::FindFirst(const wxString& spec, int flags)