-
Notifications
You must be signed in to change notification settings - Fork 228
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
New verification model #913
Merged
zhangsoledad
merged 13 commits into
nervosnetwork:develop
from
xxuejie:new-verification-model
Jun 1, 2019
Merged
New verification model #913
zhangsoledad
merged 13 commits into
nervosnetwork:develop
from
xxuejie:new-verification-model
Jun 1, 2019
Commits on Jun 1, 2019
-
feat: new transaction verification model
Based on feedbacks gathered earlier, we are revising our verification model with the following changes: * When validating a transaction, CKB will grab all lock scripts from all inputs, and group them based on lock script hash. The script in each group will only be run once. The lock script itself will have to do the validation task for all inputs in the same group * CKB will also grab all type scripts from inputs and outputs(notice different from previous version, the type scripts in inputs are included here as well), and group them based on type script hash as well. Each type script in each group will also be run once. The type script itself needs to handle the validation task within the group * Syscalls are also revised to allow fetching all the inputs/outputs/witnesses within a single group. The default secp256k1_blake160_sighash_all lock script is compiled from the following source: https://github.com/nervosnetwork/ckb-system-scripts/blob/e00ba451dccb7a516454ea9606de3a21c8f9a80c/c/secp256k1_blake160_sighash_all.c The exact gcc version used in the compilation can be located in the following docker image: xxuejie/riscv-gnu-toolchain-rv64imac:20190529
Configuration menu - View commit details
-
Copy full SHA for 19b2859 - Browse repository at this point
Copy the full SHA 19b2859View commit details -
Configuration menu - View commit details
-
Copy full SHA for 5fad3cf - Browse repository at this point
Copy the full SHA 5fad3cfView commit details -
Configuration menu - View commit details
-
Copy full SHA for 5556ef5 - Browse repository at this point
Copy the full SHA 5556ef5View commit details -
Configuration menu - View commit details
-
Copy full SHA for fd11082 - Browse repository at this point
Copy the full SHA fd11082View commit details -
Configuration menu - View commit details
-
Copy full SHA for b0142bd - Browse repository at this point
Copy the full SHA b0142bdView commit details -
Configuration menu - View commit details
-
Copy full SHA for 7da9918 - Browse repository at this point
Copy the full SHA 7da9918View commit details -
Configuration menu - View commit details
-
Copy full SHA for ddbde16 - Browse repository at this point
Copy the full SHA ddbde16View commit details -
feat: use output data field to store block number in cellbase tx
The default secp256k1_blake160_sighash_all lock script is compiled from the following source: https://github.com/nervosnetwork/ckb-system-scripts/blob/680f4287ab3211025761aaf5cae3727fb4ea7241/c/secp256k1_blake160_sighash_all.c The exact gcc version used in the compilation can be located in the following docker image: xxuejie/riscv-gnu-toolchain-rv64imac:20190529
Configuration menu - View commit details
-
Copy full SHA for 70904df - Browse repository at this point
Copy the full SHA 70904dfView commit details -
Configuration menu - View commit details
-
Copy full SHA for 3883a7e - Browse repository at this point
Copy the full SHA 3883a7eView commit details -
Configuration menu - View commit details
-
Copy full SHA for d0141b5 - Browse repository at this point
Copy the full SHA d0141b5View commit details -
Revert "feat: use output data field to store block number in cellbase…
… tx" This reverts commit 43492e9.
Configuration menu - View commit details
-
Copy full SHA for 7ecfefa - Browse repository at this point
Copy the full SHA 7ecfefaView commit details -
Configuration menu - View commit details
-
Copy full SHA for 2ed8f73 - Browse repository at this point
Copy the full SHA 2ed8f73View commit details -
Configuration menu - View commit details
-
Copy full SHA for 8e41474 - Browse repository at this point
Copy the full SHA 8e41474View commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.