Eliminate resource leaks from zip::ZipFiles and

chrome/utility/chrome_content_utility_client.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/utility/chrome_content_utility_client.h"
 
 #include "base/command_line.h"
 #include "base/files/file_path.h"
 #include "base/memory/ref_counted.h"
 #include "base/time/time.h"
@@ skipping 231 matching lines @@
 void ChromeContentUtilityClient::OnDecodeImage(
     const std::vector<unsigned char>& encoded_data, bool shrink_to_fit) {
   DecodeImageAndSend(encoded_data, shrink_to_fit);
 }
 
 #if defined(OS_CHROMEOS)
 void ChromeContentUtilityClient::OnCreateZipFile(
     const base::FilePath& src_dir,
     const std::vector<base::FilePath>& src_relative_paths,
     const base::FileDescriptor& dest_fd) {
+  // dest_fd should be closed in the function. See ipc/ipc_message_util.h for
+  // details.
+  base::ScopedFD fd_closer(dest_fd.fd);
   bool succeeded = true;
 
   // Check sanity of source relative paths. Reject if path is absolute or
   // contains any attempt to reference a parent directory ("../" tricks).
   for (std::vector<base::FilePath>::const_iterator iter =
            src_relative_paths.begin(); iter != src_relative_paths.end();
        ++iter) {
     if (iter->IsAbsolute() || iter->ReferencesParent()) {
       succeeded = false;
       break;
@@ skipping 84 matching lines @@
     const std::string& mime_type, int64 total_size, bool get_attached_images) {
   // Only one IPCDataSource may be created and added to the list of handlers.
   metadata::IPCDataSource* source = new metadata::IPCDataSource(total_size);
   handlers_.push_back(source);
 
   metadata::MediaMetadataParser* parser = new metadata::MediaMetadataParser(
       source, mime_type, get_attached_images);
   parser->Start(base::Bind(&FinishParseMediaMetadata, base::Owned(parser)));
 }
 #endif