From 29ae808472ab903a494f840c924337588cb04bec Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Miloslav=20Trma=C4=8D?= Date: Fri, 30 Oct 2015 20:46:51 +0100 Subject: [PATCH] Don't compute an unnecessary cryptographic hash MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Miloslav Trmač --- tuf/client/client.go | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/tuf/client/client.go b/tuf/client/client.go index 4d6ad0e6c3..7da9785f84 100644 --- a/tuf/client/client.go +++ b/tuf/client/client.go @@ -405,9 +405,11 @@ func (c *Client) downloadSigned(role string, size int64, expectedSha256 []byte) if err != nil { return nil, nil, err } - genHash := sha256.Sum256(raw) - if expectedSha256 != nil && !bytes.Equal(genHash[:], expectedSha256) { - return nil, nil, ErrChecksumMismatch{role: role} + if expectedSha256 != nil { + genHash := sha256.Sum256(raw) + if !bytes.Equal(genHash[:], expectedSha256) { + return nil, nil, ErrChecksumMismatch{role: role} + } } s := &data.Signed{} err = json.Unmarshal(raw, s)