-
Notifications
You must be signed in to change notification settings - Fork 18k
x/crypto/salsa20/salsa: confusing docs #21279
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
Comments
I think the key word is otherwise
Same slice is ok, and it that case they'll overlap (obviously), but otherwise they shouldn't overlap. |
Huh? I don't think that helps. |
How about
|
is how it's usually worded in BoringSSL |
The intent is to describe that there are only two valid situations:
Other configurations (i.e. |
Weak -1 on the BoringSSL wording, because I can see in and out having different len being useful and correct.
Not that my wording makes it clear that's OK, but at least it does not rule out.
|
Maybe I think this is accurate but a little obscure. |
We should also put that text in the The current text probably means this, but leaves too much open to interpretation, and implementations are already more strict anyway:
For the sake of breaking the bikeshed, I'm going to CL "must overlap entirely or not at all" tomorrow unless anyone objects. |
Change https://golang.org/cl/61133 mentions this issue: |
Change https://golang.org/cl/61132 mentions this issue: |
Updates golang/go#21279 Change-Id: I686835c644f52e3d5ea2b7e6431ef096d188c19d Reviewed-on: https://go-review.googlesource.com/61133 Reviewed-by: Ian Lance Taylor <iant@golang.org>
The docs for
XORKeyStream
say:What does this even mean: ?
If they're the same slice (same 3-tuple ptr/len/cap) then they by definition overlap.
Fix this and any copy/pastes of this wording we find.
The text was updated successfully, but these errors were encountered: