added a few error checks

lostmsu · lostmsu · commit 6deab924645d · 2020-06-17T14:07:16.000-07:00
diff --git a/src/runtime/pyobject.cs b/src/runtime/pyobject.cs
@@ -165,6 +165,8 @@ public object AsManagedObject(Type t)
         /// </remarks>
         protected virtual void Dispose(bool disposing)
         {
+            DebugUtil.EnsureGIL();
+
             if (this.obj == IntPtr.Zero)
             {
                 return;
diff --git a/src/runtime/runtime.cs b/src/runtime/runtime.cs
@@ -1536,9 +1536,10 @@ internal static string GetManagedString(IntPtr op)
             if (type == PyUnicodeType)
             {
                 IntPtr p = PyUnicode_AsUnicode(op);
-                int length = (int)PyUnicode_GetSize(op);
+                Exceptions.ErrorCheck(p);
+                int length = checked((int)PyUnicode_GetSize(op));
 
-                int size = length * UCS;
+                int size = checked(length * UCS);
                 var buffer = new byte[size];
                 Marshal.Copy(p, buffer, 0, size);
                 return PyEncoding.GetString(buffer, 0, size);

Original file line number	Diff line number	Diff line change
`@@ -165,6 +165,8 @@ public object AsManagedObject(Type t)`
`165`	`165`	`/// </remarks>`
`166`	`166`	`protected virtual void Dispose(bool disposing)`
`167`	`167`	`{`
	`168`	`+ DebugUtil.EnsureGIL();`
	`169`	`+`
`168`	`170`	`if (this.obj == IntPtr.Zero)`
`169`	`171`	`{`
`170`	`172`	`return;`