diff options
| author | Vadim Bendebury <vbendeb@chromium.org> | 2020-04-22 17:38:58 -0700 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2020-05-14 07:25:57 +0000 |
| commit | 69a887265d737c628a3c55a03b923917cf1b471e (patch) | |
| tree | f962af23da7cfdfb8695eca5731c59708bacec9d /chip/g/upgrade_fw.c | |
| parent | 8d785e9597c9db3444c01d15dff732bef85fad5a (diff) | |
| download | chrome-ec-69a887265d737c628a3c55a03b923917cf1b471e.tar.gz | |
crypto_api: use const void
This is a minor API clean up, it is not entirely clear why const void
pointers were not used originally, but using this type for input data
(and void pointer for output) makes interfacing with the library much
easier.
Also modified cases where the first parameter of DCRYPTO_SHA1_hash()
was typecasted unnecessarily.
BUG=none
TEST=make buildall succeeds, Cr50 image supports booting a Chrome OS
device just fine.
Signed-off-by: Vadim Bendebury <vbendeb@chromium.org>
Change-Id: Ic8a670aa7b26598ea323182845c184b7f1d715a1
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2163568
Reviewed-by: Andrey Pronin <apronin@chromium.org>
Diffstat (limited to 'chip/g/upgrade_fw.c')
| -rw-r--r-- | chip/g/upgrade_fw.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/chip/g/upgrade_fw.c b/chip/g/upgrade_fw.c index 276eac8f92..727af4b7d9 100644 --- a/chip/g/upgrade_fw.c +++ b/chip/g/upgrade_fw.c @@ -148,7 +148,7 @@ int usb_pdu_valid(struct upgrade_command *cmd_body, size_t cmd_size) cmd.block_base); /* Check if the block was received properly. */ - DCRYPTO_SHA1_hash((uint8_t *)&cmd_body->block_base, + DCRYPTO_SHA1_hash(&cmd_body->block_base, body_size + sizeof(cmd_body->block_base), sha1_digest); if (memcmp(sha1_digest, &cmd_body->block_digest, |