The fdt_getprop() already returns strings, use const char * and get rid of the void types and multiple type casts. No functional change.
Signed-off-by: Marek Vasut <[email protected]> --- Cc: Aristo Chen <[email protected]> Cc: Daniel Golle <[email protected]> Cc: James Hilliard <[email protected]> Cc: Simon Glass <[email protected]> Cc: Tom Rini <[email protected]> Cc: [email protected] --- V2: - Propagate const char *key_name further into fit_pre_load_data_key_rsa() and fit_pre_load_data_key_ecdsa() - Update commit message --- tools/image-host.c | 19 +++++++++---------- 1 file changed, 9 insertions(+), 10 deletions(-) diff --git a/tools/image-host.c b/tools/image-host.c index 03c4a07cf28..4b6c16ed403 100644 --- a/tools/image-host.c +++ b/tools/image-host.c @@ -1645,7 +1645,7 @@ static int fit_config_add_verification_data(const char *keydir, * 2) get public key (X509_get_pubkey) * 3) provide der format (d2i_RSAPublicKey) */ -static int read_pub_key(const char *keydir, const void *name, +static int read_pub_key(const char *keydir, const char *name, unsigned char **pubkey, int *pubkey_len) { char path[1024]; @@ -1655,7 +1655,7 @@ static int read_pub_key(const char *keydir, const void *name, int ret; memset(path, 0, 1024); - snprintf(path, sizeof(path), "%s/%s.crt", keydir, (char *)name); + snprintf(path, sizeof(path), "%s/%s.crt", keydir, name); /* Open certificate file */ f = fopen(path, "r"); @@ -1700,7 +1700,7 @@ err_cert: } static int fit_pre_load_data_key_rsa(const char *keydir, void *keydest, - int pre_load_noffset, const void *key_name) + int pre_load_noffset, const char *key_name) { unsigned char *pubkey = NULL; int ret, pubkey_len; @@ -1721,7 +1721,7 @@ static int fit_pre_load_data_key_rsa(const char *keydir, void *keydest, } static int fit_pre_load_data_key_ecdsa(const char *keydir, void *keydest, - int pre_load_noffset, const void *key_name, + int pre_load_noffset, const char *key_name, const void *algo_name) { struct image_sign_info info; @@ -1763,8 +1763,8 @@ static int fit_pre_load_data_key_ecdsa(const char *keydir, void *keydest, int fit_pre_load_data(const char *keydir, void *keydest, void *fit) { int pre_load_noffset; - const void *algo_name; - const void *key_name; + const char *algo_name; + const char *key_name; char *name; int ret; @@ -1794,10 +1794,9 @@ int fit_pre_load_data(const char *keydir, void *keydest, void *fit) } /* Is it a RSA or an ECDSA key */ - name = strchr((const char *)algo_name, ','); + name = strchr(algo_name, ','); if (!name) { - fprintf(stderr, "The name of the algo is invalid: %s\n", - (char *)algo_name); + fprintf(stderr, "The name of the algo is invalid: %s\n", algo_name); ret = -EINVAL; goto out; } @@ -1809,7 +1808,7 @@ int fit_pre_load_data(const char *keydir, void *keydest, void *fit) ret = fit_pre_load_data_key_ecdsa(keydir, keydest, pre_load_noffset, key_name, algo_name); } else { - fprintf(stderr, "The algo %s is not supported\n", (char *)algo_name); + fprintf(stderr, "The algo %s is not supported\n", algo_name); ret = -EINVAL; } -- 2.53.0
