From: Jacek Blaszczynski <biosciencenow@outlook.com>
Date: Mon, 30 Jul 2018 00:56:32 +0000 (+0200)
Subject: Fix invalid IntPtr == null comparisons, set strict mode for Roslyn (#19191)
X-Git-Tag: accepted/tizen/unified/20190422.045933~1566
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=93cf012db8396f5ea16f70b781a5de0d012fc65d;p=platform%2Fupstream%2Fcoreclr.git

Fix invalid IntPtr == null comparisons, set strict mode for Roslyn (#19191)

Issue reported in https://github.com/dotnet/corefx/issues/31456
Solution is to compare always against IntPtr.Zero and use Roslyn
stric mode for reporting warnings for IntPtr == null comparisons
---

diff --git a/Directory.Build.props b/Directory.Build.props
index 3efd9a3..2082362 100644
--- a/Directory.Build.props
+++ b/Directory.Build.props
@@ -2,4 +2,8 @@
   <PropertyGroup>
     <CL_MPCount>$(NumberOfCores)</CL_MPCount>
   </PropertyGroup>
+  <PropertyGroup>
+    <!-- Enables Strict mode for Roslyn compiler -->
+    <Features>strict</Features>
+  </PropertyGroup>
 </Project>
diff --git a/src/System.Private.CoreLib/src/System/StubHelpers.cs b/src/System.Private.CoreLib/src/System/StubHelpers.cs
index 6d9e7a8..b8e34e0 100644
--- a/src/System.Private.CoreLib/src/System/StubHelpers.cs
+++ b/src/System.Private.CoreLib/src/System/StubHelpers.cs
@@ -200,7 +200,7 @@ namespace System.StubHelpers
 
         internal static unsafe void ConvertToManaged(StringBuilder sb, IntPtr pNative)
         {
-            if (pNative == null)
+            if (pNative == IntPtr.Zero)
                 return;
 
             int nbBytes = StubHelpers.strlen((sbyte*)pNative);