-
Notifications
You must be signed in to change notification settings - Fork 5.8k
Update EIP-2537: Rephrased subgroup check part #8322
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
Closed
Closed
Changes from all commits
Commits
Show all changes
44 commits
Select commit
Hold shift + click to select a range
f3348e3
Update eip-2537.md
asanso 7ce436e
Update eip-2537.md
asanso af5f6e4
Update eip-2537.md
asanso 537f0ef
Update eip-2537.md
asanso 31454df
Create eip-2537fast_subgroup_checks.md
asanso 7b704d0
Create fast_subgroup_checks.md
asanso c11b3b3
Update eip-2537.md
asanso dd5987d
Update fast_subgroup_checks.md
asanso 81e93d8
Update fast_subgroup_checks.md
asanso b9c8588
Update eip-2537.md
asanso 422f2d9
Update fast_subgroup_checks.md
asanso f18628d
Update fast_subgroup_checks.md
asanso 0b7671a
Update fast_subgroup_checks.md
asanso 4984aae
Update fast_subgroup_checks.md
asanso b0007e7
Update fast_subgroup_checks.md
asanso a4ee4db
Update fast_subgroup_checks.md
asanso 53f91fe
Update fast_subgroup_checks.md
asanso a3d6148
Update fast_subgroup_checks.md
asanso b4ee65c
Update fast_subgroup_checks.md
asanso 9957dd2
Update fast_subgroup_checks.md
asanso c4a20a2
Update fast_subgroup_checks.md
asanso 2ce1afc
Update field_to_curve.md
asanso b4d9621
Update fast_subgroup_checks.md
asanso ce2d83c
Update fast_subgroup_checks.md
asanso 0752dfe
Update fast_subgroup_checks.md
asanso e352ed0
Update fast_subgroup_checks.md
asanso 8961bcd
Update fast_subgroup_checks.md
asanso 6907436
Update fast_subgroup_checks.md
asanso 5a32513
Update fast_subgroup_checks.md
asanso 2d6f7ef
Update fast_subgroup_checks.md
asanso e6e3134
Update fast_subgroup_checks.md
asanso c872f8b
Update fast_subgroup_checks.md
asanso 300ded6
Update fast_subgroup_checks.md
asanso 197e88c
Update fast_subgroup_checks.md
asanso ebed2da
Update fast_subgroup_checks.md
asanso 6d89b6c
Update fast_subgroup_checks.md
asanso 82ff6a6
Update fast_subgroup_checks.md
asanso 5ec74e9
Update fast_subgroup_checks.md
asanso 104c2d4
Update fast_subgroup_checks.md
asanso 453fe8c
Update fast_subgroup_checks.md
asanso 2b0a144
Update fast_subgroup_checks.md
asanso fc09b50
Update fast_subgroup_checks.md
asanso 229e523
Update eip-2537.md
asanso 1f129eb
Update fast_subgroup_checks.md
asanso File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,51 @@ | ||
# Fast subgroup checks used by EIP 2537 | ||
|
||
### Fields and Groups | ||
|
||
Field Fp is defined as the finite field of size `p` with elements represented as integers between 0 and p-1 (both inclusive). | ||
|
||
Field Fp2 is defined as `Fp[X]/(X^2-nr2)` with elements `el = c0 + c1 * v`, where `v` is the formal square root of `nr2` represented as integer pairs `(c0,c1)`. | ||
|
||
Group G1 is defined as a set of Fp pairs (points) `(x,y)` such that either `(x,y)` is `(0,0)` or `x,y` satisfy the curve Fp equation. | ||
|
||
Group G2 is defined as a set of Fp2 pairs (points) `(x',y')` such that either `(x,y)` is `(0,0)` or `(x',y')` satisfy the curve Fp2 equation. | ||
|
||
## Curve parameters | ||
|
||
The set of parameters used by fast subgroup checks: | ||
|
||
``` | ||
|x| (seed) = 15132376222941642752 | ||
x is negative = true | ||
Cube root of unity modulo p - Beta = 793479390729215512621379701633421447060886740281060493010456487427281649075476305620758731620350 | ||
r = 4002409555221667392624310435006688643935503118305586438271171395842971157480381377015405980053539358417135540939437 * v | ||
s = 2973677408986561043442465346520108879172042883009249989176415018091420807192182638567116318576472649347015917690530 + 1028732146235106349975324479215795277384839936929757896155643118032610843298655225875571310552543014690878354869257 * v | ||
``` | ||
|
||
## Helper function to compute the conjugate over Fp2 - `conjugate` | ||
|
||
`conjugate(c0 + c1 * v) := c0 - c1 * v` | ||
|
||
## G1 endomorphism - `phi` | ||
|
||
The endomorphism `phi` transform the point from `(x,y)` to `(Beta*x,y)` where `Beta` is a precomputed cube root of unity modulo `p` given above in parameters sections: | ||
|
||
`phi((x,y)) := (Beta*x,y)` | ||
|
||
## G2 endomorphism - `psi` | ||
|
||
`psi((x,y)) := (conjugate(x)*r,conjugate(y)*s)` | ||
|
||
# The G1 case | ||
|
||
Before accepting a point `P` as input that purports to be a member of G1 subject the input to the following endomorphism test: `phi(P) + x^2*P = 0` | ||
|
||
|
||
# The G2 case | ||
|
||
Before accepting a point `P` as input that purports to be a member of G2 subject the input to the following endomorphism test: `psi(P) + x*P = 0` | ||
|
||
# Resources | ||
|
||
* https://eprint.iacr.org/2021/1130.pdf, sec.4 | ||
* https://eprint.iacr.org/2022/352.pdf, sec. 4.2 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
|
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
MUST perform a subgroup check, and reject invalid point.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mratsim what do you mean? is there anything wrong with this part of the text ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
AFAIK, in RFC MUST is the way to specify that something is mandatory, and the spec should explicitly mention that it returns as an error on failure.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mratsim Is the problem the capitalization of 'must' rather than using lowercase? Or am I missing something else? :)
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To be clear, I have 2 issues:
Rationale:
I can make a PR to this PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mratsim this would be great.
One you are on it I everyone agrees you can already to the PR to this PR with this strategy:
assuming @shamatar @ralexstokes @ineffectualproperty agree
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@shamatar @ralexstokes @ineffectualproperty #8456 got merged with :
Misuse resistance approach. See @asanso, enforce subgroup check except on addition.
Unless you want me to revert it , this is what the status quo is