- add missing checks for return values, fix a typing bug (Ralf Corsepius)
authorpmatilai <pmatilai>
Sun, 28 May 2006 09:35:24 +0000 (09:35 +0000)
committerpmatilai <pmatilai>
Sun, 28 May 2006 09:35:24 +0000 (09:35 +0000)
apt-pkg/acquire.cc
apt-pkg/clean.cc
apt-pkg/contrib/cdromutl.cc
apt-pkg/contrib/strutl.cc
methods/http.cc

index 0dcd8d8..3f2f3f8 100644 (file)
@@ -438,7 +438,8 @@ bool pkgAcquire::Clean(string Dir)
         unlink(Dir->d_name);
    };
    
-   chdir(StartDir.c_str());
+   if(chdir(StartDir.c_str()) != 0)
+      return false;
    closedir(D);
    return true;   
 }
index 28267d0..3b6bfe8 100644 (file)
@@ -56,7 +56,8 @@ bool pkgArchiveCleaner::Go(string Dir,pkgCache &Cache)
       struct stat St;
       if (stat(Dir->d_name,&St) != 0)
       {
-        chdir(StartDir.c_str());
+        if(chdir(StartDir.c_str()) != 0)
+           return _error->Errno("chdir",_("Failed to chdir to %s"),StartDir.c_str());
         closedir(D);
         return _error->Errno("stat",_("Unable to stat %s."),Dir->d_name);
       }
@@ -120,7 +121,8 @@ bool pkgArchiveCleaner::Go(string Dir,pkgCache &Cache)
       Erase(Dir->d_name,Pkg,Ver,St);
    };
    
-   chdir(StartDir.c_str());
+   if(chdir(StartDir.c_str()) != 0)
+      return false;
    closedir(D);
    return true;   
 }
index a6fe4d5..b1690f1 100644 (file)
@@ -204,7 +204,8 @@ bool IdentCdrom(string CD,string &Res,unsigned int Version)
       Hash.Add(Dir->d_name);
    };
    
-   chdir(StartDir.c_str());
+   if (chdir(StartDir.c_str()) != 0)
+      return false;
    closedir(D);
    
    // Some stats from the fsys
index 1c7f7cd..a90943f 100644 (file)
@@ -1081,9 +1081,9 @@ void URI::CopyFrom(string U)
    }   
    
    // Now we parse the RFC 2732 [] hostnames.
-   unsigned long PortEnd = 0;
+   string::size_type PortEnd = 0;
    InBracket = false;
-   for (unsigned I = 0; I != Host.length();)
+   for (string::size_type  I = 0; I != Host.length();)
    {
       if (Host[I] == '[')
       {
index 6744122..9156596 100644 (file)
@@ -1018,7 +1018,8 @@ int HttpMethod::DealWithHeaders(FetchResult &Res,ServerState *Srv)
    if (Srv->StartPos >= 0)
    {
       Res.ResumePoint = Srv->StartPos;
-      ftruncate(File->Fd(),Srv->StartPos);
+      if (ftruncate(File->Fd(),Srv->StartPos) != 0)
+         return 5;
    }
       
    // Set the start point