Skip to content

Commit b33fc48

Browse files
committed
crypto/internal/hpke: use internal/byteorder instead of encoding/binary
crypto/internal/hpke is the only package under crypto that imports encoding/binary. Other packages use internal/byteorder instead, which notably doesn't depend on the reflect package. Updates #54097 Change-Id: I77a3ac5f4588527a2f82574df4cb84d30630d73f Reviewed-on: https://go-review.googlesource.com/c/go/+/622497 LUCI-TryBot-Result: Go LUCI <[email protected]> Reviewed-by: Carlos Amedee <[email protected]> Reviewed-by: Ian Lance Taylor <[email protected]> Reviewed-by: Filippo Valsorda <[email protected]>
1 parent d8f9c4c commit b33fc48

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

src/crypto/internal/hpke/hpke.go

+8-8
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,8 @@ import (
1010
"crypto/cipher"
1111
"crypto/ecdh"
1212
"crypto/rand"
13-
"encoding/binary"
1413
"errors"
14+
"internal/byteorder"
1515
"math/bits"
1616

1717
"golang.org/x/crypto/chacha20poly1305"
@@ -37,7 +37,7 @@ func (kdf *hkdfKDF) LabeledExtract(suiteID []byte, salt []byte, label string, in
3737

3838
func (kdf *hkdfKDF) LabeledExpand(suiteID []byte, randomKey []byte, label string, info []byte, length uint16) []byte {
3939
labeledInfo := make([]byte, 0, 2+7+len(suiteID)+len(label)+len(info))
40-
labeledInfo = binary.BigEndian.AppendUint16(labeledInfo, length)
40+
labeledInfo = byteorder.BeAppendUint16(labeledInfo, length)
4141
labeledInfo = append(labeledInfo, []byte("HPKE-v1")...)
4242
labeledInfo = append(labeledInfo, suiteID...)
4343
labeledInfo = append(labeledInfo, label...)
@@ -76,7 +76,7 @@ func newDHKem(kemID uint16) (*dhKEM, error) {
7676
return &dhKEM{
7777
dh: suite.curve,
7878
kdf: hkdfKDF{suite.hash},
79-
suiteID: binary.BigEndian.AppendUint16([]byte("KEM"), kemID),
79+
suiteID: byteorder.BeAppendUint16([]byte("KEM"), kemID),
8080
nSecret: suite.nSecret,
8181
}, nil
8282
}
@@ -224,9 +224,9 @@ func (s *Sender) Seal(aad, plaintext []byte) ([]byte, error) {
224224
func SuiteID(kemID, kdfID, aeadID uint16) []byte {
225225
suiteID := make([]byte, 0, 4+2+2+2)
226226
suiteID = append(suiteID, []byte("HPKE")...)
227-
suiteID = binary.BigEndian.AppendUint16(suiteID, kemID)
228-
suiteID = binary.BigEndian.AppendUint16(suiteID, kdfID)
229-
suiteID = binary.BigEndian.AppendUint16(suiteID, aeadID)
227+
suiteID = byteorder.BeAppendUint16(suiteID, kemID)
228+
suiteID = byteorder.BeAppendUint16(suiteID, kdfID)
229+
suiteID = byteorder.BeAppendUint16(suiteID, aeadID)
230230
return suiteID
231231
}
232232

@@ -253,7 +253,7 @@ func (u uint128) bitLen() int {
253253

254254
func (u uint128) bytes() []byte {
255255
b := make([]byte, 16)
256-
binary.BigEndian.PutUint64(b[0:], u.hi)
257-
binary.BigEndian.PutUint64(b[8:], u.lo)
256+
byteorder.BePutUint64(b[0:], u.hi)
257+
byteorder.BePutUint64(b[8:], u.lo)
258258
return b
259259
}

0 commit comments

Comments
 (0)