Skip to content

Beta#132

Merged
jl777 merged 122 commits intodPoWfrom
beta
Jan 25, 2017
Merged

Beta#132
jl777 merged 122 commits intodPoWfrom
beta

Conversation

@jl777
Copy link
Copy Markdown
Owner

@jl777 jl777 commented Jan 25, 2017

1.0.5 and interest rate edge case

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 bd668b9 into dPoW Jan 25, 2017
miketout referenced this pull request in miketout/VerusCoin Oct 24, 2018
Sync Overwinter and Sapling defaults
jl777 pushed a commit that referenced this pull request Feb 19, 2019
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