Fixit: Remove bunch of coverity warnings from chrome/installer/*.

BUg=12849

Review URL: http://codereview.chromium.org/151101

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@19668 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 8 insertions, 4 deletions

diff --git a/chrome/installer/setup/uninstall.cc b/chrome/installer/setup/uninstall.cc
index 37da295..a67e01b 100644
--- a/chrome/installer/setup/uninstall.cc
+++ b/chrome/installer/setup/uninstall.cc
@@ -136,8 +136,10 @@ bool DeleteFilesAndFolders(const std::wstring& exe_path, bool system_uninstall,
   file_util::AppendToPath(&setup_exe, file_util::GetFilenameFromPath(exe_path));
 
   std::wstring temp_file;
-  file_util::CreateTemporaryFileName(&temp_file);
-  file_util::Move(setup_exe, temp_file);
+  if (!file_util::CreateTemporaryFileName(&temp_file))
+    LOG(ERROR) << "Failed to create temporary file for setup.exe.";
+  else
+    file_util::Move(setup_exe, temp_file);
 
   // Move the browser's persisted local state
   FilePath user_local_state;
@@ -145,8 +147,10 @@ bool DeleteFilesAndFolders(const std::wstring& exe_path, bool system_uninstall,
     std::wstring user_local_file(
         user_local_state.Append(chrome::kLocalStateFilename).value());
 
-    file_util::CreateTemporaryFileName(local_state_path);
-    file_util::CopyFile(user_local_file, *local_state_path);
+    if (!file_util::CreateTemporaryFileName(local_state_path))
+      LOG(ERROR) << "Failed to create temporary file for Local State.";
+    else 
+      file_util::CopyFile(user_local_file, *local_state_path);
   }
 
   LOG(INFO) << "Deleting install path " << install_path;