winevt_c 0.4.7.rc → 0.4.7.rc2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: daebe6fe343ceb5ee6a226edd4528a9aa2540c9adbb502738a483d8d30f18302
-  data.tar.gz: cf3a4619ee6294496ebb4690afcc82f64c4adccab8633c47035757789c67df52
+  metadata.gz: 549d42791fee93a96721851c2d4311affcac0518c27b3f813232909c2f714ef3
+  data.tar.gz: 15ef2c5d0abf1edcd6b87affc2c8660787d6fe67c4289cf76e8743bb3f13688e
 SHA512:
-  metadata.gz: e0e0154f80009838480068032ddc7e30a0e1d734fa5d0d322829c039048c0ba1f12f6ba2fce0002029db3b6dad510d6ea6e226ff6e090ba8b222c6413e2e0ef3
-  data.tar.gz: 606760e22e450fe632053055278a755af7438b72772dba82c270b16bd8d2ca60eb2c480f4bf81ab44a8b904625f3056717970de7fe067e765b60717aabfa3198
+  metadata.gz: 89ff8942faaf4463d92319ff68215042b5582ec3fca8488f1fbc995f7a8be91270c801cf7ddd7b02cf3bcb3b2e1f02300af26ccfadbe5c2476ca104f4f28e8f5
+  data.tar.gz: 8b6f7abca4e7d1f80e82a86133e38bcbf967e53dd14f06ee02966b8d612e6eab35e4adab06ccb0e925e0512254b8211c761a67ddd0d30a5f39aa6ed539339e24

data/ext/winevt/extconf.rb

@@ -18,6 +18,7 @@ have_library("ole32")
 $LDFLAGS << " -lwevtapi -ladvapi32 -lole32"
 $CFLAGS << " -std=c99 -fPIC -fms-extensions "
 $CXXFLAGS << " -std=c++11 -fPIC -fms-extensions "
-# $CFLAGS << " -g -O0"
+# $CFLAGS << " -g -O0 -ggdb"
+# $CXXFLAGS << " -g -O0 -ggdb"
 
 create_makefile("winevt/winevt")

data/ext/winevt/winevt_channel.c

@@ -42,7 +42,7 @@ rb_winevt_channel_each(VALUE self)
 {
   EVT_HANDLE hChannels;
   struct WinevtChannel *winevtChannel;
-  char *errBuf = NULL;
+  char errBuf[256];
   LPWSTR buffer = NULL;
   LPWSTR temp = NULL;
   DWORD bufferSize = 0;
@@ -59,7 +59,7 @@ rb_winevt_channel_each(VALUE self)
   if (hChannels) {
     winevtChannel->channels = hChannels;
   } else {
-    sprintf(errBuf, "Failed to enumerate channels with %s\n", GetLastError());
+    _snprintf_s(errBuf, 256, _TRUNCATE, "Failed to enumerate channels with %s\n", GetLastError());
     rb_raise(rb_eRuntimeError, errBuf);
   }
 
@@ -81,7 +81,7 @@ rb_winevt_channel_each(VALUE self)
           rb_raise(rb_eRuntimeError, "realloc failed");
         }
       } else {
-        sprintf(errBuf, "EvtNextChannelPath failed with %lu.\n", status);
+        _snprintf_s(errBuf, 256, _TRUNCATE, "EvtNextChannelPath failed with %lu.\n", status);
         rb_raise(rb_eRuntimeError, errBuf);
       }
     }

data/ext/winevt/winevt_utils.cpp

@@ -159,7 +159,7 @@ VALUE get_values(EVT_HANDLE handle)
   SYSTEMTIME st;
   FILETIME ft;
   CHAR strTime[128];
-  std::string sResult;
+  std::unique_ptr<CHAR[]> sResult(new CHAR[256]);
   VALUE rbObj;
 
   for (int i = 0; i < propCount; i++) {
@@ -216,12 +216,12 @@ VALUE get_values(EVT_HANDLE handle)
       rb_ary_push(userValues, rbObj);
       break;
     case EvtVarTypeSingle:
-      sprintf(&sResult[0], "%f", pRenderedValues[i].SingleVal);
-      rb_ary_push(userValues, rb_utf8_str_new_cstr(sResult.c_str()));
+      _snprintf_s(sResult.get(), 256, _TRUNCATE, "%f", pRenderedValues[i].SingleVal);
+      rb_ary_push(userValues, rb_utf8_str_new_cstr(sResult.get()));
       break;
     case EvtVarTypeDouble:
-      sprintf(&sResult[0], "%lf", pRenderedValues[i].DoubleVal);
-      rb_ary_push(userValues, rb_utf8_str_new_cstr(sResult.c_str()));
+      _snprintf_s(sResult.get(), 256, _TRUNCATE, "%lf", pRenderedValues[i].DoubleVal);
+      rb_ary_push(userValues, rb_utf8_str_new_cstr(sResult.get()));
       break;
     case EvtVarTypeBoolean:
       result = const_cast<char *>(pRenderedValues[i].BooleanVal ? "true" : "false");
@@ -246,10 +246,10 @@ VALUE get_values(EVT_HANDLE handle)
       ft.dwHighDateTime = timestamp.HighPart;
       ft.dwLowDateTime  = timestamp.LowPart;
       if (FileTimeToSystemTime( &ft, &st )) {
-        sprintf(strTime, "%04d-%02d-%02d %02d:%02d:%02d.%dZ",
-                st.wYear , st.wMonth , st.wDay ,
-                st.wHour , st.wMinute , st.wSecond,
-                st.wMilliseconds);
+        _snprintf_s(strTime, 128, _TRUNCATE, "%04d-%02d-%02d %02d:%02d:%02d.%dZ",
+                    st.wYear , st.wMonth , st.wDay ,
+                    st.wHour , st.wMinute , st.wSecond,
+                    st.wMilliseconds);
         rb_ary_push(userValues, rb_utf8_str_new_cstr(strTime));
       } else {
         rb_ary_push(userValues, rb_utf8_str_new_cstr("?"));
@@ -258,10 +258,10 @@ VALUE get_values(EVT_HANDLE handle)
     case EvtVarTypeSysTime:
       if (pRenderedValues[i].SysTimeVal != nullptr) {
         st = *pRenderedValues[i].SysTimeVal;
-        sprintf(strTime, "%04d-%02d-%02d %02d:%02d:%02d.%dZ",
-                st.wYear , st.wMonth , st.wDay ,
-                st.wHour , st.wMinute , st.wSecond,
-                st.wMilliseconds);
+        _snprintf_s(strTime, 128, _TRUNCATE, "%04d-%02d-%02d %02d:%02d:%02d.%dZ",
+                    st.wYear , st.wMonth , st.wDay ,
+                    st.wHour , st.wMinute , st.wSecond,
+                    st.wMilliseconds);
         rb_ary_push(userValues, rb_utf8_str_new_cstr(strTime));
       } else {
         rb_ary_push(userValues, rb_utf8_str_new_cstr("?"));

data/lib/winevt/version.rb

@@ -1,3 +1,3 @@
 module Winevt
-  VERSION = "0.4.7.rc"
+  VERSION = "0.4.7.rc2"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: winevt_c
 version: !ruby/object:Gem::Version
-  version: 0.4.7.rc
+  version: 0.4.7.rc2
 platform: ruby
 authors:
 - Hiroshi Hatake