Suppres implicit-fallthrough from gcc 7

Some kind of comments are recognized by gcc7 but they are ignored with
-Wimplicit-fallthrough=5 and only attributes disable the warning.

Reviewed-by: Fabiano Fidêncio <fidencio@redhat.com>

5 files changed, 8 insertions, 0 deletions

diff --git a/src/providers/ad/ad_id.c b/src/providers/ad/ad_id.c
index 46a41a37b..8f26cb874 100644
--- a/src/providers/ad/ad_id.c
+++ b/src/providers/ad/ad_id.c
@@ -337,6 +337,7 @@ static bool ad_account_can_shortcut(struct be_ctx *be_ctx,
             goto done;
         }
         /* fall through */
+        SSS_ATTRIBUTE_FALLTHROUGH;
     case BE_FILTER_SECID:
         csid = sid == NULL ? filter_value : sid;
 
diff --git a/src/providers/fail_over.c b/src/providers/fail_over.c
index 770840988..5d3c26d4a 100644
--- a/src/providers/fail_over.c
+++ b/src/providers/fail_over.c
@@ -1145,6 +1145,7 @@ fo_resolve_service_server(struct tevent_req *req)
                                 state->server->common);
         fo_set_server_status(state->server, SERVER_RESOLVING_NAME);
         /* FALLTHROUGH */
+        SSS_ATTRIBUTE_FALLTHROUGH;
     case SERVER_RESOLVING_NAME:
         /* Name resolution is already under way. Just add ourselves into the
          * waiting queue so we get notified after the operation is finished. */
@@ -1284,6 +1285,7 @@ resolve_srv_send(TALLOC_CTX *mem_ctx, struct tevent_context *ev,
          * "server" might be invalid now if the SRV
          * query collapsed
          * */
+        SSS_ATTRIBUTE_FALLTHROUGH;
     case SRV_NEUTRAL: /* Request SRV lookup */
         if (server != NULL && server != state->meta) {
             /* A server created by expansion of meta server was marked as
@@ -1443,9 +1445,11 @@ resolve_srv_done(struct tevent_req *subreq)
         break;
     case ERR_SRV_NOT_FOUND:
         /* fall through */
+        SSS_ATTRIBUTE_FALLTHROUGH;
     case ERR_SRV_LOOKUP_ERROR:
         fo_set_port_status(state->meta, PORT_NOT_WORKING);
         /* fall through */
+        SSS_ATTRIBUTE_FALLTHROUGH;
     default:
         DEBUG(SSSDBG_OP_FAILURE, "Unable to resolve SRV [%d]: %s\n",
                                   ret, sss_strerror(ret));
diff --git a/src/providers/krb5/krb5_auth.c b/src/providers/krb5/krb5_auth.c
index bdd8e2411..0e685618e 100644
--- a/src/providers/krb5/krb5_auth.c
+++ b/src/providers/krb5/krb5_auth.c
@@ -965,6 +965,7 @@ static void krb5_auth_done(struct tevent_req *subreq)
             DEBUG(SSSDBG_CRIT_FAILURE, "krb5_delete_ccname failed.\n");
         }
         /* FALLTHROUGH */
+        SSS_ATTRIBUTE_FALLTHROUGH;
 
     case ERR_CREDS_EXPIRED:
         /* If the password is expired we can safely remove the ccache from the
diff --git a/src/providers/ldap/sdap_idmap.c b/src/providers/ldap/sdap_idmap.c
index b5dfc6cef..0fda81522 100644
--- a/src/providers/ldap/sdap_idmap.c
+++ b/src/providers/ldap/sdap_idmap.c
@@ -516,6 +516,7 @@ sdap_idmap_sid_to_unix(struct sdap_idmap_ctx *idmap_ctx,
               "sssd-ad(5) for an explanation of how to resolve this issue.\n",
               sid_str);
         /* Fall through intentionally */
+        SSS_ATTRIBUTE_FALLTHROUGH;
     default:
         DEBUG(SSSDBG_MINOR_FAILURE,
               "Could not convert objectSID [%s] to a UNIX ID\n",
diff --git a/src/providers/proxy/proxy_id.c b/src/providers/proxy/proxy_id.c
index 3d272897b..9b83f7a3c 100644
--- a/src/providers/proxy/proxy_id.c
+++ b/src/providers/proxy/proxy_id.c
@@ -1403,6 +1403,7 @@ static int get_initgr_groups_process(TALLOC_CTX *memctx,
                                  "Assume the user is only member of its "
                                  "primary group (%"SPRIgid")\n", pwd->pw_gid);
         /* fall through */
+        SSS_ATTRIBUTE_FALLTHROUGH;
     case NSS_STATUS_SUCCESS:
         DEBUG(SSSDBG_CONF_SETTINGS, "User [%s] appears to be member of %lu "
               "groups\n", pwd->pw_name, num_gids);