Use some semantically more correct buffer operations

[apt.git] / apt-inst / contrib / arfile.cc

diff --git a/apt-inst/contrib/arfile.cc b/apt-inst/contrib/arfile.cc
index 8018f4d30792dd99a4fc67da30850f05081edce1..905110781a9afdecd43e5583326ab1a75e762871 100644 (file)
--- a/apt-inst/contrib/arfile.cc
+++ b/apt-inst/contrib/arfile.cc
@@ -6,7 +6,7 @@
    AR File - Handle an 'AR' archive
    
    AR Archives have plain text headers at the start of each file
    AR File - Handle an 'AR' archive
    
    AR Archives have plain text headers at the start of each file

-   section. The headers are aligned on a 2 byte boundry.
+   section. The headers are aligned on a 2 byte boundary.

    
    Information about the structure of AR files can be found in ar(5)
    on a BSD system, or in the binutils source.
    
    Information about the structure of AR files can be found in ar(5)
    on a BSD system, or in the binutils source.


@@ -14,13 +14,19 @@
    ##################################################################### */
                                                                        /*}}}*/
 // Include Files                                                       /*{{{*/
    ##################################################################### */
                                                                        /*}}}*/
 // Include Files                                                       /*{{{*/

+#include<config.h>
+

 #include <apt-pkg/arfile.h>
 #include <apt-pkg/strutl.h>
 #include <apt-pkg/arfile.h>
 #include <apt-pkg/strutl.h>

+#include <apt-pkg/fileutl.h>

 #include <apt-pkg/error.h>
 
 #include <apt-pkg/error.h>
 

-#include <stdlib.h>
-                                                                       /*}}}*/
+#include <string.h>
+#include <sys/types.h>
+#include <string>
+

 #include <apti18n.h>
 #include <apti18n.h>

+                                                                       /*}}}*/

 
 struct ARArchive::MemberHeader
 {
 
 struct ARArchive::MemberHeader
 {


@@ -60,7 +66,7 @@ ARArchive::~ARArchive()
    byte plain text header then the file data, another header, data, etc */
 bool ARArchive::LoadHeaders()
 {
    byte plain text header then the file data, another header, data, etc */
 bool ARArchive::LoadHeaders()
 {

-   signed long Left = File.Size();
+   off_t Left = File.Size();

    
    // Check the magic byte
    char Magic[8];
    
    // Check the magic byte
    char Magic[8];


@@ -102,7 +108,10 @@ bool ARArchive::LoadHeaders()
            return _error->Error(_("Invalid archive member header"));
         }
         if (File.Read(S,Len) == false)
            return _error->Error(_("Invalid archive member header"));
         }
         if (File.Read(S,Len) == false)

+        {
+           delete Memb;

            return false;
            return false;

+        }

         S[Len] = 0;
         Memb->Name = S;
         Memb->Size -= Len;
         S[Len] = 0;
         Memb->Name = S;
         Memb->Size -= Len;


@@ -112,11 +121,11 @@ bool ARArchive::LoadHeaders()
       {
         unsigned int I = sizeof(Head.Name) - 1;
         for (; Head.Name[I] == ' ' || Head.Name[I] == '/'; I--);
       {
         unsigned int I = sizeof(Head.Name) - 1;
         for (; Head.Name[I] == ' ' || Head.Name[I] == '/'; I--);

-        Memb->Name = string(Head.Name,I+1);
+        Memb->Name = std::string(Head.Name,I+1);

       }
 
       // Account for the AR header alignment 
       }
 
       // Account for the AR header alignment 

-      unsigned Skip = Memb->Size % 2;
+      off_t Skip = Memb->Size % 2;

       
       // Add it to the list
       Memb->Next = List;
       
       // Add it to the list
       Memb->Next = List;


@@ -124,7 +133,7 @@ bool ARArchive::LoadHeaders()
       Memb->Start = File.Tell();
       if (File.Skip(Memb->Size + Skip) == false)
         return false;
       Memb->Start = File.Tell();
       if (File.Skip(Memb->Size + Skip) == false)
         return false;

-      if (Left < (signed)(Memb->Size + Skip))
+      if (Left < (off_t)(Memb->Size + Skip))

         return _error->Error(_("Archive is too short"));
       Left -= Memb->Size + Skip;
    }   
         return _error->Error(_("Archive is too short"));
       Left -= Memb->Size + Skip;
    }