Skip to content

Commit

Permalink
Remove useless cast in setsockopt call (closes #721)
Browse files Browse the repository at this point in the history
  • Loading branch information
stephane committed Jul 17, 2024
1 parent 8e769e7 commit dde16d5
Showing 1 changed file with 4 additions and 6 deletions.
10 changes: 4 additions & 6 deletions src/modbus-tcp.c
Original file line number Diff line number Diff line change
Expand Up @@ -230,7 +230,7 @@ static int _modbus_tcp_set_ipv4_options(int s)
/* Set the TCP no delay flag */
/* SOL_TCP = IPPROTO_TCP */
option = 1;
rc = setsockopt(s, IPPROTO_TCP, TCP_NODELAY, (const void *) &option, sizeof(int));
rc = setsockopt(s, IPPROTO_TCP, TCP_NODELAY, &option, sizeof(int));
if (rc == -1) {
return -1;
}
Expand Down Expand Up @@ -258,7 +258,7 @@ static int _modbus_tcp_set_ipv4_options(int s)
**/
/* Set the IP low delay option */
option = IPTOS_LOWDELAY;
rc = setsockopt(s, IPPROTO_IP, IP_TOS, (const void *) &option, sizeof(int));
rc = setsockopt(s, IPPROTO_IP, IP_TOS, &option, sizeof(int));
if (rc == -1) {
return -1;
}
Expand Down Expand Up @@ -547,8 +547,7 @@ int modbus_tcp_listen(modbus_t *ctx, int nb_connection)
}

enable = 1;
if (setsockopt(new_s, SOL_SOCKET, SO_REUSEADDR, (char *) &enable, sizeof(enable)) ==
-1) {
if (setsockopt(new_s, SOL_SOCKET, SO_REUSEADDR, &enable, sizeof(enable)) == -1) {
close(new_s);
return -1;
}
Expand Down Expand Up @@ -665,8 +664,7 @@ int modbus_tcp_pi_listen(modbus_t *ctx, int nb_connection)
continue;
} else {
int enable = 1;
rc =
setsockopt(s, SOL_SOCKET, SO_REUSEADDR, (void *) &enable, sizeof(enable));
rc = setsockopt(s, SOL_SOCKET, SO_REUSEADDR, &enable, sizeof(enable));
if (rc != 0) {
close(s);
if (ctx->debug) {
Expand Down

0 comments on commit dde16d5

Please sign in to comment.