]> git.ipfire.org Git - thirdparty/freeradius-server.git/commitdiff
remove struct signing
authorAlan T. DeKok <aland@freeradius.org>
Tue, 7 Aug 2018 13:18:09 +0000 (09:18 -0400)
committerAlan T. DeKok <aland@freeradius.org>
Tue, 7 Aug 2018 13:18:09 +0000 (09:18 -0400)
src/lib/io/master.c
src/lib/server/client.c

index 3f36a444e4ab608c55c6a95b0e5330315c4b23d9..394a92c3d5b9700953aad12151f5a01d164c2d7b 100644 (file)
@@ -324,9 +324,6 @@ static RADCLIENT *radclient_clone(TALLOC_CTX *ctx, RADCLIENT const *parent)
        COPY_FIELD(tls_required);
 #endif
 
-       FR_STRUCT_MEMBER_SIGN(c, secret, talloc_array_length(c->secret));
-       FR_STRUCT_SIGN(c);
-
        return c;
 
        /*
@@ -449,7 +446,6 @@ static fr_io_connection_t *fr_io_connection_alloc(fr_io_instance_t *inst, fr_io_
         */
        radclient->dynamic = true;
        radclient->active = true;
-       FR_STRUCT_SIGN(radclient);
 
        /*
         *      address->client points to a "static" client.  We want
@@ -506,7 +502,6 @@ static fr_io_connection_t *fr_io_connection_alloc(fr_io_instance_t *inst, fr_io_
                 *      that define a dynamic client.
                 */
                radclient->active = false;
-               FR_STRUCT_SIGN(radclient);
                break;
 
        case PR_CLIENT_STATIC:
@@ -1120,7 +1115,6 @@ do_read:
                         */
                        MEM(radclient = radclient_clone(inst, radclient));
                        radclient->active = true;
-                       FR_STRUCT_SIGN(radclient);
 
                } else if (inst->dynamic_clients) {
                        if (inst->max_clients && (inst->num_clients >= inst->max_clients)) {
@@ -1262,8 +1256,6 @@ have_client:
         *      Track this packet and return it if necessary.
         */
        if (connection || !client->use_connected) {
-               FR_STRUCT_MEMBER_VERIFY(client->radclient, secret, talloc_array_length(client->radclient->secret));
-               FR_STRUCT_VERIFY(client->radclient);
 
                /*
                 *      Add the packet to the tracking table, if it's
@@ -2077,8 +2069,6 @@ static ssize_t mod_write(void *instance, void *packet_ctx, fr_time_t request_tim
        radclient->server_cs = inst->server_cs;
        radclient->server = cf_section_name2(inst->server_cs);
        radclient->cs = NULL;
-       FR_STRUCT_MEMBER_SIGN(radclient, secret, talloc_array_length(radclient->secret));
-       FR_STRUCT_SIGN(radclient);
 
        /*
         *      This is a connected socket, and it's just been
@@ -2093,7 +2083,6 @@ static ssize_t mod_write(void *instance, void *packet_ctx, fr_time_t request_tim
                client->state = PR_CLIENT_CONNECTED;
 
                radclient->active = true;
-               FR_STRUCT_SIGN(radclient);
 
                /*
                 *      Connections can't spawn new connections.
@@ -2157,7 +2146,6 @@ static ssize_t mod_write(void *instance, void *packet_ctx, fr_time_t request_tim
                 */
                client->state = PR_CLIENT_DYNAMIC;
                client->radclient->active = true;
-               FR_STRUCT_SIGN(radclient);
        }
 
        /*
index dae220ce1467add590aa4f35597a7905896ccaf5..f258d232194cb647d268a4676bde8be4636231ed 100644 (file)
@@ -331,9 +331,6 @@ bool client_add(RADCLIENT_LIST *clients, RADCLIENT *client)
         */
        (void) talloc_steal(clients, client); /* reparent it */
 
-       FR_STRUCT_MEMBER_SIGN(client, secret, talloc_array_length(client->secret));
-       FR_STRUCT_SIGN(client);
-
        return true;
 }
 
@@ -411,8 +408,6 @@ RADCLIENT *client_find(RADCLIENT_LIST const *clients, fr_ipaddr_t const *ipaddr,
                fr_ipaddr_mask(&my_client.ipaddr, i);
                client = rbtree_finddata(clients->tree[i], &my_client);
                if (client) {
-                       FR_STRUCT_MEMBER_VERIFY(client, secret, talloc_array_length(client->secret));
-                       FR_STRUCT_VERIFY(client);
                        return client;
                }
        }