More robust use of $_FILES['imagefile']

diff --git a/user/edit.php b/user/edit.php
index 4eb3c328a6c80ec7c42f966707456db5e76ba578..71371fb7f2719a191eb56b0e5976ee850ccf1063 100644 (file)
--- a/user/edit.php
+++ b/user/edit.php
@@ -46,6 +46,10 @@
         $usernew->firstname = strip_tags($usernew->firstname);
         $usernew->lastname  = strip_tags($usernew->lastname);
 
+        if (empty($_FILES['imagefile'])) {
+            $_FILES['imagefile'] = NULL;    // To avoid using uninitialised variable later
+        }
+
         if (find_form_errors($user, $usernew, $err)) {
             if ($filename = valid_uploaded_file($_FILES['imagefile'])) { 
                 $usernew->picture = save_user_image($user->id, $filename);