Index: modules/system/system.admin.inc
===================================================================
RCS file: /cvs/drupal/drupal/modules/system/system.admin.inc,v
retrieving revision 1.326
diff -u -r1.326 system.admin.inc
--- modules/system/system.admin.inc	23 Nov 2010 02:54:13 -0000	1.326
+++ modules/system/system.admin.inc	21 Dec 2010 19:58:36 -0000
@@ -621,13 +621,13 @@
 
   // If the user provided a path for a logo or favicon file, make sure a file
   // exists at that path.
-  if ($form_state['values']['logo_path']) {
+  if (!empty($form_state['values']['logo_path'])) {
     $path = _system_theme_settings_validate_path($form_state['values']['logo_path']);
     if (!$path) {
       form_set_error('logo_path', t('The custom logo path is invalid.'));
     }
   }
-  if ($form_state['values']['favicon_path']) {
+  if (!empty($form_state['values']['favicon_path'])) {
     $path = _system_theme_settings_validate_path($form_state['values']['favicon_path']);
     if (!$path) {
       form_set_error('favicon_path', t('The custom favicon path is invalid.'));
@@ -669,16 +669,16 @@
 
   // If the user uploaded a new logo or favicon, save it to a permanent location
   // and use it in place of the default theme-provided file.
-  if ($file = $values['logo_upload']) {
+  if (!empty($values['logo_upload'])) {
     unset($values['logo_upload']);
-    $filename = file_unmanaged_copy($file->uri);
+    $filename = file_unmanaged_copy($values['logo_upload']->uri);
     $values['default_logo'] = 0;
     $values['logo_path'] = $filename;
     $values['toggle_logo'] = 1;
   }
-  if ($file = $values['favicon_upload']) {
+  if (!empty($values['favicon_upload'])) {
     unset($values['favicon_upload']);
-    $filename = file_unmanaged_copy($file->uri);
+    $filename = file_unmanaged_copy($values['logo_upload']->uri);
     $values['default_favicon'] = 0;
     $values['favicon_path'] = $filename;
     $values['toggle_favicon'] = 1;