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 <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>

Reply via email to