From 58a9ca87db1a8d9e75f830a508ab1040c1bd889f Mon Sep 17 00:00:00 2001 From: Rob Austein Date: Mon, 14 Nov 2016 00:23:26 -0500 Subject: Remove now-gratuitous check which kept attribute deletion from working. This check made sense when attribute deletion was a separate operation, but now that it has been folded into set_attributes(), this check was worse than useless. --- hal_internal.h | 4 ---- 1 file changed, 4 deletions(-) diff --git a/hal_internal.h b/hal_internal.h index dc00568..756de90 100644 --- a/hal_internal.h +++ b/hal_internal.h @@ -666,10 +666,6 @@ static inline hal_error_t hal_ks_set_attributes(hal_ks_t *ks, if (ks->driver->set_attributes == NULL) return HAL_ERROR_NOT_IMPLEMENTED; - for (int i = 0; i < attributes_len; i++) - if (attributes[i].length == 0 || attributes[i].value == NULL) - return HAL_ERROR_BAD_ARGUMENTS; - return ks->driver->set_attributes(ks, slot, attributes, attributes_len); } -- cgit v1.2.3