Merge pull request #1482 from dpogue/crashhandler-warnings

Resolve 2 clang warnings in pfCrashHandler
H-uru · Sep 9, 2023 · be3910d · be3910d
2 parents 70f379c + 5ae6ed9
commit be3910d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/Sources/Plasma/FeatureLib/pfCrashHandler/plCrashSrv.cpp b/Sources/Plasma/FeatureLib/pfCrashHandler/plCrashSrv.cpp
@@ -83,7 +83,7 @@ plCrashSrv::~plCrashSrv()
 static inline bool IGetCrashedThreadContext(HANDLE process, const LPEXCEPTION_POINTERS ptrs, LPCONTEXT context)
 {
     if (process == GetCurrentProcess()) {
-        memcpy(context, ptrs->ContextRecord, sizeof(ptrs->ContextRecord));
+        memcpy(context, ptrs->ContextRecord, sizeof(CONTEXT));
         return true;
     }
 

diff --git a/Sources/Plasma/FeatureLib/pfCrashHandler/plWindowsStackWalker.cpp b/Sources/Plasma/FeatureLib/pfCrashHandler/plWindowsStackWalker.cpp
@@ -137,7 +137,7 @@ bool plStackWalker::iterator::next()
 class plStackWalkError : public std::runtime_error
 {
 public:
-    plStackWalkError() = default;
+    plStackWalkError() = delete;
     explicit plStackWalkError(const char* msg) : std::runtime_error(msg) { }
     explicit plStackWalkError(const ST::string& msg) : std::runtime_error(msg.c_str()) { }
 };