Skip to content

DPoW#133

Merged
jl777 merged 123 commits intomasterfrom
dPoW
Jan 25, 2017
Merged

DPoW#133
jl777 merged 123 commits intomasterfrom
dPoW

Conversation

@jl777
Copy link
Copy Markdown
Owner

@jl777 jl777 commented Jan 25, 2017

Release candidate
1.0.5

str4d and others added 30 commits December 9, 2016 21:06
…checkpoints, r=str4d

Skip JoinSplit verification before the last checkpoint

Part of #1749
Additional well-formedness check for G2 elements

libsnark currently checks that G<sub>1</sub> and G<sub>2</sub> elements are well-formed by ensuring that they satisfy their respective curve equations, and although this is enough for G<sub>1</sub> (which is instantiated as an order r curve E/F<sub>p</sub>: y^2 = x^3 + b), G<sub>2</sub> is the order r *subgroup* of the composite order r(2q-r) curve E'/Fp<sup>2</sup>: y^2 = x^3 + b/e constructed via a sextic twisting isomorphism. This means we need to ensure these points are order r as well.

None of the proofs on the Zcash blockchain violate this check, and it may not even be possible for them to violate this check (bilinearity is not preserved). Let's be cautious and do it anyway.
Only increment new notes on reindex

Addresses another issue in zcash#1904.

When an existing one of our notes was found again, its cache was reset and it was re-witnessed. This would cause encountered notes to get out-of-sync with the otherwise-ignored newer notes, which could be a problem if the wallet data happens to be written out during a reindex.
…ebfull

Add total number of commitments to getblockchaininfo
after a successful sign and send, meaning that the logged hash fragment
would be different from the txid logged by "AddToWallet".  This issue
occured when sending from transparent addresses, as utxo inputs must be
signed.  It did not occur when sending from shielded addresses.
@jl777 jl777 merged commit cd8ff23 into master Jan 25, 2017
miketout referenced this pull request in miketout/VerusCoin Oct 24, 2018
jl777 pushed a commit that referenced this pull request Feb 19, 2019
DeckerSU pushed a commit to DeckerSU/komodo that referenced this pull request Jan 30, 2023
add interest calculations via CCoinsViewCache::GetValueIn test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants