laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/39444?usp=email ) ( 1 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: ccid dev: fix sizeof abuse ...................................................................... ccid dev: fix sizeof abuse Change-Id: Ia96247af6f2a887d3c894a65983e795b9e5a7cbe --- M ccid_common/ccid_device.c 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified diff --git a/ccid_common/ccid_device.c b/ccid_common/ccid_device.c index 405fb2e..398a2b4 100644 --- a/ccid_common/ccid_device.c +++ b/ccid_common/ccid_device.c @@ -137,7 +137,7 @@ static struct ccid_slot *get_ccid_slot(struct ccid_instance *ci, uint8_t slot_nr) { - if (slot_nr >= sizeof(ci->slot)) + if (slot_nr >= ARRAY_SIZE(ci->slot)) return NULL; else return &ci->slot[slot_nr]; -- To view, visit https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/39444?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-ccid-firmware Gerrit-Branch: master Gerrit-Change-Id: Ia96247af6f2a887d3c894a65983e795b9e5a7cbe Gerrit-Change-Number: 39444 Gerrit-PatchSet: 6 Gerrit-Owner: Hoernchen <ewild(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org>