win: Only call GetFileVersionInfo() once per module

snapshot/win/module_snapshot_win.cc

 // Copyright 2015 The Crashpad Authors. All rights reserved.
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.
 // You may obtain a copy of the License at
 //
 //     http://www.apache.org/licenses/LICENSE-2.0
 //
 // Unless required by applicable law or agreed to in writing, software
 // distributed under the License is distributed on an "AS IS" BASIS,
@@ skipping 15 matching lines @@
 
 ModuleSnapshotWin::ModuleSnapshotWin()
     : ModuleSnapshot(),
       name_(),
       pdb_name_(),
       uuid_(),
       pe_image_reader_(),
       process_reader_(nullptr),
       timestamp_(0),
       age_(0),
-      initialized_() {
+      initialized_(),
+      vs_fixed_file_info_(),
+      initialized_vs_fixed_file_info_() {
 }
 
 ModuleSnapshotWin::~ModuleSnapshotWin() {
 }
 
 bool ModuleSnapshotWin::Initialize(
     ProcessReaderWin* process_reader,
     const ProcessInfo::Module& process_reader_module) {
   INITIALIZATION_STATE_SET_INITIALIZING(initialized_);
 
@@ skipping 53 matching lines @@
 time_t ModuleSnapshotWin::Timestamp() const {
   INITIALIZATION_STATE_DCHECK_VALID(initialized_);
   return timestamp_;
 }
 
 void ModuleSnapshotWin::FileVersion(uint16_t* version_0,
                                     uint16_t* version_1,
                                     uint16_t* version_2,
                                     uint16_t* version_3) const {
   INITIALIZATION_STATE_DCHECK_VALID(initialized_);
-  VS_FIXEDFILEINFO ffi;
-  if (GetModuleVersionAndType(base::FilePath(name_), &ffi)) {
-    *version_0 = ffi.dwFileVersionMS >> 16;
-    *version_1 = ffi.dwFileVersionMS & 0xffff;
-    *version_2 = ffi.dwFileVersionLS >> 16;
-    *version_3 = ffi.dwFileVersionLS & 0xffff;
+  const VS_FIXEDFILEINFO* ffi = VSFixedFileInfo();
+  if (ffi) {
+    *version_0 = ffi->dwFileVersionMS >> 16;
+    *version_1 = ffi->dwFileVersionMS & 0xffff;
+    *version_2 = ffi->dwFileVersionLS >> 16;
+    *version_3 = ffi->dwFileVersionLS & 0xffff;
   } else {
     *version_0 = 0;
     *version_1 = 0;
     *version_2 = 0;
     *version_3 = 0;
   }
 }
 
 void ModuleSnapshotWin::SourceVersion(uint16_t* version_0,
                                       uint16_t* version_1,
                                       uint16_t* version_2,
                                       uint16_t* version_3) const {
   INITIALIZATION_STATE_DCHECK_VALID(initialized_);
-  VS_FIXEDFILEINFO ffi;
-  if (GetModuleVersionAndType(base::FilePath(name_), &ffi)) {
-    *version_0 = ffi.dwProductVersionMS >> 16;
-    *version_1 = ffi.dwProductVersionMS & 0xffff;
-    *version_2 = ffi.dwProductVersionLS >> 16;
-    *version_3 = ffi.dwProductVersionLS & 0xffff;
+  const VS_FIXEDFILEINFO* ffi = VSFixedFileInfo();
+  if (ffi) {
+    *version_0 = ffi->dwProductVersionMS >> 16;
+    *version_1 = ffi->dwProductVersionMS & 0xffff;
+    *version_2 = ffi->dwProductVersionLS >> 16;
+    *version_3 = ffi->dwProductVersionLS & 0xffff;
   } else {
     *version_0 = 0;
     *version_1 = 0;
     *version_2 = 0;
     *version_3 = 0;
   }
 }
 
 ModuleSnapshot::ModuleType ModuleSnapshotWin::GetModuleType() const {
   INITIALIZATION_STATE_DCHECK_VALID(initialized_);
-  VS_FIXEDFILEINFO ffi;
-  if (GetModuleVersionAndType(base::FilePath(name_), &ffi)) {
-    if (ffi.dwFileType == VFT_APP)
-      return ModuleSnapshot::kModuleTypeExecutable;
-    if (ffi.dwFileType == VFT_DLL)
-      return ModuleSnapshot::kModuleTypeSharedLibrary;
-    if (ffi.dwFileType == VFT_DRV || ffi.dwFileType == VFT_VXD)
-      return ModuleSnapshot::kModuleTypeLoadableModule;
+  const VS_FIXEDFILEINFO* ffi = VSFixedFileInfo();
+  if (ffi) {
+    switch (ffi->dwFileType) {
+      case VFT_APP:
+        return ModuleSnapshot::kModuleTypeExecutable;
+      case VFT_DLL:
+        return ModuleSnapshot::kModuleTypeSharedLibrary;
+      case VFT_DRV:
+      case VFT_VXD:
+        return ModuleSnapshot::kModuleTypeLoadableModule;
+    }
   }
   return ModuleSnapshot::kModuleTypeUnknown;
 }
 
 void ModuleSnapshotWin::UUIDAndAge(crashpad::UUID* uuid, uint32_t* age) const {
   INITIALIZATION_STATE_DCHECK_VALID(initialized_);
   *uuid = uuid_;
   *age = age_;
 }
 
@@ skipping 30 matching lines @@
 
   options->crashpad_handler_behavior =
       CrashpadInfoClientOptions::TriStateFromCrashpadInfo(
           crashpad_info.crashpad_handler_behavior);
 
   options->system_crash_reporter_forwarding =
       CrashpadInfoClientOptions::TriStateFromCrashpadInfo(
           crashpad_info.system_crash_reporter_forwarding);
 }
 
+const VS_FIXEDFILEINFO* ModuleSnapshotWin::VSFixedFileInfo() const {
+  INITIALIZATION_STATE_DCHECK_VALID(initialized_);
+
+  if (initialized_vs_fixed_file_info_.is_uninitialized()) {
+    initialized_vs_fixed_file_info_.set_invalid();
+    if (GetModuleVersionAndType(base::FilePath(name_), &vs_fixed_file_info_)) {
+      initialized_vs_fixed_file_info_.set_valid();
+    }
+  }
+
+  return initialized_vs_fixed_file_info_.is_valid() ? &vs_fixed_file_info_
+                                                    : nullptr;
+}
+
 }  // namespace internal
 }  // namespace crashpad