From 05e51adf10ec4b0e1330cbf557abd118824faa97 Mon Sep 17 00:00:00 2001 From: Waldemar Rymarkiewicz Date: Tue, 31 May 2011 12:46:50 +0200 Subject: [PATCH] Simplify return value in disconnect_req --- sap/server.c | 21 +++++++-------------- 1 file changed, 7 insertions(+), 14 deletions(-) diff --git a/sap/server.c b/sap/server.c index a6712354d..d50a24118 100644 --- a/sap/server.c +++ b/sap/server.c @@ -349,11 +349,9 @@ static int disconnect_req(struct sap_connection *conn, uint8_t disc_type) switch (disc_type) { case SAP_DISCONNECTION_TYPE_GRACEFUL: - if (conn->state == SAP_STATE_DISCONNECTED) - goto error_req; - - if (conn->state == SAP_STATE_CONNECT_IN_PROGRESS) - goto error_req; + if (conn->state == SAP_STATE_DISCONNECTED || + conn->state == SAP_STATE_CONNECT_IN_PROGRESS) + return -EPERM; if (conn->state == SAP_STATE_CONNECTED) { conn->state = SAP_STATE_GRACEFUL_DISCONNECT; @@ -367,11 +365,9 @@ static int disconnect_req(struct sap_connection *conn, uint8_t disc_type) return 0; case SAP_DISCONNECTION_TYPE_IMMEDIATE: - if (conn->state == SAP_STATE_DISCONNECTED) - goto error_req; - - if (conn->state == SAP_STATE_CONNECT_IN_PROGRESS) - goto error_req; + if (conn->state == SAP_STATE_DISCONNECTED || + conn->state == SAP_STATE_CONNECT_IN_PROGRESS) + return -EPERM; if (conn->state == SAP_STATE_CONNECTED || conn->state == SAP_STATE_GRACEFUL_DISCONNECT) { @@ -389,7 +385,7 @@ static int disconnect_req(struct sap_connection *conn, uint8_t disc_type) if (conn->state != SAP_STATE_CONNECTED && conn->state != SAP_STATE_GRACEFUL_DISCONNECT) { sap_error_rsp(conn); - goto error_req; + return -EPERM; } conn->state = SAP_STATE_CLIENT_DISCONNECT; @@ -404,9 +400,6 @@ static int disconnect_req(struct sap_connection *conn, uint8_t disc_type) error("Unknown disconnection type (0x%02x).", disc_type); return -EINVAL; } - -error_req: - return -EPERM; } static void transfer_apdu_req(struct sap_connection *conn, -- 2.47.3