From cf1cdb3328dacde829db8bbb3db4f5373ac07d61 Mon Sep 17 00:00:00 2001
From: David Kalnischkies <kalnischkies@gmail.com>
Date: Mon, 6 Sep 2010 12:17:15 +0200
Subject: * methods/{gzip,bzip2}.cc:   - empty files can never be valid
 archives (Closes: #595691)

---
 methods/bzip2.cc | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

(limited to 'methods/bzip2.cc')

diff --git a/methods/bzip2.cc b/methods/bzip2.cc
index 241f21c66..c668141a2 100644
--- a/methods/bzip2.cc
+++ b/methods/bzip2.cc
@@ -56,12 +56,9 @@ bool Bzip2Method::Fetch(FetchItem *Itm)
    // Open the source and destination files
    FileFd From(Path,FileFd::ReadOnly);
 
-   // if the file is empty, just rename it and return
-   if(From.Size() == 0) 
-   {
-      rename(Path.c_str(), Itm->DestFile.c_str());
-      return true;
-   }
+   // FIXME add an error message saying that empty files can't be valid archives
+   if(From.Size() == 0)
+      return false;
 
    int GzOut[2];   
    if (pipe(GzOut) < 0)
-- 
cgit v1.2.3