[Spice-devel] [vdagent-win PATCH] log: avoid crash on failure in log_version

[Frediano Ziglio]

The "throw" statement alone are used to throw an already existing
exception however in log_version they were meant to raise an exception.
This would have caused a termination of the program.

Signed-off-by: Frediano Ziglio <fziglio at redhat.com>
---
 common/vdlog.cpp | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)

diff --git a/common/vdlog.cpp b/common/vdlog.cpp
index c983127..0a4c816 100644
--- a/common/vdlog.cpp
+++ b/common/vdlog.cpp
@@ -79,33 +79,34 @@ void VDLog::printf(const char* format, ...)
 
 void log_version()
 {
-    DWORD handle;
-    TCHAR module_fname[MAX_PATH];
+    if (![]() -> bool {
+        DWORD handle;
+        TCHAR module_fname[MAX_PATH];
 
-    try {
         if (!GetModuleFileName(NULL, module_fname, MAX_PATH)) {
-            throw;
+            return false;
         }
         DWORD version_inf_size = GetFileVersionInfoSize(module_fname, &handle);
         if (version_inf_size == 0) {
-            throw;
+            return false;
         }
         std::vector<TCHAR> info_buf(version_inf_size);
         if (!GetFileVersionInfo(module_fname, handle, version_inf_size, &info_buf[0])) {
-            throw;
+            return false;
         }
         UINT size;
         VS_FIXEDFILEINFO* file_info;
         if (!VerQueryValue(&info_buf[0], L"\\", (VOID**)&file_info, &size) ||
                 size < sizeof(VS_FIXEDFILEINFO)) {
-            throw;
+            return false;
         }
         vd_printf("%lu.%lu.%lu.%lu",
             file_info->dwFileVersionMS >> 16,
             file_info->dwFileVersionMS & 0x0ffff,
             file_info->dwFileVersionLS >> 16,
             file_info->dwFileVersionLS & 0x0ffff);
-    } catch (...) {
+        return true;
+    }()) {
         vd_printf("get version info failed");
     }
 }
-- 
2.13.3