fix: reverse logic valdiating spentproof shares

caught by reviewer of pr 121. The previous code was only checking that one share matches, not all shares
maidsafe · Feb 17, 2022 · b9d2e5c · b9d2e5c
1 parent 9c64a1d
commit b9d2e5c
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/builder.rs b/src/builder.rs
@@ -264,18 +264,18 @@ impl ReissueRequestBuilder {
                     .next()
                     .ok_or(Error::ReissueRequestMissingSpentProofShare(*input_index))?;
 
-                if !shares
+                if shares
                     .iter()
                     .map(SpentProofShare::spentbook_pks)
-                    .any(|pks| pks == any_share.spentbook_pks())
+                    .any(|pks| pks != any_share.spentbook_pks())
                 {
                     return Err(Error::ReissueRequestPublicKeySetMismatch);
                 }
 
-                if !shares
+                if shares
                     .iter()
                     .map(|s| &s.public_commitments)
-                    .any(|pc| *pc == any_share.public_commitments)
+                    .any(|pc| *pc != any_share.public_commitments)
                 {
                     return Err(Error::ReissueRequestPublicCommitmentMismatch);
                 }