don't move imported files to target path if they are not correct file…

… type
emoncms · May 10, 2016 · 8ad0aca · 8ad0aca
1 parent 9a79edf
commit 8ad0aca
Showing 1 changed file with 14 additions and 12 deletions.
diff --git a/backup/backup_controller.php b/backup/backup_controller.php
@@ -83,23 +83,25 @@ function backup_controller()
         {
             $result="Sorry, only .tar.gz files are allowed.";
             $uploadOk = 0;
-        }
 
-        if ((move_uploaded_file($_FILES['file']['tmp_name'], $target_path)) && ($uploadOk == 1)) {
+            if ((move_uploaded_file($_FILES['file']['tmp_name'], $target_path)) && ($uploadOk == 1)) {
+
+                $fh = @fopen($import_flag,"w");
+                if (!$fh) {
+                    $result = "ERROR: Can't write the flag $import_flag.";
+                } else {
+                    fwrite($fh,"$import_script>$import_logfile");
+                    $result = "Backup flag set";
+                }
+                @fclose($fh);
 
-            $fh = @fopen($import_flag,"w");
-            if (!$fh) {
-                $result = "ERROR: Can't write the flag $import_flag.";
+                header('Location: '.$path.'backup');
             } else {
-                fwrite($fh,"$import_script>$import_logfile");
-                $result = "Backup flag set";
+                $result = "Sorry, there was an error uploading the file";
             }
-            @fclose($fh);
-
-            header('Location: '.$path.'backup');
-        } else {
-            $result = "Sorry, there was an error uploading the file";
         }
+
+
     }
 
     return array('content'=>$result);