From 275a9a3f9b6a2158bfb7826074b72d5bdfb2ac35 Mon Sep 17 00:00:00 2001 From: Herbert Xu Date: Tue, 21 Oct 2025 12:07:56 +0800 Subject: [PATCH] KEYS: trusted: Pass argument by pointer in dump_options Instead of passing pkey_info into dump_options by value, using a pointer instead. Reported-by: Stephen Rothwell Signed-off-by: Herbert Xu --- security/keys/trusted-keys/trusted_caam.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/security/keys/trusted-keys/trusted_caam.c b/security/keys/trusted-keys/trusted_caam.c index 090099d1b04d1..601943ce0d60f 100644 --- a/security/keys/trusted-keys/trusted_caam.c +++ b/security/keys/trusted-keys/trusted_caam.c @@ -29,12 +29,12 @@ static const match_table_t key_tokens = { }; #ifdef CAAM_DEBUG -static inline void dump_options(struct caam_pkey_info pkey_info) +static inline void dump_options(const struct caam_pkey_info *pkey_info) { - pr_info("key encryption algo %d\n", pkey_info.key_enc_algo); + pr_info("key encryption algo %d\n", pkey_info->key_enc_algo); } #else -static inline void dump_options(struct caam_pkey_info pkey_info) +static inline void dump_options(const struct caam_pkey_info *pkey_info) { } #endif @@ -108,7 +108,7 @@ static int trusted_caam_seal(struct trusted_key_payload *p, char *datablob) ret = get_pkey_options(datablob, &info.pkey_info); if (ret < 0) return 0; - dump_options(info.pkey_info); + dump_options(&info.pkey_info); } ret = caam_encap_blob(blobifier, &info); @@ -140,7 +140,7 @@ static int trusted_caam_unseal(struct trusted_key_payload *p, char *datablob) ret = get_pkey_options(datablob, &info.pkey_info); if (ret < 0) return 0; - dump_options(info.pkey_info); + dump_options(&info.pkey_info); p->key_len = p->blob_len + sizeof(struct caam_pkey_info); memcpy(p->key, &info.pkey_info, sizeof(struct caam_pkey_info)); -- 2.47.3