diff --git a/src/internal.c b/src/internal.c index 9855a885..080ded6f 100644 --- a/src/internal.c +++ b/src/internal.c @@ -8286,7 +8286,7 @@ static int DoChannelOpen(WOLFSSH* ssh, #ifdef WOLFSSH_FWD case ID_CHANTYPE_TCPIP_DIRECT: isDirect = 1; - NO_BREAK; + FALL_THROUGH; case ID_CHANTYPE_TCPIP_FORWARD: ret = DoChannelOpenForward(ssh, &host, &hostPort, &origin, &originPort, @@ -9973,7 +9973,7 @@ int DoReceive(WOLFSSH* ssh) break; } } - NO_BREAK; + FALL_THROUGH; case PROCESS_PACKET_LENGTH: if (ssh->inputBuffer.idx + UINT32_SZ > ssh->inputBuffer.bufferSz) { @@ -9990,7 +9990,7 @@ int DoReceive(WOLFSSH* ssh) return WS_FATAL_ERROR; } ssh->processReplyState = PROCESS_PACKET_FINISH; - NO_BREAK; + FALL_THROUGH; case PROCESS_PACKET_FINISH: /* readSz is the full packet size */ @@ -10064,7 +10064,7 @@ int DoReceive(WOLFSSH* ssh) ret = WS_FATAL_ERROR; break; } - NO_BREAK; + FALL_THROUGH; case PROCESS_PACKET: ret = DoPacket(ssh, &bufferConsumed); @@ -10818,7 +10818,7 @@ static int SendKexGetSigningKey(WOLFSSH* ssh, #ifdef WOLFSSH_CERTS case ID_X509V3_SSH_RSA: isCert = 1; - NO_BREAK; + FALL_THROUGH; #endif case ID_SSH_RSA: case ID_RSA_SHA2_256: @@ -10922,7 +10922,7 @@ static int SendKexGetSigningKey(WOLFSSH* ssh, case ID_X509V3_ECDSA_SHA2_NISTP384: case ID_X509V3_ECDSA_SHA2_NISTP521: isCert = 1; - NO_BREAK; + FALL_THROUGH; #endif case ID_ECDSA_SHA2_NISTP256: case ID_ECDSA_SHA2_NISTP384: diff --git a/src/ssh.c b/src/ssh.c index 6d2012ad..33aff33a 100644 --- a/src/ssh.c +++ b/src/ssh.c @@ -515,7 +515,7 @@ int wolfSSH_accept(WOLFSSH* ssh) } ssh->acceptState = ACCEPT_SERVER_VERSION_SENT; WLOG(WS_LOG_DEBUG, acceptState, "SERVER_VERSION_SENT"); - NO_BREAK; + FALL_THROUGH; case ACCEPT_SERVER_VERSION_SENT: while (ssh->clientState < CLIENT_VERSION_DONE) { @@ -527,7 +527,7 @@ int wolfSSH_accept(WOLFSSH* ssh) } ssh->acceptState = ACCEPT_CLIENT_VERSION_DONE; WLOG(WS_LOG_DEBUG, acceptState, "CLIENT_VERSION_DONE"); - NO_BREAK; + FALL_THROUGH; case ACCEPT_CLIENT_VERSION_DONE: if ( (ssh->error = SendKexInit(ssh)) < WS_SUCCESS) { @@ -537,7 +537,7 @@ int wolfSSH_accept(WOLFSSH* ssh) } ssh->acceptState = ACCEPT_SERVER_KEXINIT_SENT; WLOG(WS_LOG_DEBUG, acceptState, "SERVER_KEXINIT_SENT"); - NO_BREAK; + FALL_THROUGH; case ACCEPT_SERVER_KEXINIT_SENT: while (ssh->isKeying) { @@ -549,7 +549,7 @@ int wolfSSH_accept(WOLFSSH* ssh) } ssh->acceptState = ACCEPT_KEYED; WLOG(WS_LOG_DEBUG, acceptState, "KEYED"); - NO_BREAK; + FALL_THROUGH; case ACCEPT_KEYED: while (ssh->clientState < CLIENT_USERAUTH_REQUEST_DONE) { @@ -561,7 +561,7 @@ int wolfSSH_accept(WOLFSSH* ssh) } ssh->acceptState = ACCEPT_CLIENT_USERAUTH_REQUEST_DONE; WLOG(WS_LOG_DEBUG, acceptState, "CLIENT_USERAUTH_REQUEST_DONE"); - NO_BREAK; + FALL_THROUGH; case ACCEPT_CLIENT_USERAUTH_REQUEST_DONE: if ( (ssh->error = SendServiceAccept(ssh, @@ -573,7 +573,7 @@ int wolfSSH_accept(WOLFSSH* ssh) ssh->acceptState = ACCEPT_SERVER_USERAUTH_ACCEPT_SENT; WLOG(WS_LOG_DEBUG, acceptState, "ACCEPT_SERVER_USERAUTH_ACCEPT_SENT"); - NO_BREAK; + FALL_THROUGH; case ACCEPT_SERVER_USERAUTH_ACCEPT_SENT: while (ssh->clientState < CLIENT_USERAUTH_DONE) { @@ -585,7 +585,7 @@ int wolfSSH_accept(WOLFSSH* ssh) } ssh->acceptState = ACCEPT_CLIENT_USERAUTH_DONE; WLOG(WS_LOG_DEBUG, acceptState, "CLIENT_USERAUTH_DONE"); - NO_BREAK; + FALL_THROUGH; case ACCEPT_CLIENT_USERAUTH_DONE: if ( (ssh->error = SendUserAuthSuccess(ssh)) < WS_SUCCESS) { @@ -595,7 +595,7 @@ int wolfSSH_accept(WOLFSSH* ssh) } ssh->acceptState = ACCEPT_SERVER_USERAUTH_SENT; WLOG(WS_LOG_DEBUG, acceptState, "SERVER_USERAUTH_SENT"); - NO_BREAK; + FALL_THROUGH; case ACCEPT_SERVER_USERAUTH_SENT: while (ssh->clientState < CLIENT_CHANNEL_OPEN_DONE) { @@ -607,7 +607,7 @@ int wolfSSH_accept(WOLFSSH* ssh) } ssh->acceptState = ACCEPT_SERVER_CHANNEL_ACCEPT_SENT; WLOG(WS_LOG_DEBUG, acceptState, "SERVER_CHANNEL_ACCEPT_SENT"); - NO_BREAK; + FALL_THROUGH; case ACCEPT_SERVER_CHANNEL_ACCEPT_SENT: while (ssh->clientState < CLIENT_DONE) { @@ -770,7 +770,7 @@ int wolfSSH_connect(WOLFSSH* ssh) } ssh->connectState = CONNECT_CLIENT_VERSION_SENT; WLOG(WS_LOG_DEBUG, connectState, "CLIENT_VERSION_SENT"); - NO_BREAK; + FALL_THROUGH; case CONNECT_CLIENT_VERSION_SENT: while (ssh->serverState < SERVER_VERSION_DONE) { @@ -782,7 +782,7 @@ int wolfSSH_connect(WOLFSSH* ssh) } ssh->connectState = CONNECT_SERVER_VERSION_DONE; WLOG(WS_LOG_DEBUG, connectState, "SERVER_VERSION_DONE"); - NO_BREAK; + FALL_THROUGH; case CONNECT_SERVER_VERSION_DONE: if ( (ssh->error = SendKexInit(ssh)) < WS_SUCCESS) { @@ -792,7 +792,7 @@ int wolfSSH_connect(WOLFSSH* ssh) } ssh->connectState = CONNECT_CLIENT_KEXINIT_SENT; WLOG(WS_LOG_DEBUG, connectState, "CLIENT_KEXINIT_SENT"); - NO_BREAK; + FALL_THROUGH; case CONNECT_CLIENT_KEXINIT_SENT: while (ssh->serverState < SERVER_KEXINIT_DONE) { @@ -804,7 +804,7 @@ int wolfSSH_connect(WOLFSSH* ssh) } ssh->connectState = CONNECT_SERVER_KEXINIT_DONE; WLOG(WS_LOG_DEBUG, connectState, "SERVER_KEXINIT_DONE"); - NO_BREAK; + FALL_THROUGH; case CONNECT_SERVER_KEXINIT_DONE: if (ssh->handshake == NULL) { @@ -825,7 +825,7 @@ int wolfSSH_connect(WOLFSSH* ssh) } ssh->connectState = CONNECT_CLIENT_KEXDH_INIT_SENT; WLOG(WS_LOG_DEBUG, connectState, "CLIENT_KEXDH_INIT_SENT"); - NO_BREAK; + FALL_THROUGH; case CONNECT_CLIENT_KEXDH_INIT_SENT: while (ssh->isKeying) { @@ -837,7 +837,7 @@ int wolfSSH_connect(WOLFSSH* ssh) } ssh->connectState = CONNECT_KEYED; WLOG(WS_LOG_DEBUG, connectState, "KEYED"); - NO_BREAK; + FALL_THROUGH; case CONNECT_KEYED: if ( (ssh->error = SendServiceRequest(ssh, ID_SERVICE_USERAUTH)) < @@ -847,7 +847,7 @@ int wolfSSH_connect(WOLFSSH* ssh) } ssh->connectState = CONNECT_CLIENT_USERAUTH_REQUEST_SENT; WLOG(WS_LOG_DEBUG, connectState, "CLIENT_USERAUTH_REQUEST_SENT"); - NO_BREAK; + FALL_THROUGH; case CONNECT_CLIENT_USERAUTH_REQUEST_SENT: while (ssh->serverState < SERVER_USERAUTH_REQUEST_DONE) { @@ -859,7 +859,7 @@ int wolfSSH_connect(WOLFSSH* ssh) } ssh->connectState = CONNECT_SERVER_USERAUTH_REQUEST_DONE; WLOG(WS_LOG_DEBUG, connectState, "SERVER_USERAUTH_REQUEST_DONE"); - NO_BREAK; + FALL_THROUGH; case CONNECT_SERVER_USERAUTH_REQUEST_DONE: #ifdef WOLFSSH_AGENT @@ -880,7 +880,7 @@ int wolfSSH_connect(WOLFSSH* ssh) } ssh->connectState = CONNECT_CLIENT_USERAUTH_SENT; WLOG(WS_LOG_DEBUG, connectState, "CLIENT_USERAUTH_SENT"); - NO_BREAK; + FALL_THROUGH; case CONNECT_CLIENT_USERAUTH_SENT: while (ssh->serverState < SERVER_USERAUTH_ACCEPT_DONE) { @@ -892,7 +892,7 @@ int wolfSSH_connect(WOLFSSH* ssh) } ssh->connectState = CONNECT_SERVER_USERAUTH_ACCEPT_DONE; WLOG(WS_LOG_DEBUG, connectState, "SERVER_USERAUTH_ACCEPT_DONE"); - NO_BREAK; + FALL_THROUGH; case CONNECT_SERVER_USERAUTH_ACCEPT_DONE: { @@ -924,7 +924,7 @@ int wolfSSH_connect(WOLFSSH* ssh) ssh->connectState = CONNECT_CLIENT_CHANNEL_OPEN_SESSION_SENT; WLOG(WS_LOG_DEBUG, connectState, "CLIENT_CHANNEL_OPEN_SESSION_SENT"); - NO_BREAK; + FALL_THROUGH; case CONNECT_CLIENT_CHANNEL_OPEN_SESSION_SENT: while (ssh->serverState < SERVER_CHANNEL_OPEN_DONE) { @@ -937,7 +937,7 @@ int wolfSSH_connect(WOLFSSH* ssh) ssh->connectState = CONNECT_SERVER_CHANNEL_OPEN_SESSION_DONE; WLOG(WS_LOG_DEBUG, connectState, "SERVER_CHANNEL_OPEN_SESSION_DONE"); - NO_BREAK; + FALL_THROUGH; case CONNECT_SERVER_CHANNEL_OPEN_SESSION_DONE: #ifdef WOLFSSH_AGENT @@ -953,7 +953,7 @@ int wolfSSH_connect(WOLFSSH* ssh) WLOG(WS_LOG_DEBUG, connectState, "CLIENT_CHANNEL_AGENT_REQUEST_SENT"); ssh->connectState = CONNECT_CLIENT_CHANNEL_AGENT_REQUEST_SENT; - NO_BREAK; + FALL_THROUGH; case CONNECT_CLIENT_CHANNEL_AGENT_REQUEST_SENT: #if defined(WOLFSSH_TERM) && !defined(NO_FILESYSTEM) @@ -969,7 +969,7 @@ int wolfSSH_connect(WOLFSSH* ssh) WLOG(WS_LOG_DEBUG, connectState, "CLIENT_CHANNEL_TERMINAL_REQUEST_SENT"); ssh->connectState = CONNECT_CLIENT_CHANNEL_TERMINAL_REQUEST_SENT; - NO_BREAK; + FALL_THROUGH; case CONNECT_CLIENT_CHANNEL_TERMINAL_REQUEST_SENT: if ( (ssh->error = SendChannelRequest(ssh, ssh->channelName, @@ -981,7 +981,7 @@ int wolfSSH_connect(WOLFSSH* ssh) ssh->connectState = CONNECT_CLIENT_CHANNEL_REQUEST_SENT; WLOG(WS_LOG_DEBUG, connectState, "CLIENT_CHANNEL_REQUEST_SENT"); - NO_BREAK; + FALL_THROUGH; case CONNECT_CLIENT_CHANNEL_REQUEST_SENT: while (ssh->serverState < SERVER_DONE) { @@ -1530,7 +1530,7 @@ int wolfSSH_SetChannelType(WOLFSSH* ssh, byte type, byte* name, word32 nameSz) WLOG(WS_LOG_DEBUG, "Server side exec unsupported"); return WS_BAD_ARGUMENT; } - NO_BREAK; + FALL_THROUGH; case WOLFSSH_SESSION_SUBSYSTEM: ssh->connectChannelId = type; diff --git a/src/wolfscp.c b/src/wolfscp.c index d490b1fb..9a7dfc88 100644 --- a/src/wolfscp.c +++ b/src/wolfscp.c @@ -1408,7 +1408,7 @@ int ReceiveScpMessage(WOLFSSH* ssh) switch (buf[0]) { case 'C': - NO_BREAK; + FALL_THROUGH; case 'D': if (buf[0] == 'C') { @@ -1512,7 +1512,7 @@ int SendScpConfirmation(WOLFSSH* ssh) case WS_SCP_CONTINUE: /* default to ok confirmation */ - NO_BREAK; + FALL_THROUGH; default: msg[0] = SCP_CONFIRM_OK; @@ -1566,9 +1566,9 @@ int ReceiveScpConfirmation(WOLFSSH* ssh) case SCP_CONFIRM_OK: break; case SCP_CONFIRM_ERR: - NO_BREAK; + FALL_THROUGH; case SCP_CONFIRM_FATAL: - NO_BREAK; + FALL_THROUGH; default: WLOG(WS_LOG_ERROR, "scp error: peer sent error confirmation (code: %d)", diff --git a/src/wolfsftp.c b/src/wolfsftp.c index e593d2b4..cf434cda 100644 --- a/src/wolfsftp.c +++ b/src/wolfsftp.c @@ -1071,7 +1071,7 @@ static int SFTP_ServerRecvInit(WOLFSSH* ssh) { state->extSz = sz - MSG_ID_SZ - UINT32_SZ; ssh->sftpState = SFTP_EXT; - NO_BREAK; + FALL_THROUGH; case SFTP_EXT: /* silently ignore extensions if not supported */ @@ -1157,7 +1157,7 @@ int wolfSSH_SFTP_accept(WOLFSSH* ssh) return ret; } ssh->sftpState = SFTP_RECV; - NO_BREAK; + FALL_THROUGH; case SFTP_RECV: ret = SFTP_ServerSendInit(ssh); @@ -1394,7 +1394,7 @@ int wolfSSH_SFTP_read(WOLFSSH* ssh) ssh->reqId = state->reqId; state->state = STATE_RECV_DO; - NO_BREAK; + FALL_THROUGH; case STATE_RECV_DO: ret = wolfSSH_SFTP_buffer_read(ssh, &state->buffer, @@ -1556,7 +1556,7 @@ int wolfSSH_SFTP_read(WOLFSSH* ssh) } state->buffer.idx = 0; state->state = STATE_RECV_SEND; - NO_BREAK; + FALL_THROUGH; case STATE_RECV_SEND: if (state->toSend) { @@ -5684,7 +5684,7 @@ static int SFTP_ClientRecvInit(WOLFSSH* ssh) { sz = sz - MSG_ID_SZ - UINT32_SZ; ssh->sftpExtSz = sz; ssh->sftpState = SFTP_EXT; - NO_BREAK; + FALL_THROUGH; case SFTP_EXT: /* silently ignore extensions if not supported */ @@ -5776,7 +5776,7 @@ int wolfSSH_SFTP_connect(WOLFSSH* ssh) return WS_FATAL_ERROR; } ssh->sftpState = SFTP_RECV; - NO_BREAK; + FALL_THROUGH; case SFTP_RECV: case SFTP_EXT: @@ -5888,7 +5888,7 @@ int SendPacketType(WOLFSSH* ssh, byte type, byte* buf, word32 bufSz) wolfSSH_SFTP_buffer_rewind(&state->buffer); } state->state = SFTP_SEND_PACKET; - NO_BREAK; + FALL_THROUGH; case SFTP_SEND_PACKET: /* send header and type specific state->data, looping over send @@ -6344,7 +6344,7 @@ static WS_SFTPNAME* wolfSSH_SFTP_DoName(WOLFSSH* ssh) wolfSSH_SFTP_ClearState(ssh, STATE_ID_NAME); return NULL; } - NO_BREAK; + FALL_THROUGH; case SFTP_NAME_DO_STATUS: if (state->state == SFTP_NAME_DO_STATUS) { @@ -6364,7 +6364,7 @@ static WS_SFTPNAME* wolfSSH_SFTP_DoName(WOLFSSH* ssh) } return NULL; } - NO_BREAK; + FALL_THROUGH; case SFTP_NAME_GET_PACKET: @@ -6530,7 +6530,7 @@ static int wolfSSH_SFTP_GetHandle(WOLFSSH* ssh, byte* handle, word32* handleSz) case STATE_GET_HANDLE_INIT: WLOG(WS_LOG_SFTP, "SFTP GET HANDLE STATE: INIT"); state->state = STATE_GET_HANDLE_GET_HEADER; - NO_BREAK; + FALL_THROUGH; case STATE_GET_HANDLE_GET_HEADER: WLOG(WS_LOG_SFTP, "SFTP GET HANDLE STATE: GET_HEADER"); @@ -6602,7 +6602,7 @@ static int wolfSSH_SFTP_GetHandle(WOLFSSH* ssh, byte* handle, word32* handleSz) ssh->reqId++; state->state = STATE_GET_HANDLE_READ; - NO_BREAK; + FALL_THROUGH; case STATE_GET_HANDLE_READ: WLOG(WS_LOG_SFTP, "SFTP GET HANDLE STATE: READ"); @@ -6630,7 +6630,7 @@ static int wolfSSH_SFTP_GetHandle(WOLFSSH* ssh, byte* handle, word32* handleSz) (wolfSSH_SFTP_buffer_data(&state->buffer) + UINT32_SZ), *handleSz); state->state = STATE_GET_HANDLE_CLEANUP; - NO_BREAK; + FALL_THROUGH; case STATE_GET_HANDLE_CLEANUP: WLOG(WS_LOG_SFTP, "SFTP GET HANDLE STATE: CLEANUP"); @@ -6692,7 +6692,7 @@ WS_SFTPNAME* wolfSSH_SFTP_LS(WOLFSSH* ssh, char* dir) return NULL; } state->state = STATE_LS_OPENDIR; - NO_BREAK; + FALL_THROUGH; case STATE_LS_OPENDIR: if (wolfSSH_SFTP_OpenDir(ssh, (byte*)state->name->fName, @@ -6709,7 +6709,7 @@ WS_SFTPNAME* wolfSSH_SFTP_LS(WOLFSSH* ssh, char* dir) wolfSSH_SFTPNAME_list_free(state->name); state->name = NULL; state->sz = WOLFSSH_MAX_HANDLE; state->state = STATE_LS_GETHANDLE; - NO_BREAK; + FALL_THROUGH; case STATE_LS_GETHANDLE: /* get the handle from opening the directory and read with it */ @@ -6724,7 +6724,7 @@ WS_SFTPNAME* wolfSSH_SFTP_LS(WOLFSSH* ssh, char* dir) return NULL; } state->state = STATE_LS_READDIR; - NO_BREAK; + FALL_THROUGH; case STATE_LS_READDIR: /* Now read the dir. Note in non-blocking we might get here multiple @@ -6767,7 +6767,7 @@ WS_SFTPNAME* wolfSSH_SFTP_LS(WOLFSSH* ssh, char* dir) } state->state = STATE_LS_CLOSE; - NO_BREAK; + FALL_THROUGH; case STATE_LS_CLOSE: /* close dir when finished */ @@ -6843,7 +6843,7 @@ int wolfSSH_SFTP_CHMOD(WOLFSSH* ssh, char* n, char* oct) /* update permissions */ state->atr.per = mode; state->state = STATE_CHMOD_SEND; - NO_BREAK; + FALL_THROUGH; case STATE_CHMOD_SEND: ret = wolfSSH_SFTP_SetSTAT(ssh, n, &state->atr); @@ -6901,7 +6901,7 @@ static int SFTP_STAT(WOLFSSH* ssh, char* dir, WS_SFTP_FILEATRB* atr, byte type) WLOG(WS_LOG_SFTP, "SFTP LSTAT STATE: INIT"); state->dirSz = (word32)WSTRLEN(dir); state->state = STATE_LSTAT_SEND_TYPE_REQ; - NO_BREAK; + FALL_THROUGH; case STATE_LSTAT_SEND_TYPE_REQ: WLOG(WS_LOG_SFTP, "SFTP LSTAT STATE: SEND_TYPE_REQ"); @@ -6917,7 +6917,7 @@ static int SFTP_STAT(WOLFSSH* ssh, char* dir, WS_SFTP_FILEATRB* atr, byte type) } } state->state = STATE_LSTAT_GET_HEADER; - NO_BREAK; + FALL_THROUGH; case STATE_LSTAT_GET_HEADER: WLOG(WS_LOG_SFTP, "SFTP LSTAT STATE: GET_HEADER"); @@ -6941,7 +6941,7 @@ static int SFTP_STAT(WOLFSSH* ssh, char* dir, WS_SFTP_FILEATRB* atr, byte type) ssh->error = WS_MEMORY_E; return WS_FATAL_ERROR; } - NO_BREAK; + FALL_THROUGH; case STATE_LSTAT_CHECK_REQ_ID: /* check request ID */ @@ -6953,7 +6953,7 @@ static int SFTP_STAT(WOLFSSH* ssh, char* dir, WS_SFTP_FILEATRB* atr, byte type) ssh->reqId++; } state->state = STATE_LSTAT_PARSE_REPLY; - NO_BREAK; + FALL_THROUGH; case STATE_LSTAT_PARSE_REPLY: ret = wolfSSH_SFTP_buffer_read(ssh, &state->buffer, @@ -7003,7 +7003,7 @@ static int SFTP_STAT(WOLFSSH* ssh, char* dir, WS_SFTP_FILEATRB* atr, byte type) return WS_FATAL_ERROR; } state->state = STATE_LSTAT_CLEANUP; - NO_BREAK; + FALL_THROUGH; case STATE_LSTAT_CLEANUP: WLOG(WS_LOG_SFTP, "SFTP LSTAT STATE: CLEANUP"); @@ -7123,7 +7123,7 @@ int wolfSSH_SFTP_SetSTAT(WOLFSSH* ssh, char* dir, WS_SFTP_FILEATRB* atr) wolfSSH_SFTP_buffer_rewind(&state->buffer); state->state = STATE_SET_ATR_SEND; - NO_BREAK; + FALL_THROUGH; /* send header and type specific data */ case STATE_SET_ATR_SEND: @@ -7140,7 +7140,7 @@ int wolfSSH_SFTP_SetSTAT(WOLFSSH* ssh, char* dir, WS_SFTP_FILEATRB* atr) * can be created when next reading the attribute packet header */ wolfSSH_SFTP_buffer_free(ssh, &state->buffer); state->state = STATE_SET_ATR_GET; - NO_BREAK; + FALL_THROUGH; case STATE_SET_ATR_GET: @@ -7167,7 +7167,7 @@ int wolfSSH_SFTP_SetSTAT(WOLFSSH* ssh, char* dir, WS_SFTP_FILEATRB* atr) } state->state = STATE_SET_ATR_STATUS; - NO_BREAK; + FALL_THROUGH; case STATE_SET_ATR_STATUS: @@ -7282,7 +7282,7 @@ int wolfSSH_SFTP_Open(WOLFSSH* ssh, char* dir, word32 reason, wolfSSH_SFTP_buffer_rewind(&state->buffer); state->state = STATE_OPEN_SEND; - NO_BREAK; + FALL_THROUGH; case STATE_OPEN_SEND: WLOG(WS_LOG_SFTP, "SFTP OPEN STATE: SEND"); @@ -7299,7 +7299,7 @@ int wolfSSH_SFTP_Open(WOLFSSH* ssh, char* dir, word32 reason, } wolfSSH_SFTP_buffer_free(ssh, &state->buffer); state->state = STATE_OPEN_GETHANDLE; - NO_BREAK; + FALL_THROUGH; case STATE_OPEN_GETHANDLE: WLOG(WS_LOG_SFTP, "SFTP OPEN STATE: GETHANDLE"); @@ -7314,7 +7314,7 @@ int wolfSSH_SFTP_Open(WOLFSSH* ssh, char* dir, word32 reason, } } state->state = STATE_OPEN_CLEANUP; - NO_BREAK; + FALL_THROUGH; case STATE_OPEN_CLEANUP: WLOG(WS_LOG_SFTP, "SFTP OPEN STATE: CLEANUP"); @@ -7416,7 +7416,7 @@ int wolfSSH_SFTP_SendWritePacket(WOLFSSH* ssh, byte* handle, word32 handleSz, wolfSSH_SFTP_buffer_rewind(&state->buffer); state->state = STATE_SEND_WRITE_SEND_HEADER; - NO_BREAK; + FALL_THROUGH; case STATE_SEND_WRITE_SEND_HEADER: WLOG(WS_LOG_SFTP, "SFTP SEND_WRITE STATE: SEND_HEADER"); @@ -7431,7 +7431,7 @@ int wolfSSH_SFTP_SendWritePacket(WOLFSSH* ssh, byte* handle, word32 handleSz, continue; } state->state = STATE_SEND_WRITE_SEND_BODY; - NO_BREAK; + FALL_THROUGH; case STATE_SEND_WRITE_SEND_BODY: WLOG(WS_LOG_SFTP, "SFTP SEND_WRITE STATE: SEND_BODY"); @@ -7459,7 +7459,7 @@ int wolfSSH_SFTP_SendWritePacket(WOLFSSH* ssh, byte* handle, word32 handleSz, wolfSSH_SFTP_buffer_free(ssh, &state->buffer); state->state = STATE_SEND_WRITE_GET_HEADER; - NO_BREAK; + FALL_THROUGH; case STATE_SEND_WRITE_GET_HEADER: WLOG(WS_LOG_SFTP, "SFTP SEND_WRITE STATE: GET_HEADER"); @@ -7504,7 +7504,7 @@ int wolfSSH_SFTP_SendWritePacket(WOLFSSH* ssh, byte* handle, word32 handleSz, continue; } state->state = STATE_SEND_WRITE_READ_STATUS; - NO_BREAK; + FALL_THROUGH; case STATE_SEND_WRITE_READ_STATUS: WLOG(WS_LOG_SFTP, "SFTP SEND_WRITE STATE: READ_STATUS"); @@ -7520,7 +7520,7 @@ int wolfSSH_SFTP_SendWritePacket(WOLFSSH* ssh, byte* handle, word32 handleSz, } wolfSSH_SFTP_buffer_rewind(&state->buffer); state->state = STATE_SEND_WRITE_DO_STATUS; - NO_BREAK; + FALL_THROUGH; case STATE_SEND_WRITE_DO_STATUS: WLOG(WS_LOG_SFTP, "SFTP SEND_WRITE STATE: DO_STATUS"); @@ -7536,7 +7536,7 @@ int wolfSSH_SFTP_SendWritePacket(WOLFSSH* ssh, byte* handle, word32 handleSz, if (ret >= WS_SUCCESS) ret = state->sentSzSave; state->state = STATE_SEND_WRITE_CLEANUP; - NO_BREAK; + FALL_THROUGH; case STATE_SEND_WRITE_CLEANUP: WLOG(WS_LOG_SFTP, "SFTP SEND_WRITE STATE: CLEANUP"); @@ -7640,7 +7640,7 @@ int wolfSSH_SFTP_SendReadPacket(WOLFSSH* ssh, byte* handle, word32 handleSz, state->state = STATE_SEND_READ_SEND_REQ; - NO_BREAK; + FALL_THROUGH; case STATE_SEND_READ_SEND_REQ: WLOG(WS_LOG_SFTP, "SFTP SEND_READ STATE: SEND_REQ"); @@ -7659,7 +7659,7 @@ int wolfSSH_SFTP_SendReadPacket(WOLFSSH* ssh, byte* handle, word32 handleSz, } wolfSSH_SFTP_buffer_free(ssh, &state->buffer); state->state = STATE_SEND_READ_GET_HEADER; - NO_BREAK; + FALL_THROUGH; case STATE_SEND_READ_GET_HEADER: WLOG(WS_LOG_SFTP, "SFTP SEND_READ STATE: GET_HEADER"); @@ -7680,7 +7680,7 @@ int wolfSSH_SFTP_SendReadPacket(WOLFSSH* ssh, byte* handle, word32 handleSz, continue; } state->state = STATE_SEND_READ_CHECK_REQ_ID; - NO_BREAK; + FALL_THROUGH; case STATE_SEND_READ_CHECK_REQ_ID: WLOG(WS_LOG_SFTP, "SFTP SEND_READ STATE: CHECK_REQ_ID"); @@ -7727,7 +7727,7 @@ int wolfSSH_SFTP_SendReadPacket(WOLFSSH* ssh, byte* handle, word32 handleSz, } state->state = STATE_SEND_READ_REMAINDER; - NO_BREAK; + FALL_THROUGH; case STATE_SEND_READ_REMAINDER: WLOG(WS_LOG_SFTP, "SFTP SEND_READ STATE: READ_REMAINDER"); @@ -7794,7 +7794,7 @@ int wolfSSH_SFTP_SendReadPacket(WOLFSSH* ssh, byte* handle, word32 handleSz, } } state->state = STATE_SEND_READ_CLEANUP; - NO_BREAK; + FALL_THROUGH; case STATE_SEND_READ_CLEANUP: WLOG(WS_LOG_SFTP, "SFTP SEND_READ STATE: CLEANUP"); @@ -7898,7 +7898,7 @@ int wolfSSH_SFTP_MKDIR(WOLFSSH* ssh, char* dir, WS_SFTP_FILEATRB* atr) /* free data pointer to reuse it later */ wolfSSH_SFTP_buffer_free(ssh, &state->buffer); state->state = STATE_MKDIR_GET; - NO_BREAK; + FALL_THROUGH; case STATE_MKDIR_GET: /* Get response */ @@ -7930,7 +7930,7 @@ int wolfSSH_SFTP_MKDIR(WOLFSSH* ssh, char* dir, WS_SFTP_FILEATRB* atr) return WS_FATAL_ERROR; } state->state = STATE_MKDIR_STATUS; - NO_BREAK; + FALL_THROUGH; case STATE_MKDIR_STATUS: ret = wolfSSH_SFTP_buffer_read(ssh, &state->buffer, @@ -7999,7 +7999,7 @@ WS_SFTPNAME* wolfSSH_SFTP_ReadDir(WOLFSSH* ssh, byte* handle, return NULL; } state->state = STATE_READDIR_NAME; - NO_BREAK; + FALL_THROUGH; case STATE_READDIR_NAME: name = wolfSSH_SFTP_DoName(ssh); @@ -8053,7 +8053,7 @@ int wolfSSH_SFTP_Close(WOLFSSH* ssh, byte* handle, word32 handleSz) case STATE_CLOSE_INIT: WLOG(WS_LOG_SFTP, "SFTP CLOSE STATE: INIT"); state->state = STATE_CLOSE_SEND; - NO_BREAK; + FALL_THROUGH; case STATE_CLOSE_SEND: WLOG(WS_LOG_SFTP, "SFTP CLOSE STATE: SEND"); @@ -8068,7 +8068,7 @@ int wolfSSH_SFTP_Close(WOLFSSH* ssh, byte* handle, word32 handleSz) continue; } state->state = STATE_CLOSE_GET_HEADER; - NO_BREAK; + FALL_THROUGH; case STATE_CLOSE_GET_HEADER: WLOG(WS_LOG_SFTP, "SFTP CLOSE STATE: GET_HEADER"); @@ -8091,7 +8091,7 @@ int wolfSSH_SFTP_Close(WOLFSSH* ssh, byte* handle, word32 handleSz) continue; } state->state = STATE_CLOSE_DO_STATUS; - NO_BREAK; + FALL_THROUGH; case STATE_CLOSE_DO_STATUS: WLOG(WS_LOG_SFTP, "SFTP CLOSE STATE: DO_STATUS"); @@ -8112,7 +8112,7 @@ int wolfSSH_SFTP_Close(WOLFSSH* ssh, byte* handle, word32 handleSz) else ret = WS_FATAL_ERROR; state->state = STATE_CLOSE_CLEANUP; - NO_BREAK; + FALL_THROUGH; case STATE_CLOSE_CLEANUP: WLOG(WS_LOG_SFTP, "SFTP CLOSE STATE: CLEANUP"); @@ -8156,7 +8156,7 @@ WS_SFTPNAME* wolfSSH_SFTP_RealPath(WOLFSSH* ssh, char* dir) return NULL; } ssh->realState = SFTP_REAL_GET_PACKET; - NO_BREAK; + FALL_THROUGH; case SFTP_REAL_GET_PACKET: /* read name response from Real Path packet */ @@ -8230,7 +8230,7 @@ int wolfSSH_SFTP_Rename(WOLFSSH* ssh, const char* old, const char* nw) case STATE_RENAME_INIT: WLOG(WS_LOG_SFTP, "SFTP RENAME STATE: INIT"); - NO_BREAK; + FALL_THROUGH; case STATE_RENAME_GET_STAT: WLOG(WS_LOG_SFTP, "SFTP RENAME STATE: GET_STAT"); @@ -8295,7 +8295,7 @@ int wolfSSH_SFTP_Rename(WOLFSSH* ssh, const char* old, const char* nw) wolfSSH_SFTP_buffer_rewind(&state->buffer); state->state = STATE_RENAME_SEND; - NO_BREAK; + FALL_THROUGH; case STATE_RENAME_SEND: WLOG(WS_LOG_SFTP, "SFTP RENAME STATE: SEND"); @@ -8311,7 +8311,7 @@ int wolfSSH_SFTP_Rename(WOLFSSH* ssh, const char* old, const char* nw) } wolfSSH_SFTP_buffer_free(ssh, &state->buffer); state->state = STATE_RENAME_GET_HEADER; - NO_BREAK; + FALL_THROUGH; case STATE_RENAME_GET_HEADER: WLOG(WS_LOG_SFTP, "SFTP RENAME STATE: GET_HEADER"); @@ -8353,7 +8353,7 @@ int wolfSSH_SFTP_Rename(WOLFSSH* ssh, const char* old, const char* nw) continue; } state->state = STATE_RENAME_READ_STATUS; - NO_BREAK; + FALL_THROUGH; case STATE_RENAME_READ_STATUS: WLOG(WS_LOG_SFTP, "SFTP RENAME STATE: READ_STATUS"); @@ -8369,7 +8369,7 @@ int wolfSSH_SFTP_Rename(WOLFSSH* ssh, const char* old, const char* nw) } wolfSSH_SFTP_buffer_rewind(&state->buffer); state->state = STATE_RENAME_DO_STATUS; - NO_BREAK; + FALL_THROUGH; case STATE_RENAME_DO_STATUS: WLOG(WS_LOG_SFTP, "SFTP RENAME STATE: DO_STATUS"); @@ -8386,7 +8386,7 @@ int wolfSSH_SFTP_Rename(WOLFSSH* ssh, const char* old, const char* nw) ret = WS_SFTP_STATUS_NOT_OK; } state->state = STATE_RENAME_CLEANUP; - NO_BREAK; + FALL_THROUGH; case STATE_RENAME_CLEANUP: WLOG(WS_LOG_SFTP, "SFTP RENAME STATE: CLEANUP"); @@ -8453,7 +8453,7 @@ int wolfSSH_SFTP_Remove(WOLFSSH* ssh, char* f) return ret; } state->state = STATE_RM_SEND; - NO_BREAK; + FALL_THROUGH; case STATE_RM_SEND: ret = SendPacketType(ssh, WOLFSSH_FTP_REMOVE, (byte*)f, @@ -8466,7 +8466,7 @@ int wolfSSH_SFTP_Remove(WOLFSSH* ssh, char* f) return ret; } state->state = STATE_RM_GET; - NO_BREAK; + FALL_THROUGH; case STATE_RM_GET: ret = SFTP_GetHeader(ssh, &state->reqId, &type, &state->buffer); @@ -8484,7 +8484,7 @@ int wolfSSH_SFTP_Remove(WOLFSSH* ssh, char* f) return WS_FATAL_ERROR; } state->state = STATE_RM_DOSTATUS; - NO_BREAK; + FALL_THROUGH; case STATE_RM_DOSTATUS: ret = wolfSSH_SFTP_buffer_read(ssh, &state->buffer, @@ -8559,7 +8559,7 @@ int wolfSSH_SFTP_RMDIR(WOLFSSH* ssh, char* dir) return ret; } state->state = STATE_RMDIR_GET; - NO_BREAK; + FALL_THROUGH; case STATE_RMDIR_GET: ret = SFTP_GetHeader(ssh, &state->reqId, &type, &state->buffer); @@ -8575,7 +8575,7 @@ int wolfSSH_SFTP_RMDIR(WOLFSSH* ssh, char* dir) return WS_MEMORY_E; } state->state = STATE_RMDIR_STATUS; - NO_BREAK; + FALL_THROUGH; case STATE_RMDIR_STATUS: ret = wolfSSH_SFTP_buffer_read(ssh, &state->buffer, @@ -8783,7 +8783,7 @@ int wolfSSH_SFTP_Get(WOLFSSH* ssh, char* from, case STATE_GET_INIT: WLOG(WS_LOG_SFTP, "SFTP GET STATE: INIT"); state->state = STATE_GET_LSTAT; - NO_BREAK; + FALL_THROUGH; case STATE_GET_LSTAT: WLOG(WS_LOG_SFTP, "SFTP GET STATE: STAT"); @@ -8808,7 +8808,7 @@ int wolfSSH_SFTP_Get(WOLFSSH* ssh, char* from, } state->handleSz = WOLFSSH_MAX_HANDLE; state->state = STATE_GET_OPEN_REMOTE; - NO_BREAK; + FALL_THROUGH; case STATE_GET_OPEN_REMOTE: WLOG(WS_LOG_SFTP, "SFTP GET STATE: OPEN REMOTE"); @@ -8825,7 +8825,7 @@ int wolfSSH_SFTP_Get(WOLFSSH* ssh, char* from, continue; } state->state = STATE_GET_LOOKUP_OFFSET; - NO_BREAK; + FALL_THROUGH; case STATE_GET_LOOKUP_OFFSET: WLOG(WS_LOG_SFTP, "SFTP GET STATE: LOOKUP OFFSET"); @@ -8834,7 +8834,7 @@ int wolfSSH_SFTP_Get(WOLFSSH* ssh, char* from, wolfSSH_SFTP_GetOfst(ssh, from, to, state->gOfst); } state->state = STATE_GET_OPEN_LOCAL; - NO_BREAK; + FALL_THROUGH; case STATE_GET_OPEN_LOCAL: WLOG(WS_LOG_SFTP, "SFTP GET STATE: OPEN LOCAL"); @@ -8872,7 +8872,7 @@ int wolfSSH_SFTP_Get(WOLFSSH* ssh, char* from, continue; } state->state = STATE_GET_READ; - NO_BREAK; + FALL_THROUGH; case STATE_GET_READ: WLOG(WS_LOG_SFTP, "SFTP GET STATE: GET READ"); @@ -8934,7 +8934,7 @@ int wolfSSH_SFTP_Get(WOLFSSH* ssh, char* from, } ssh->sftpInt = 0; state->state = STATE_GET_CLOSE_REMOTE; - NO_BREAK; + FALL_THROUGH; case STATE_GET_CLOSE_REMOTE: if (state->handleSz > 0) { @@ -8950,7 +8950,7 @@ int wolfSSH_SFTP_Get(WOLFSSH* ssh, char* from, } } state->state = STATE_GET_CLOSE_LOCAL; - NO_BREAK; + FALL_THROUGH; case STATE_GET_CLOSE_LOCAL: WLOG(WS_LOG_SFTP, "SFTP GET STATE: CLOSE LOCAL"); @@ -8964,7 +8964,7 @@ int wolfSSH_SFTP_Get(WOLFSSH* ssh, char* from, } #endif /* USE_WINDOWS_API */ state->state = STATE_GET_CLEANUP; - NO_BREAK; + FALL_THROUGH; case STATE_GET_CLEANUP: WLOG(WS_LOG_SFTP, "SFTP GET STATE: CLEANUP"); @@ -9029,7 +9029,7 @@ int wolfSSH_SFTP_Put(WOLFSSH* ssh, char* from, char* to, byte resume, state->pOfst[0] = 0; state->pOfst[1] = 0; state->state = STATE_PUT_LOOKUP_OFFSET; - NO_BREAK; + FALL_THROUGH; case STATE_PUT_LOOKUP_OFFSET: WLOG(WS_LOG_SFTP, "SFTP PUT STATE: LOOKUP OFFSET"); @@ -9039,7 +9039,7 @@ int wolfSSH_SFTP_Put(WOLFSSH* ssh, char* from, char* to, byte resume, } state->handleSz = WOLFSSH_MAX_HANDLE; state->state = STATE_PUT_OPEN_LOCAL; - NO_BREAK; + FALL_THROUGH; case STATE_PUT_OPEN_LOCAL: WLOG(WS_LOG_SFTP, "SFTP PUT STATE: OPEN LOCAL"); @@ -9098,7 +9098,7 @@ int wolfSSH_SFTP_Put(WOLFSSH* ssh, char* from, char* to, byte resume, #endif /* USE_WINDOWS_API */ state->rSz = 0; state->state = STATE_PUT_OPEN_REMOTE; - NO_BREAK; + FALL_THROUGH; case STATE_PUT_OPEN_REMOTE: WLOG(WS_LOG_SFTP, "SFTP PUT STATE: OPEN REMOTE"); @@ -9116,7 +9116,7 @@ int wolfSSH_SFTP_Put(WOLFSSH* ssh, char* from, char* to, byte resume, continue; } state->state = STATE_PUT_WRITE; - NO_BREAK; + FALL_THROUGH; case STATE_PUT_WRITE: WLOG(WS_LOG_SFTP, "SFTP PUT STATE: WRITE"); @@ -9162,7 +9162,7 @@ int wolfSSH_SFTP_Put(WOLFSSH* ssh, char* from, char* to, byte resume, wolfSSH_SFTP_SaveOfst(ssh, from, to, state->pOfst); ssh->sftpInt = 0; } - NO_BREAK; + FALL_THROUGH; case STATE_PUT_CLOSE_LOCAL: WLOG(WS_LOG_SFTP, "SFTP PUT STATE: CLOSE LOCAL"); @@ -9172,7 +9172,7 @@ int wolfSSH_SFTP_Put(WOLFSSH* ssh, char* from, char* to, byte resume, CloseHandle(state->fileHandle); #endif /* USE_WINDOWS_API */ state->state = STATE_PUT_CLOSE_REMOTE; - NO_BREAK; + FALL_THROUGH; case STATE_PUT_CLOSE_REMOTE: if (state->handleSz > 0) { @@ -9190,7 +9190,7 @@ int wolfSSH_SFTP_Put(WOLFSSH* ssh, char* from, char* to, byte resume, } } state->state = STATE_PUT_CLEANUP; - NO_BREAK; + FALL_THROUGH; case STATE_PUT_CLEANUP: WLOG(WS_LOG_SFTP, "SFTP PUT STATE: CLEANUP"); diff --git a/wolfssh/port.h b/wolfssh/port.h index 59da7b8d..eb870bac 100644 --- a/wolfssh/port.h +++ b/wolfssh/port.h @@ -1510,17 +1510,6 @@ extern "C" { #endif /* INLINE */ -/* GCC 7 has new switch() fall-through detection */ -#if defined(__GNUC__) && !defined(NO_BREAK) - #if ((__GNUC__ > 7) || ((__GNUC__ == 7) && (__GNUC_MINOR__ >= 1))) - #define NO_BREAK __attribute__ ((fallthrough)) - #endif -#endif -#ifndef NO_BREAK - #define NO_BREAK -#endif - - #ifndef WOLFSSH_UNUSED #define WOLFSSH_UNUSED(arg) (void)(arg) #endif