- eliminate several GCC warnings and C'isms, part 1 (Ralf)
authorpmatilai <pmatilai>
Fri, 9 Jun 2006 14:38:39 +0000 (14:38 +0000)
committerpmatilai <pmatilai>
Fri, 9 Jun 2006 14:38:39 +0000 (14:38 +0000)
cmdline/apt-cache.cc
cmdline/apt-cdrom.cc
cmdline/apt-shell.cc
tools/genpkglist.cc
tools/gensrclist.cc

index 6d4e748..9eaef54 100644 (file)
@@ -462,8 +462,7 @@ bool DumpAvail(CommandLine &Cmd)
         const char *Start;
         const char *End;
         P.GetRec(Start,End);
-        fwrite(Start,End-Start,1,stdout);
-        putc('\n',stdout);
+        cout << string(Start,End-Start) << endl;
       }
    }
    return !_error->PendingError();
@@ -1492,8 +1491,7 @@ bool DisplayRecord(pkgCache::VerIterator V)
    const char *Start;
    const char *End;
    P.GetRec(Start,End);
-   fwrite(Start,End-Start,1,stdout);
-   putc('\n',stdout);
+   cout << string(Start,End-Start) << endl;
 #else
    // Check and load the package list file
    pkgCache::PkgFileIterator I = Vf.File();
@@ -1671,8 +1669,7 @@ bool Search(CommandLine &CmdL)
            const char *Start;
            const char *End;
            P.GetRec(Start,End);
-           fwrite(Start,End-Start,1,stdout);
-           putc('\n',stdout);
+           cout << string(Start,End-Start) << endl;
         }       
         else
            printf("%s - %s\n",P.Name().c_str(),P.ShortDesc().c_str());
index 266cb83..3388aa2 100644 (file)
@@ -611,8 +611,11 @@ bool DoAdd(CommandLine &)
       return false;
    }
    
-   chdir(StartDir.c_str());
-
+   if (chdir(StartDir.c_str()) != 0)
+   {
+      return false;
+   }
+   
    if (_config->FindB("Debug::aptcdrom",false) == true)
    {
       cout << _("I found (binary):") << endl;
index 69daa39..f77b363 100644 (file)
@@ -3189,8 +3189,7 @@ bool DisplayRecord(pkgCache::VerIterator V)
    const char *Start;
    const char *End;
    P.GetRec(Start,End);
-   fwrite(Start,End-Start,1,stdout);
-   putc('\n',stdout);
+   cout << string(Start,End-Start) << endl;
 #else
    // Check and load the package list file
    pkgCache::PkgFileIterator I = Vf.File();
@@ -3339,11 +3338,10 @@ bool Search(CommandLine &CmdL)
            const char *Start;
            const char *End;
            P.GetRec(Start,End);
-           fwrite(Start,End-Start,1,stdout);
-           putc('\n',stdout);
+           cout << string(Start,End-Start) << endl;
         }       
         else
-           printf("%s - %s\n",P.Name().c_str(),P.ShortDesc().c_str());
+           cout << P.Name() << " - " << P.ShortDesc() << endl;
       }
    }
    
index 220a5d1..aa100fe 100644 (file)
@@ -560,9 +560,13 @@ int main(int argc, char ** argv)
    }
    
    {
-      char cwd[200];
+      char cwd[PATH_MAX];
       
-      getcwd(cwd, 200);
+      if (getcwd(cwd, PATH_MAX) == 0)
+      {
+         cerr << argv[0] << strerror(errno) << endl;
+         exit(1);
+      }
       if (*op_dir != '/') {
         rpmsdir = string(cwd) + "/" + string(op_dir);
       } else {
@@ -581,7 +585,11 @@ int main(int argc, char ** argv)
       return 1;
    }
    
-   chdir(rpmsdir.c_str());
+   if (chdir(rpmsdir.c_str()) != 0)
+   {
+      cerr << argv[0] << strerror(errno) << endl;
+      return 1;
+   }
    
    if (pkgListSuffix != NULL)
           pkglist_path = pkglist_path + "/base/pkglist." + pkgListSuffix;
index f0a95c7..992c6f7 100644 (file)
@@ -143,7 +143,7 @@ int headerGetRawEntry(Header h, int_32 tag, int_32 * type,
 int main(int argc, char ** argv) 
 {
    char buf[300];
-   char cwd[200];
+   char cwd[PATH_MAX];
    string srpmdir;
    FD_t outfd, fd;
    struct dirent **dirEntries;
@@ -205,7 +205,12 @@ int main(int argc, char ** argv)
    
    md5cache = new CachedMD5(string(arg_dir)+string(arg_suffix), "gensrclist");
 
-   getcwd(cwd, 200);
+   if(getcwd(cwd, PATH_MAX) == 0)
+   {
+      cerr << argv[0] << ":" << strerror(errno) << endl;
+      exit(1);
+   }
+   
    if (*arg_dir != '/') {
       strcpy(buf, cwd);
       strcat(buf, "/");
@@ -253,7 +258,11 @@ int main(int argc, char ** argv)
       return 1;
    }
 
-   chdir(buf);
+   if(chdir(buf) != 0)
+   {
+      cerr << argv[0] << ":" << strerror(errno) << endl;
+      exit(1);
+   }
    
    if (srcListSuffix != NULL)
       sprintf(buf, "%s/srclist.%s", cwd, srcListSuffix);