NeonXP/securecookie
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Merge pull request #16 from elithrar/patch-1

Browse source 
Removed redundant calls to len()
This commit is contained in:
Kamil Kisiel 2015-03-17 16:33:49 -07:00
commit 1dea542d12
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
securecookie.go
View file

@ -271,7 +271,7 @@ func createMac(h hash.Hash, value []byte) []byte {
// verifyMac verifies that a message authentication code (MAC) is valid.
func verifyMac(h hash.Hash, value []byte, mac []byte) error {
mac2 := createMac(h, value)
if len(mac) == len(mac2) && subtle.ConstantTimeCompare(mac, mac2) == 1 {
if subtle.ConstantTimeCompare(mac, mac2) == 1 {
return nil
}
return ErrMacInvalid