diff options
| author | Rob Austein <sra@hactrn.net> | 2016-10-24 15:15:51 -0400 |
|---|---|---|
| committer | Rob Austein <sra@hactrn.net> | 2016-10-24 15:15:51 -0400 |
| commit | dcf3c671314b36285277073c0a3d3a09bf4d93e6 (patch) | |
| tree | e41e68a44ce08dfd3bf976d66634daf9c8097e9b /hal_internal.h | |
| parent | 87f0998afe6dc6946d01429fc74f8696f0da2f0a (diff) | |
Make previous_uuid an input-only argument to hal_rpc_pkey_match().
In retrospect it's obvious that this never needed to be an
input/output argument, as its value will always be the same as the
last value in the returned array. Doh. So simplify the RPC and call
sequence slightly by removing the unnecessary output value.
Diffstat (limited to 'hal_internal.h')
| -rw-r--r-- | hal_internal.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/hal_internal.h b/hal_internal.h index 3e6cf29..44deaf6 100644 --- a/hal_internal.h +++ b/hal_internal.h @@ -252,7 +252,7 @@ typedef struct { hal_uuid_t *result, unsigned *result_len, const unsigned result_max, - hal_uuid_t *previous_uuid); + const hal_uuid_t * const previous_uuid); hal_error_t (*set_attribute)(const hal_pkey_handle_t pkey, const uint32_t type, @@ -499,7 +499,7 @@ struct hal_ks_driver { hal_uuid_t *result, unsigned *result_len, const unsigned result_max, - hal_uuid_t *previous_uuid); + const hal_uuid_t * const previous_uuid); hal_error_t (*set_attribute)(hal_ks_t *ks, hal_pkey_slot_t *slot, @@ -624,7 +624,7 @@ static inline hal_error_t hal_ks_match(hal_ks_t *ks, hal_uuid_t *result, unsigned *result_len, const unsigned result_max, - hal_uuid_t *previous_uuid) + const hal_uuid_t * const previous_uuid) { if (ks == NULL || ks->driver == NULL || ks->driver->match == NULL) return HAL_ERROR_BAD_ARGUMENTS; |