lynxis lazus has submitted this change. (
https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42160?usp=email )
Change subject: ccid_common: iso_handle_fsm_events: add missing return
......................................................................
ccid_common: iso_handle_fsm_events: add missing return
Fix warning about control reaches end of non-void function.
Change-Id: I4066c8a1ab1eadb3d8f8cbaf5aa0afbbb2e5dd80
---
M ccid_common/ccid_slot_fsm.c
1 file changed, 4 insertions(+), 1 deletion(-)
Approvals:
laforge: Looks good to me, but someone else must approve
Jenkins Builder: Verified
pespin: Looks good to me, but someone else must approve
lynxis lazus: Looks good to me, approved
diff --git a/ccid_common/ccid_slot_fsm.c b/ccid_common/ccid_slot_fsm.c
index 0c56275..c34ad72 100644
--- a/ccid_common/ccid_slot_fsm.c
+++ b/ccid_common/ccid_slot_fsm.c
@@ -199,7 +199,8 @@
}
/* do not free msgbs passed from the fsms, they are statically allocated! */
-static int iso_handle_fsm_events(struct ccid_slot *cs, bool enable){
+static int iso_handle_fsm_events(struct ccid_slot *cs, bool enable)
+{
struct iso_fsm_slot *ss = ccid_slot2iso_fsm_slot(cs);
struct msgb *tpdu, *resp;
volatile uint32_t event = cs->event;
@@ -338,6 +339,8 @@
__func__, event, data);
break;
}
+
+ return 0;
}
static int iso_fsm_slot_xfr_block_async(struct ccid_slot *cs, struct msgb *msg,
--
To view, visit https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42160?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: I4066c8a1ab1eadb3d8f8cbaf5aa0afbbb2e5dd80
Gerrit-Change-Number: 42160
Gerrit-PatchSet: 1
Gerrit-Owner: lynxis lazus <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: lynxis lazus <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
