aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRuss Cox <rsc@golang.org>2016-01-23 23:43:08 -0500
committerRuss Cox <rsc@golang.org>2016-01-26 16:27:16 +0000
commitb4c9d01d81d90a78a4330d3e3567bc5533e8c659 (patch)
tree4c7ebd508f850e7926deb55678937fce1c333784
parent038b8139433054c70f17494f4cd96dd4f3ddd696 (diff)
downloadgo-b4c9d01d81d90a78a4330d3e3567bc5533e8c659.tar.gz
go-b4c9d01d81d90a78a4330d3e3567bc5533e8c659.zip
crypto: document that Signer.Sign does not hash
Fixes #13938. Change-Id: I0b4842b8bc22dc79323d6894c123cde638f52d3f Reviewed-on: https://go-review.googlesource.com/18856 Reviewed-by: Adam Langley <agl@golang.org>
-rw-r--r--src/crypto/crypto.go8
1 files changed, 6 insertions, 2 deletions
diff --git a/src/crypto/crypto.go b/src/crypto/crypto.go
index 184ea9d4d6..a80ebd3693 100644
--- a/src/crypto/crypto.go
+++ b/src/crypto/crypto.go
@@ -109,7 +109,7 @@ type Signer interface {
// private key.
Public() PublicKey
- // Sign signs msg with the private key, possibly using entropy from
+ // Sign signs digest with the private key, possibly using entropy from
// rand. For an RSA key, the resulting signature should be either a
// PKCS#1 v1.5 or PSS signature (as indicated by opts). For an (EC)DSA
// key, it should be a DER-serialised, ASN.1 signature structure.
@@ -118,7 +118,11 @@ type Signer interface {
// simply pass in the hash function used as opts. Sign may also attempt
// to type assert opts to other types in order to obtain algorithm
// specific values. See the documentation in each package for details.
- Sign(rand io.Reader, msg []byte, opts SignerOpts) (signature []byte, err error)
+ //
+ // Note that when a signature of a hash of a larger message is needed,
+ // the caller is responsible for hashing the larger message and passing
+ // the hash (as digest) and the hash function (as opts) to Sign.
+ Sign(rand io.Reader, digest []byte, opts SignerOpts) (signature []byte, err error)
}
// SignerOpts contains options for signing with a Signer.