Don't compute an unnecessary cryptographic hash

Signed-off-by: Miloslav Trmač <mitr@redhat.com>
This commit is contained in:
Miloslav Trmač 2015-10-30 20:46:51 +01:00
parent dfaa8daf9b
commit 29ae808472
1 changed files with 5 additions and 3 deletions

View File

@ -405,9 +405,11 @@ func (c *Client) downloadSigned(role string, size int64, expectedSha256 []byte)
if err != nil { if err != nil {
return nil, nil, err return nil, nil, err
} }
genHash := sha256.Sum256(raw) if expectedSha256 != nil {
if expectedSha256 != nil && !bytes.Equal(genHash[:], expectedSha256) { genHash := sha256.Sum256(raw)
return nil, nil, ErrChecksumMismatch{role: role} if !bytes.Equal(genHash[:], expectedSha256) {
return nil, nil, ErrChecksumMismatch{role: role}
}
} }
s := &data.Signed{} s := &data.Signed{}
err = json.Unmarshal(raw, s) err = json.Unmarshal(raw, s)