Hi
On Fri, Feb 2, 2018 at 12:23 AM, Stefan Berger
<stef...@linux.vnet.ibm.com> wrote:
> Wrap the calls to stl_be_p and stw_be_p in tpm_cmd_set_XYZ functions
> that are similar to existing getters.
why not,
Reviewed-by: Marc-André Lureau <marcandre.lur...@redhat.com>
>
> Signed-off-by: Stefan Berger <stef...@linux.vnet.ibm.com>
> ---
> hw/tpm/tpm_util.c | 6 +++---
> hw/tpm/tpm_util.h | 15 +++++++++++++++
> 2 files changed, 18 insertions(+), 3 deletions(-)
>
> diff --git a/hw/tpm/tpm_util.c b/hw/tpm/tpm_util.c
> index 8abde59..2de52a0 100644
> --- a/hw/tpm/tpm_util.c
> +++ b/hw/tpm/tpm_util.c
> @@ -106,9 +106,9 @@ const PropertyInfo qdev_prop_tpm = {
> void tpm_util_write_fatal_error_response(uint8_t *out, uint32_t out_len)
> {
> if (out_len >= sizeof(struct tpm_resp_hdr)) {
> - stw_be_p(out, TPM_TAG_RSP_COMMAND);
> - stl_be_p(out + 2, sizeof(struct tpm_resp_hdr));
> - stl_be_p(out + 6, TPM_FAIL);
> + tpm_cmd_set_tag(out, TPM_TAG_RSP_COMMAND);
> + tpm_cmd_set_size(out, sizeof(struct tpm_resp_hdr));
> + tpm_cmd_set_error(out, TPM_FAIL);
> }
> }
>
> diff --git a/hw/tpm/tpm_util.h b/hw/tpm/tpm_util.h
> index f003d15..f397ac2 100644
> --- a/hw/tpm/tpm_util.h
> +++ b/hw/tpm/tpm_util.h
> @@ -36,11 +36,21 @@ static inline uint16_t tpm_cmd_get_tag(const void *b)
> return lduw_be_p(b);
> }
>
> +static inline void tpm_cmd_set_tag(void *b, uint16_t tag)
> +{
> + stw_be_p(b, tag);
> +}
> +
> static inline uint32_t tpm_cmd_get_size(const void *b)
> {
> return ldl_be_p(b + 2);
> }
>
> +static inline void tpm_cmd_set_size(void *b, uint32_t size)
> +{
> + stl_be_p(b + 2, size);
> +}
> +
> static inline uint32_t tpm_cmd_get_ordinal(const void *b)
> {
> return ldl_be_p(b + 6);
> @@ -51,6 +61,11 @@ static inline uint32_t tpm_cmd_get_errcode(const void *b)
> return ldl_be_p(b + 6);
> }
>
> +static inline void tpm_cmd_set_error(void *b, uint32_t error)
> +{
> + stl_be_p(b + 6, error);
> +}
> +
> int tpm_util_get_buffer_size(int tpm_fd, TPMVersion tpm_version,
> size_t *buffersize);
>
> --
> 2.5.5
>
