]> git.ipfire.org Git - thirdparty/freeradius-server.git/commitdiff
fix const / threading issues
authorAlan T. DeKok <aland@freeradius.org>
Wed, 24 Oct 2018 17:07:46 +0000 (13:07 -0400)
committerAlan T. DeKok <aland@freeradius.org>
Wed, 24 Oct 2018 17:07:46 +0000 (13:07 -0400)
src/modules/proto_dhcpv4/proto_dhcpv4_udp.c
src/modules/proto_radius/proto_radius_udp.c

index 00b9d2273abd52921c37aef374f57636f0d8703e..f8acba95c7b9e0862deb82b2a56f1757c32c5763 100644 (file)
@@ -221,7 +221,7 @@ static ssize_t mod_read(fr_listen_t *li, void **packet_ctx, fr_time_t **recv_tim
 static ssize_t mod_write(fr_listen_t *li, void *packet_ctx, UNUSED fr_time_t request_time,
                         uint8_t *buffer, size_t buffer_len, UNUSED size_t written)
 {
-       proto_dhcpv4_udp_t              *inst = talloc_get_type_abort(li->app_io_instance, proto_dhcpv4_udp_t);
+       proto_dhcpv4_udp_t const        *inst = talloc_get_type_abort_const(li->app_io_instance, proto_dhcpv4_udp_t);
        proto_dhcpv4_udp_thread_t       *thread = talloc_get_type_abort(li->thread_instance, proto_dhcpv4_udp_thread_t);
 
        fr_io_track_t                   *track = talloc_get_type_abort(packet_ctx, fr_io_track_t);
@@ -466,7 +466,7 @@ static void mod_network_get(void *instance, int *ipproto, bool *dynamic_clients,
  */
 static int mod_open(fr_listen_t *li)
 {
-       proto_dhcpv4_udp_t              *inst = talloc_get_type_abort(li->app_io_instance, proto_dhcpv4_udp_t);
+       proto_dhcpv4_udp_t const        *inst = talloc_get_type_abort_const(li->app_io_instance, proto_dhcpv4_udp_t);
        proto_dhcpv4_udp_thread_t       *thread = talloc_get_type_abort(li->thread_instance, proto_dhcpv4_udp_thread_t);
 
        int                             sockfd;
@@ -520,9 +520,9 @@ static int mod_open(fr_listen_t *li)
 
        server_cs = cf_item_to_section(ci);
 
-       thread->name = fr_app_io_socket_name(inst, &proto_dhcpv4_udp,
-                                          NULL, 0,
-                                          &inst->ipaddr, inst->port);
+       thread->name = fr_app_io_socket_name(thread, &proto_dhcpv4_udp,
+                                            NULL, 0,
+                                            &inst->ipaddr, inst->port);
 
        DEBUG("Listening on dhcpv4 address %s bound to virtual server %s",
              thread->name, cf_section_name2(server_cs));
@@ -536,14 +536,14 @@ static int mod_open(fr_listen_t *li)
  */
 static int mod_fd_set(fr_listen_t *li, int fd)
 {
-       proto_dhcpv4_udp_t              *inst = talloc_get_type_abort(li->app_io_instance, proto_dhcpv4_udp_t);
+       proto_dhcpv4_udp_t const        *inst = talloc_get_type_abort_const(li->app_io_instance, proto_dhcpv4_udp_t);
        proto_dhcpv4_udp_thread_t       *thread = talloc_get_type_abort(li->thread_instance, proto_dhcpv4_udp_thread_t);
 
        thread->sockfd = fd;
 
-       thread->name = fr_app_io_socket_name(inst, &proto_dhcpv4_udp,
-                                          &thread->connection->src_ipaddr, thread->connection->src_port,
-                                          &inst->ipaddr, inst->port);
+       thread->name = fr_app_io_socket_name(thread, &proto_dhcpv4_udp,
+                                            &thread->connection->src_ipaddr, thread->connection->src_port,
+                                            &inst->ipaddr, inst->port);
 
        return 0;
 }
index 5e09ef6d5917d1efe25561885b8bc7fbc0cfdbe1..7d74e9699d81bda809856182b9079029591cd09b 100644 (file)
@@ -111,7 +111,7 @@ static const CONF_PARSER udp_listen_config[] = {
 
 static ssize_t mod_read(fr_listen_t *li, void **packet_ctx, fr_time_t **recv_time, uint8_t *buffer, size_t buffer_len, size_t *leftover, UNUSED uint32_t *priority, UNUSED bool *is_dup)
 {
-       proto_radius_udp_t              *inst = talloc_get_type_abort(li->app_io_instance, proto_radius_udp_t);
+       proto_radius_udp_t const        *inst = talloc_get_type_abort_const(li->app_io_instance, proto_radius_udp_t);
        proto_radius_udp_thread_t       *thread = talloc_get_type_abort(li->thread_instance, proto_radius_udp_thread_t);
        fr_io_address_t                 *address, **address_p;
 
@@ -302,7 +302,7 @@ static void mod_network_get(void *instance, int *ipproto, bool *dynamic_clients,
  */
 static int mod_open(fr_listen_t *li)
 {
-       proto_radius_udp_t              *inst = talloc_get_type_abort(li->app_io_instance, proto_radius_udp_t);
+       proto_radius_udp_t const        *inst = talloc_get_type_abort_const(li->app_io_instance, proto_radius_udp_t);
        proto_radius_udp_thread_t       *thread = talloc_get_type_abort(li->thread_instance, proto_radius_udp_thread_t);
 
        int                             sockfd;
@@ -367,9 +367,9 @@ static int mod_open(fr_listen_t *li)
 
        server_cs = cf_item_to_section(ci);
 
-       thread->name = fr_app_io_socket_name(inst, &proto_radius_udp,
-                                          NULL, 0,
-                                          &inst->ipaddr, inst->port);
+       thread->name = fr_app_io_socket_name(thread, &proto_radius_udp,
+                                            NULL, 0,
+                                            &inst->ipaddr, inst->port);
 
        // @todo - also print out auth / acct / coa, etc.
        DEBUG("Listening on radius address %s bound to virtual server %s",
@@ -383,14 +383,14 @@ static int mod_open(fr_listen_t *li)
  */
 static int mod_fd_set(fr_listen_t *li, int fd)
 {
-       proto_radius_udp_t              *inst = talloc_get_type_abort(li->app_io_instance, proto_radius_udp_t);
+       proto_radius_udp_t const        *inst = talloc_get_type_abort_const(li->app_io_instance, proto_radius_udp_t);
        proto_radius_udp_thread_t       *thread = talloc_get_type_abort(li->thread_instance, proto_radius_udp_thread_t);
 
        thread->sockfd = fd;
 
-       thread->name = fr_app_io_socket_name(inst, &proto_radius_udp,
-                                          &thread->connection->src_ipaddr, thread->connection->src_port,
-                                          &inst->ipaddr, inst->port);
+       thread->name = fr_app_io_socket_name(thread, &proto_radius_udp,
+                                            &thread->connection->src_ipaddr, thread->connection->src_port,
+                                            &inst->ipaddr, inst->port);
 
        return 0;
 }
@@ -519,7 +519,7 @@ static int mod_bootstrap(void *instance, CONF_SECTION *cs)
 
 static RADCLIENT *mod_client_find(fr_listen_t *li, fr_ipaddr_t const *ipaddr, int ipproto)
 {
-       proto_radius_udp_t      *inst = talloc_get_type_abort(li->app_io_instance, proto_radius_udp_t);
+       proto_radius_udp_t const        *inst = talloc_get_type_abort_const(li->app_io_instance, proto_radius_udp_t);
        RADCLIENT               *client;
 
        /*