[Analyzers][CPP] Turn on warning 26492

CppRuleSet.ruleset

@@ -79,7 +79,7 @@
     <Rule Id="C26485" Action="Hidden" />
     <Rule Id="C26490" Action="Hidden" />
     <Rule Id="C26491" Action="Hidden" />
-    <Rule Id="C26492" Action="Hidden" />
+    <Rule Id="C26492" Action="Error" />
     <Rule Id="C26493" Action="Hidden" />
     <Rule Id="C26494" Action="Hidden" />
     <Rule Id="C26495" Action="Error" />

src/common/utils/elevation.h

@@ -294,8 +294,9 @@ inline bool run_non_elevated(const std::wstring& file, const std::wstring& param
     siex.StartupInfo.cb = sizeof(siex);
 
     PROCESS_INFORMATION pi = { 0 };
+
     auto succeeded = CreateProcessW(file.c_str(),
-                                    const_cast<LPWSTR>(executable_args.c_str()),
+                                    &executable_args[0],
                                     nullptr,
                                     nullptr,
                                     FALSE,
@@ -395,8 +396,9 @@ inline bool run_same_elevation(const std::wstring& file, const std::wstring& par
 
     STARTUPINFO si = { sizeof(STARTUPINFO) };
     PROCESS_INFORMATION pi = { 0 };
+
     auto succeeded = CreateProcessW(file.c_str(),
-                                    const_cast<LPWSTR>(executable_args.c_str()),
+                                    &executable_args[0],
                                     nullptr,
                                     nullptr,
                                     FALSE,

src/common/utils/exec.h

@@ -4,9 +4,10 @@
 #include <Windows.h>
 
 // disable warning 26471 - Don't use reinterpret_cast. A cast from void* can use static_cast
+// disable warning 26492 - Don't use const_cast to cast away const
 // Disable 26497 for winrt - This function function-name could be marked constexpr if compile-time evaluation is desired.
 #pragma warning(push)
-#pragma warning(disable : 26471 26497)
+#pragma warning(disable : 26471 26492 26497)
 #include <wil/resource.h>
 #pragma warning(pop)
 

src/modules/videoconference/VideoConferenceProxyFilter/VideoCaptureDevice.cpp

@@ -247,6 +247,10 @@ struct VideoCaptureReceiverPin : winrt::implements<VideoCaptureReceiverPin, IPin
 
         if (_captureInputPin)
         {
+// disable warning 26492 - Don't use const_cast to cast away const
+// reset needs 'pmt' to be non-const, we can't easily change the query accept prototype
+// because of the inheritance.
+#pragma warning(suppress : 26492)
             _inputCaptureMediaType.reset(const_cast<AM_MEDIA_TYPE*>(pmt));
         }
 

tools/WebcamReportTool/DirectShowUtils.h

@@ -6,9 +6,10 @@
 #include <dshow.h>
 
 // disable warning 26471 - Don't use reinterpret_cast. A cast from void* can use static_cast
+// disable warning 26492 - Don't use const_cast to cast away const on winrt
 // Disable 26497 for winrt - This function function-name could be marked constexpr if compile-time evaluation is desired.
 #pragma warning(push)
-#pragma warning(disable : 26471 26497)
+#pragma warning(disable : 26471 26492 26497)
 #include <wil/com.h>
 #pragma warning(push)
 

tools/WebcamReportTool/main.cpp

@@ -6,9 +6,10 @@
 #include <cguid.h>
 
 // disable warning 26471 - Don't use reinterpret_cast. A cast from void* can use static_cast
+// disable warning 26492 - Don't use const_cast to cast away const
 // Disable 26497 for winrt - This function function-name could be marked constexpr if compile-time evaluation is desired.
 #pragma warning(push)
-#pragma warning(disable : 26471 26497)
+#pragma warning(disable : 26471 26492 26497)
 #include <wil/com.h>
 #pragma warning(push)