Fixed ballots to free the memory even tho it thinks it is already closed. (#328)

Author: SteveMaier-IRT

bindings/netstandard/ElectionGuard/ElectionGuard.Encryption.Tests/TestEncrypt.cs

@@ -7,9 +7,9 @@
     <!-- Project -->
     <RootNamespace>ElectionGuard</RootNamespace>
     <AssemblyName>ElectionGuard.Encryption</AssemblyName>
-    <Version>0.1.16</Version>
-    <AssemblyVersion>0.1.16.0</AssemblyVersion>
-    <AssemblyFileVersion>0.1.16.0</AssemblyFileVersion>
+    <Version>0.1.17</Version>
+    <AssemblyVersion>0.1.17.0</AssemblyVersion>
+    <AssemblyFileVersion>0.1.17.0</AssemblyFileVersion>
   </PropertyGroup>
 
   <PropertyGroup>
@@ -19,7 +19,7 @@
     <Title>ElectionGuard Encryption</Title>
     <Description>Open source implementation of ElectionGuard's ballot encryption.</Description>
     <Authors>Microsoft</Authors>
-    <PackageVersion>0.1.16</PackageVersion>
+    <PackageVersion>0.1.17</PackageVersion>
     <PackageLicenseExpression>MIT</PackageLicenseExpression>
     <PackageProjectUrl>https://github.com/microsoft/electionguard-cpp</PackageProjectUrl>
     <RepositoryUrl>https://github.com/microsoft/electionguard-cpp</RepositoryUrl>

bindings/netstandard/ElectionGuard/ElectionGuard.Encryption/ElectionGuard.Encryption.csproj

@@ -50,6 +50,8 @@ internal unsafe ElectionguardSafeHandle(
 
         public IntPtr Ptr => handle;
 
+        protected bool IsFreed = false;
+
         protected abstract bool Free();
 
         protected override bool ReleaseHandle()
@@ -59,6 +61,7 @@ protected override bool ReleaseHandle()
                 var freed = Free();
                 if (freed)
                 {
+                    IsFreed = true;
                     Close();
                 }
                 return freed;

bindings/netstandard/ElectionGuard/ElectionGuard.Encryption/ElectionguardSafeHandle.cs

@@ -2793,7 +2793,7 @@ internal class PlaintextBallotHandle
                 [ReliabilityContract(Consistency.WillNotCorruptState, Cer.MayFail)]
                 protected override bool Free()
                 {
-                    if (IsClosed) return true;
+                    if (IsFreed) return true;
 
                     var status = PlaintextBallot.Free(TypedPtr);
                     if (status != Status.ELECTIONGUARD_STATUS_SUCCESS)
@@ -2924,7 +2924,7 @@ internal class CiphertextBallotHandle
                 [ReliabilityContract(Consistency.WillNotCorruptState, Cer.MayFail)]
                 protected override bool Free()
                 {
-                    if (IsClosed) return true;
+                    if (IsFreed) return true;
 
                     var status = CiphertextBallot.Free(TypedPtr);
                     if (status != Status.ELECTIONGUARD_STATUS_SUCCESS)
@@ -3118,7 +3118,7 @@ internal class SubmittedBallotHandle
                 [ReliabilityContract(Consistency.WillNotCorruptState, Cer.MayFail)]
                 protected override bool Free()
                 {
-                    if (IsClosed) return true;
+                    if (IsFreed) return true;
 
                     var status = SubmittedBallot.Free(TypedPtr);
                     if (status != Status.ELECTIONGUARD_STATUS_SUCCESS)