diff options
| author | Amitay Isaacs <amitay@gmail.com> | 2018-03-04 02:09:33 +1100 |
|---|---|---|
| committer | Martin Schwenke <martins@samba.org> | 2018-03-09 07:08:23 +0100 |
| commit | f70164d65958956c689f0e2ee3bdd0c7634c146f (patch) | |
| tree | 83d90097e755fe45384603956157273132998ff8 /ctdb/ib/ibwrapper.c | |
| parent | f0bebcc4e046ee668c83030c432f99586a17207c (diff) | |
| download | samba-f70164d65958956c689f0e2ee3bdd0c7634c146f.tar.gz | |
ctdb-ib: Avoid fall through case statements
This is clearly unintended. Noticed with gcc 7.3.
Signed-off-by: Amitay Isaacs <amitay@gmail.com>
Reviewed-by: Martin Schwenke <martin@meltin.net>
Diffstat (limited to 'ctdb/ib/ibwrapper.c')
| -rw-r--r-- | ctdb/ib/ibwrapper.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/ctdb/ib/ibwrapper.c b/ctdb/ib/ibwrapper.c index f7334b528e7..5ca3b946f00 100644 --- a/ctdb/ib/ibwrapper.c +++ b/ctdb/ib/ibwrapper.c @@ -503,10 +503,13 @@ static void ibw_event_handler_cm(struct tevent_context *ev, case RDMA_CM_EVENT_ADDR_ERROR: sprintf(ibw_lasterr, "RDMA_CM_EVENT_ADDR_ERROR, error %d\n", event->status); + goto error; case RDMA_CM_EVENT_ROUTE_ERROR: sprintf(ibw_lasterr, "RDMA_CM_EVENT_ROUTE_ERROR, error %d\n", event->status); + goto error; case RDMA_CM_EVENT_CONNECT_ERROR: sprintf(ibw_lasterr, "RDMA_CM_EVENT_CONNECT_ERROR, error %d\n", event->status); + goto error; case RDMA_CM_EVENT_UNREACHABLE: sprintf(ibw_lasterr, "RDMA_CM_EVENT_UNREACHABLE, error %d\n", event->status); goto error; |