summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDavid Kalnischkies <kalnischkies@gmail.com>2012-03-20 21:08:15 +0100
committerDavid Kalnischkies <kalnischkies@gmail.com>2012-03-20 21:08:15 +0100
commitfca9751603f4f1ee0612208c009326490dca9a53 (patch)
tree2efebf88aacd6cd3420c0205abac8afa875dba47
parent319790f4f86f595724fb2bd5aa6274d345469010 (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.cc3
-rw-r--r--test/interactive-helper/testextract.cc3
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] == '/')