diff --git a/src/debug/daccess/dacdbiimpl.cpp b/src/debug/daccess/dacdbiimpl.cpp index 26e3d6c169fc..ae266e8a6fc3 100644 --- a/src/debug/daccess/dacdbiimpl.cpp +++ b/src/debug/daccess/dacdbiimpl.cpp @@ -90,7 +90,7 @@ IDacDbiInterface::IAllocator * g_pAllocator = NULL; // // Need a class to serve as a tag that we can use to overload New/Delete. -#define forDbi (*(forDbiWorker *)NULL) +forDbiWorker forDbi; void * operator new(size_t lenBytes, const forDbiWorker &) { diff --git a/src/debug/di/rsmain.cpp b/src/debug/di/rsmain.cpp index b5685750dbc1..0f5778789fcb 100644 --- a/src/debug/di/rsmain.cpp +++ b/src/debug/di/rsmain.cpp @@ -40,6 +40,8 @@ RSDebuggingInfo g_RSDebuggingInfo_OutOfProc = {0 }; // set to NULL RSDebuggingInfo * g_pRSDebuggingInfo = &g_RSDebuggingInfo_OutOfProc; +// The following instances are used for invoking overloaded new/delete +forDbiWorker forDbi; #ifdef _DEBUG // For logs, we can print the string name for the debug codes. diff --git a/src/debug/di/rspriv.h b/src/debug/di/rspriv.h index bc0ea5989590..18920add5d50 100644 --- a/src/debug/di/rspriv.h +++ b/src/debug/di/rspriv.h @@ -177,7 +177,7 @@ struct MachineInfo USHORT m_usPort; }; -#define forDbi (*(forDbiWorker *)NULL) +extern forDbiWorker forDbi; // for dbi we just default to new, but we need to have these defined for both dac and dbi inline void * operator new(size_t lenBytes, const forDbiWorker &) diff --git a/src/debug/ee/debugger.cpp b/src/debug/ee/debugger.cpp index a06811c8172c..d67cb4134c8a 100644 --- a/src/debug/ee/debugger.cpp +++ b/src/debug/ee/debugger.cpp @@ -75,6 +75,9 @@ SVAL_IMPL_INIT(BOOL, Debugger, s_fCanChangeNgenFlags, TRUE); bool g_EnableSIS = false; +// The following instances are used for invoking overloaded new/delete +InteropSafe interopsafe; +InteropSafeExecutable interopsafeEXEC; #ifndef DACCESS_COMPILE diff --git a/src/debug/ee/debugger.h b/src/debug/ee/debugger.h index 636864794659..9cdf546290c5 100644 --- a/src/debug/ee/debugger.h +++ b/src/debug/ee/debugger.h @@ -3512,10 +3512,10 @@ class DebuggerEval * ------------------------------------------------------------------------ */ class InteropSafe {}; -#define interopsafe (*(InteropSafe*)NULL) +extern InteropSafe interopsafe; class InteropSafeExecutable {}; -#define interopsafeEXEC (*(InteropSafeExecutable*)NULL) +extern InteropSafeExecutable interopsafeEXEC; #ifndef DACCESS_COMPILE inline void * __cdecl operator new(size_t n, const InteropSafe&) diff --git a/src/debug/ildbsymlib/symwrite.h b/src/debug/ildbsymlib/symwrite.h index 055b8ec21fa4..54ab11a9a2c1 100644 --- a/src/debug/ildbsymlib/symwrite.h +++ b/src/debug/ildbsymlib/symwrite.h @@ -839,7 +839,8 @@ class ArrayStorage { // Help mitigate the impact of buffer overflow // Fail fast with a null-reference AV - return *(static_cast(0)) ; + volatile char* nullPointer = nullptr; + *nullPointer; } return m_array[ i ]; } diff --git a/src/debug/inc/dacdbiinterface.h b/src/debug/inc/dacdbiinterface.h index e61e240ad179..569ccbaca7af 100644 --- a/src/debug/inc/dacdbiinterface.h +++ b/src/debug/inc/dacdbiinterface.h @@ -32,7 +32,7 @@ template void DeleteDbiMemory(T *p); // Need a class to serve as a tag that we can use to overload New/Delete. class forDbiWorker {}; -#define forDbi (*(forDbiWorker *)NULL) +extern forDbiWorker forDbi; extern void * operator new(size_t lenBytes, const forDbiWorker &); extern void * operator new[](size_t lenBytes, const forDbiWorker &); extern void operator delete(void *p, const forDbiWorker &); diff --git a/src/pal/src/exception/seh.cpp b/src/pal/src/exception/seh.cpp index 473c490de53f..ad09e028841c 100644 --- a/src/pal/src/exception/seh.cpp +++ b/src/pal/src/exception/seh.cpp @@ -274,7 +274,7 @@ SEHProcessException(PAL_SEHException* exception) { // The exception happened in the page right below the stack limit, // so it is a stack overflow - write(STDERR_FILENO, StackOverflowMessage, sizeof(StackOverflowMessage) - 1); + (void)write(STDERR_FILENO, StackOverflowMessage, sizeof(StackOverflowMessage) - 1); PROCAbort(); } } diff --git a/tests/src/Common/Platform/platformdefines.cpp b/tests/src/Common/Platform/platformdefines.cpp index 4bef17078553..82061ac90d18 100644 --- a/tests/src/Common/Platform/platformdefines.cpp +++ b/tests/src/Common/Platform/platformdefines.cpp @@ -277,7 +277,7 @@ DWORD TP_GetFullPathName(LPWSTR fileName, DWORD nBufferLength, LPWSTR lpBuffer) return GetFullPathNameW(fileName, nBufferLength, lpBuffer, NULL); #else char nativeFullPath[MAX_PATH]; - realpath(HackyConvertToSTR(fileName), nativeFullPath); + (void)realpath(HackyConvertToSTR(fileName), nativeFullPath); LPWSTR fullPathForCLR = HackyConvertToWSTR(nativeFullPath); wcscpy_s(lpBuffer, MAX_PATH, fullPathForCLR); return wcslen(lpBuffer); diff --git a/tests/src/Common/Platform/platformdefines.h b/tests/src/Common/Platform/platformdefines.h index 49e8f888460a..c196b0c5d7a7 100644 --- a/tests/src/Common/Platform/platformdefines.h +++ b/tests/src/Common/Platform/platformdefines.h @@ -87,7 +87,7 @@ typedef void* HMODULE; typedef void* ULONG_PTR; typedef unsigned error_t; typedef void* LPVOID; -typedef char BYTE; +typedef unsigned char BYTE; typedef WCHAR OLECHAR; #endif diff --git a/tests/src/Interop/common/types.h b/tests/src/Interop/common/types.h index 7d7f7768a6e6..cb59c42e9d53 100755 --- a/tests/src/Interop/common/types.h +++ b/tests/src/Interop/common/types.h @@ -28,7 +28,7 @@ typedef void* HMODULE; typedef void* ULONG_PTR; typedef unsigned error_t; typedef void* LPVOID; -typedef char BYTE; +typedef unsigned char BYTE; typedef WCHAR OLECHAR; typedef unsigned int UINT_PTR; @@ -54,4 +54,4 @@ typedef int* DWORD_PTR; #define FALSE 0 #endif -#endif //_INTEROP_TYPES__H \ No newline at end of file +#endif //_INTEROP_TYPES__H