s3:auth_sam: replace confusing FALL_THROUGH; with break;
authorStefan Metzmacher <metze@samba.org>
Thu, 23 Jan 2020 14:48:39 +0000 (15:48 +0100)
committerAndreas Schneider <asn@cryptomilk.org>
Wed, 5 Feb 2020 16:30:42 +0000 (16:30 +0000)
There's no real logic change here, but is makes it easier to
understand.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=14247

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
source3/auth/auth_sam.c

index 9310ef0e1f055ba1afab00408b7b7879ab87c064..bf93f19aa774bfa47d5d03f0bf458850e63ce3b8 100644 (file)
@@ -92,7 +92,7 @@ static NTSTATUS auth_samstrict_auth(const struct auth_context *auth_context,
                                return NT_STATUS_NOT_IMPLEMENTED;
                        }
 
-                       FALL_THROUGH;
+                       break;
                case ROLE_DOMAIN_PDC:
                case ROLE_DOMAIN_BDC:
                        if ( !is_local_name && !is_my_domain ) {
@@ -101,7 +101,7 @@ static NTSTATUS auth_samstrict_auth(const struct auth_context *auth_context,
                                return NT_STATUS_NOT_IMPLEMENTED;
                        }
 
-                       FALL_THROUGH;
+                       break;
                default: /* name is ok */
                        break;
        }