diff options
author | Rob Austein <sra@hactrn.net> | 2016-11-05 00:44:42 -0400 |
---|---|---|
committer | Rob Austein <sra@hactrn.net> | 2016-11-05 00:44:42 -0400 |
commit | 1d7ff81bd24dc14730f84f040f63a8da5a344d39 (patch) | |
tree | 5501b077db1eea26b045dead5d02bb18ab330288 /ks_flash.c | |
parent | 8ffe3e5451fa0a4494d4d15ee3ed58245b287b19 (diff) |
ks_list() was returning garbage for keys not visible to current session.
Diffstat (limited to 'ks_flash.c')
-rw-r--r-- | ks_flash.c | 9 |
1 files changed, 4 insertions, 5 deletions
@@ -1097,17 +1097,16 @@ static hal_error_t ks_list(hal_ks_t *ks, if (ks != &db.ks || result == NULL || result_len == NULL) return HAL_ERROR_BAD_ARGUMENTS; - if (db.ksi.used > result_max) - return HAL_ERROR_RESULT_TOO_LONG; - flash_block_t *block; hal_error_t err; - unsigned b; *result_len = 0; for (int i = 0; i < db.ksi.used; i++) { - b = db.ksi.index[i]; + unsigned b = db.ksi.index[i]; + + if (*result_len >= result_max) + return HAL_ERROR_RESULT_TOO_LONG; if ((err = block_read_cached(b, &block)) != HAL_OK) return err; |