]> www.infradead.org Git - users/dwmw2/openconnect.git/commitdiff
Fix compiler warnings when buliding on AppVeyor
authorDimitri Papadopoulos Orfanos <3350651-DimitriPapadopoulos@users.noreply.gitlab.com>
Sun, 28 Jan 2024 19:43:03 +0000 (20:43 +0100)
committerDimitri Papadopoulos Orfanos <3350651-DimitriPapadopoulos@users.noreply.gitlab.com>
Sun, 28 Jan 2024 19:43:03 +0000 (20:43 +0100)
../mtucalc.c: In function 'calculate_mtu':
../mtucalc.c:75:33: warning: passing argument 4 of 'getsockopt' from incompatible pointer type [-Wincompatible-pointer-types]
   75 |                                 &mss, &mss_size)) {
      |                                 ^~~~
      |                                 |
      |                                 int *
In file included from ../openconnect-internal.h:31,
                 from ../mtucalc.c:20:
C:/msys64/mingw64/include/winsock2.h:1010:82: note: expected 'char *' but argument is of type 'int *'
 1010 |   WINSOCK_API_LINKAGE int WSAAPI getsockopt(SOCKET s,int level,int optname,char *optval,int *optlen);
      |                                                                            ~~~~~~^~~~~~
  CC       libopenconnect_la-lzo.lo

../cstp.c: In function 'calculate_dtls_mtu':
../cstp.c:134:33: warning: passing argument 4 of 'getsockopt' from incompatible pointer type [-Wincompatible-pointer-types]
  134 |                                 &mss, &mss_size)) {
      |                                 ^~~~
      |                                 |
      |                                 int *
In file included from ../openconnect-internal.h:31,
                 from ../cstp.c:21:
C:/msys64/mingw64/include/winsock2.h:1010:82: note: expected 'char *' but argument is of type 'int *'
 1010 |   WINSOCK_API_LINKAGE int WSAAPI getsockopt(SOCKET s,int level,int optname,char *optval,int *optlen);
      |                                                                            ~~~~~~^~~~~~

Signed-off-by: Dimitri Papadopoulos Orfanos <3350651-DimitriPapadopoulos@users.noreply.gitlab.com>
cstp.c
mtucalc.c

diff --git a/cstp.c b/cstp.c
index 8c3ebfbcf676849e030889c706ae4d59ad21daa2..f3dda05e95d1aa08300b450c0df73deb2204ae96 100644 (file)
--- a/cstp.c
+++ b/cstp.c
@@ -131,7 +131,7 @@ static void calculate_dtls_mtu(struct openconnect_info *vpninfo, int *base_mtu,
                int mss;
                socklen_t mss_size = sizeof(mss);
                if (!getsockopt(vpninfo->ssl_fd, IPPROTO_TCP, TCP_MAXSEG,
-                               &mss, &mss_size)) {
+                               (void *)&mss, &mss_size)) {
                        vpn_progress(vpninfo, PRG_DEBUG, _("TCP_MAXSEG %d\n"), mss);
                        *base_mtu = mss - 13;
                }
index 0b96be05cd723c2a7bed0542bcbc5c9636d18e2a..3c5ef3897fab2fa866e30a9fc17253fdac50ee21 100644 (file)
--- a/mtucalc.c
+++ b/mtucalc.c
@@ -72,7 +72,7 @@ int calculate_mtu(struct openconnect_info *vpninfo, int is_udp,
        if (!mtu && !mss) {
                socklen_t mss_size = sizeof(mss);
                if (!getsockopt(vpninfo->ssl_fd, IPPROTO_TCP, TCP_MAXSEG,
-                               &mss, &mss_size)) {
+                               (void *)&mss, &mss_size)) {
                        vpn_progress(vpninfo, PRG_DEBUG, _("TCP_MAXSEG %d\n"), mss);
                }
        }