diff options
author | David Kalnischkies <kalnischkies@gmail.com> | 2012-03-20 21:08:15 +0100 |
---|---|---|
committer | David Kalnischkies <kalnischkies@gmail.com> | 2012-03-20 21:08:15 +0100 |
commit | fca9751603f4f1ee0612208c009326490dca9a53 (patch) | |
tree | 2efebf88aacd6cd3420c0205abac8afa875dba47 | |
parent | 319790f4f86f595724fb2bd5aa6274d345469010 (diff) |
for these helpers we don't need too much checks,
but no warnings from gcc helps
-rw-r--r-- | test/interactive-helper/extract-control.cc | 3 | ||||
-rw-r--r-- | test/interactive-helper/testextract.cc | 3 |
2 files changed, 3 insertions, 3 deletions
diff --git a/test/interactive-helper/extract-control.cc b/test/interactive-helper/extract-control.cc index a1b3600aa..3f7feabcb 100644 --- a/test/interactive-helper/extract-control.cc +++ b/test/interactive-helper/extract-control.cc @@ -21,8 +21,7 @@ bool ExtractMember(const char *File,const char *Member) if (Extract.Control == 0) return true; - write(STDOUT_FILENO,Extract.Control,Extract.Length); - return true; + return write(STDOUT_FILENO,Extract.Control,Extract.Length) != -1; } int main(int argc, const char *argv[]) diff --git a/test/interactive-helper/testextract.cc b/test/interactive-helper/testextract.cc index f7ddb72f0..8dfe68fc9 100644 --- a/test/interactive-helper/testextract.cc +++ b/test/interactive-helper/testextract.cc @@ -66,7 +66,8 @@ bool Go(int argc,char *argv[]) while (feof(F) == 0) { char Line[300]; - fgets(Line,sizeof(Line),F); + if (fgets(Line,sizeof(Line),F) == NULL) + return false; Itm.Name = _strstrip(Line); Itm.Type = pkgDirStream::Item::File; if (Line[strlen(Line)-1] == '/') |