sctp: Reduce switch/case indent
authorJoe Perches <joe@perches.com>
Fri, 1 Jul 2011 09:43:11 +0000 (09:43 +0000)
committerDavid S. Miller <davem@davemloft.net>
Fri, 1 Jul 2011 23:11:16 +0000 (16:11 -0700)
Make the case labels the same indent as the switch.

git diff -w shows useless break;s removed after returns
and a comment added to an unnecessary default: break;
because of a dubious gcc warning.

Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/sctp/sm_statefuns.c
net/sctp/socket.c

index a297283154d5035c6342e54916f10445aa610d83..7d00b1777c6363e11a30c52a94453d07f423017d 100644 (file)
@@ -4008,31 +4008,32 @@ sctp_disposition_t sctp_sf_eat_auth(const struct sctp_endpoint *ep,
        auth_hdr = (struct sctp_authhdr *)chunk->skb->data;
        error = sctp_sf_authenticate(ep, asoc, type, chunk);
        switch (error) {
-               case SCTP_IERROR_AUTH_BAD_HMAC:
-                       /* Generate the ERROR chunk and discard the rest
-                        * of the packet
-                        */
-                       err_chunk = sctp_make_op_error(asoc, chunk,
-                                                       SCTP_ERROR_UNSUP_HMAC,
-                                                       &auth_hdr->hmac_id,
-                                                       sizeof(__u16), 0);
-                       if (err_chunk) {
-                               sctp_add_cmd_sf(commands, SCTP_CMD_REPLY,
-                                               SCTP_CHUNK(err_chunk));
-                       }
-                       /* Fall Through */
-               case SCTP_IERROR_AUTH_BAD_KEYID:
-               case SCTP_IERROR_BAD_SIG:
-                       return sctp_sf_pdiscard(ep, asoc, type, arg, commands);
-                       break;
-               case SCTP_IERROR_PROTO_VIOLATION:
-                       return sctp_sf_violation_chunklen(ep, asoc, type, arg,
-                                                         commands);
-                       break;
-               case SCTP_IERROR_NOMEM:
-                       return SCTP_DISPOSITION_NOMEM;
-               default:
-                       break;
+       case SCTP_IERROR_AUTH_BAD_HMAC:
+               /* Generate the ERROR chunk and discard the rest
+                * of the packet
+                */
+               err_chunk = sctp_make_op_error(asoc, chunk,
+                                              SCTP_ERROR_UNSUP_HMAC,
+                                              &auth_hdr->hmac_id,
+                                              sizeof(__u16), 0);
+               if (err_chunk) {
+                       sctp_add_cmd_sf(commands, SCTP_CMD_REPLY,
+                                       SCTP_CHUNK(err_chunk));
+               }
+               /* Fall Through */
+       case SCTP_IERROR_AUTH_BAD_KEYID:
+       case SCTP_IERROR_BAD_SIG:
+               return sctp_sf_pdiscard(ep, asoc, type, arg, commands);
+
+       case SCTP_IERROR_PROTO_VIOLATION:
+               return sctp_sf_violation_chunklen(ep, asoc, type, arg,
+                                                 commands);
+
+       case SCTP_IERROR_NOMEM:
+               return SCTP_DISPOSITION_NOMEM;
+
+       default:                        /* Prevent gcc warnings */
+               break;
        }
 
        if (asoc->active_key_id != ntohs(auth_hdr->shkey_id)) {
index fd31b3616a33e22365b2f12652ddb40ce2b970e4..1c6aec1f9ec4dacba72accbfb97754b9322b60b8 100644 (file)
@@ -3257,11 +3257,11 @@ static int sctp_setsockopt_auth_chunk(struct sock *sk,
                return -EFAULT;
 
        switch (val.sauth_chunk) {
-               case SCTP_CID_INIT:
-               case SCTP_CID_INIT_ACK:
-               case SCTP_CID_SHUTDOWN_COMPLETE:
-               case SCTP_CID_AUTH:
-                       return -EINVAL;
+       case SCTP_CID_INIT:
+       case SCTP_CID_INIT_ACK:
+       case SCTP_CID_SHUTDOWN_COMPLETE:
+       case SCTP_CID_AUTH:
+               return -EINVAL;
        }
 
        /* add this chunk id to the endpoint */