sctp: remove the typedef sctp_ierror_t
authorXin Long <lucien.xin@gmail.com>
Sat, 5 Aug 2017 11:59:58 +0000 (19:59 +0800)
committerDavid S. Miller <davem@davemloft.net>
Mon, 7 Aug 2017 04:33:42 +0000 (21:33 -0700)
This patch is to remove the typedef sctp_ierror_t, and
replace with enum sctp_ierror in the places where it's
using this typedef.

Signed-off-by: Xin Long <lucien.xin@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
include/net/sctp/constants.h
net/sctp/sm_make_chunk.c
net/sctp/sm_statefuns.c

index 311b2e3773e857574d14d46c038e82552d14f2f2..9a694653b7086be28ea96923585f918e6373e49e 100644 (file)
@@ -155,8 +155,7 @@ SCTP_SUBTYPE_CONSTRUCTOR(PRIMITIVE, sctp_event_primitive_t, primitive)
                                - sizeof(struct sctp_data_chunk)))
 
 /* Internal error codes */
-typedef enum {
-
+enum sctp_ierror {
        SCTP_IERROR_NO_ERROR            = 0,
        SCTP_IERROR_BASE                = 1000,
        SCTP_IERROR_NO_COOKIE,
@@ -177,7 +176,7 @@ typedef enum {
        SCTP_IERROR_PROTO_VIOLATION,
        SCTP_IERROR_ERROR,
        SCTP_IERROR_ABORT,
-} sctp_ierror_t;
+};
 
 
 
index a034d842e335760b672e5baee83771e976b64411..3a8fb1dffbc106dc64b4312f7dd2d8d28dda8fa2 100644 (file)
@@ -2065,10 +2065,11 @@ static void sctp_process_ext_param(struct sctp_association *asoc,
  *     SCTP_IERROR_ERROR    - stop and report an error.
  *     SCTP_IERROR_NOMEME   - out of memory.
  */
-static sctp_ierror_t sctp_process_unk_param(const struct sctp_association *asoc,
-                                           union sctp_params param,
-                                           struct sctp_chunk *chunk,
-                                           struct sctp_chunk **errp)
+static enum sctp_ierror sctp_process_unk_param(
+                                       const struct sctp_association *asoc,
+                                       union sctp_params param,
+                                       struct sctp_chunk *chunk,
+                                       struct sctp_chunk **errp)
 {
        int retval = SCTP_IERROR_NO_ERROR;
 
@@ -2117,13 +2118,13 @@ static sctp_ierror_t sctp_process_unk_param(const struct sctp_association *asoc,
  *     SCTP_IERROR_ERROR - stop processing, trigger an ERROR
  *     SCTP_IERROR_NO_ERROR - continue with the chunk
  */
-static sctp_ierror_t sctp_verify_param(struct net *net,
-                                       const struct sctp_endpoint *ep,
-                                       const struct sctp_association *asoc,
-                                       union sctp_params param,
-                                       enum sctp_cid cid,
-                                       struct sctp_chunk *chunk,
-                                       struct sctp_chunk **err_chunk)
+static enum sctp_ierror sctp_verify_param(struct net *net,
+                                         const struct sctp_endpoint *ep,
+                                         const struct sctp_association *asoc,
+                                         union sctp_params param,
+                                         enum sctp_cid cid,
+                                         struct sctp_chunk *chunk,
+                                         struct sctp_chunk **err_chunk)
 {
        struct sctp_hmac_algo_param *hmacs;
        int retval = SCTP_IERROR_NO_ERROR;
index 8af90a5f23cd829a877ac35a9e1063adc2dfdc4d..5381697333df9507356ffc93253bafc598a47dbd 100644 (file)
@@ -144,11 +144,12 @@ static sctp_disposition_t sctp_sf_violation_chunk(
                                     void *arg,
                                     sctp_cmd_seq_t *commands);
 
-static sctp_ierror_t sctp_sf_authenticate(struct net *net,
-                                   const struct sctp_endpoint *ep,
-                                   const struct sctp_association *asoc,
-                                   const sctp_subtype_t type,
-                                   struct sctp_chunk *chunk);
+static enum sctp_ierror sctp_sf_authenticate(
+                               struct net *net,
+                               const struct sctp_endpoint *ep,
+                               const struct sctp_association *asoc,
+                               const sctp_subtype_t type,
+                               struct sctp_chunk *chunk);
 
 static sctp_disposition_t __sctp_sf_do_9_1_abort(struct net *net,
                                        const struct sctp_endpoint *ep,
@@ -756,7 +757,7 @@ sctp_disposition_t sctp_sf_do_5_1D_ce(struct net *net,
         */
        if (chunk->auth_chunk) {
                struct sctp_chunk auth;
-               sctp_ierror_t ret;
+               enum sctp_ierror ret;
 
                /* Make sure that we and the peer are AUTH capable */
                if (!net->sctp.auth_enable || !new_asoc->peer.auth_capable) {
@@ -4077,11 +4078,12 @@ gen_shutdown:
  *
  * The return value is the disposition of the chunk.
  */
-static sctp_ierror_t sctp_sf_authenticate(struct net *net,
-                                   const struct sctp_endpoint *ep,
-                                   const struct sctp_association *asoc,
-                                   const sctp_subtype_t type,
-                                   struct sctp_chunk *chunk)
+static enum sctp_ierror sctp_sf_authenticate(
+                               struct net *net,
+                               const struct sctp_endpoint *ep,
+                               const struct sctp_association *asoc,
+                               const sctp_subtype_t type,
+                               struct sctp_chunk *chunk)
 {
        struct sctp_authhdr *auth_hdr;
        struct sctp_hmac *hmac;
@@ -4159,10 +4161,10 @@ sctp_disposition_t sctp_sf_eat_auth(struct net *net,
                                    void *arg,
                                    sctp_cmd_seq_t *commands)
 {
-       struct sctp_authhdr *auth_hdr;
        struct sctp_chunk *chunk = arg;
+       struct sctp_authhdr *auth_hdr;
        struct sctp_chunk *err_chunk;
-       sctp_ierror_t error;
+       enum sctp_ierror error;
 
        /* Make sure that the peer has AUTH capable */
        if (!asoc->peer.auth_capable)