From 04b9418dd9dd8175a25130c909dd9586a8fb1227 Mon Sep 17 00:00:00 2001 From: Andrea Ciliberti Date: Mon, 9 Sep 2024 15:19:50 +0200 Subject: [PATCH] Use macro expansion for maximum sockaddr storage length rather than hardcoded value --- libctru/source/services/soc/soc_accept.c | 6 +++--- libctru/source/services/soc/soc_bind.c | 6 +++--- libctru/source/services/soc/soc_common.h | 1 + libctru/source/services/soc/soc_connect.c | 6 +++--- libctru/source/services/soc/soc_getnameinfo.c | 4 ++-- libctru/source/services/soc/soc_getpeername.c | 6 +++--- libctru/source/services/soc/soc_getsockname.c | 6 +++--- libctru/source/services/soc/soc_recvfrom.c | 12 ++++++------ libctru/source/services/soc/soc_sendto.c | 12 ++++++------ 9 files changed, 30 insertions(+), 29 deletions(-) diff --git a/libctru/source/services/soc/soc_accept.c b/libctru/source/services/soc/soc_accept.c index 67b80d4..ef8be22 100644 --- a/libctru/source/services/soc/soc_accept.c +++ b/libctru/source/services/soc/soc_accept.c @@ -6,10 +6,10 @@ int accept(int sockfd, struct sockaddr *addr, socklen_t *addrlen) { int ret = 0; - int tmp_addrlen = 0x1c; + int tmp_addrlen = ADDR_STORAGE_LEN; int fd, dev; u32 *cmdbuf = getThreadCommandBuffer(); - u8 tmpaddr[0x1c]; + u8 tmpaddr[ADDR_STORAGE_LEN]; u32 saved_threadstorage[2]; sockfd = soc_get_fd(sockfd); @@ -27,7 +27,7 @@ int accept(int sockfd, struct sockaddr *addr, socklen_t *addrlen) fd = __alloc_handle(dev); if(fd < 0) return fd; - memset(tmpaddr, 0, 0x1c); + memset(tmpaddr, 0, ADDR_STORAGE_LEN); cmdbuf[0] = IPC_MakeHeader(0x4,2,2); // 0x40082 cmdbuf[1] = (u32)sockfd; diff --git a/libctru/source/services/soc/soc_bind.c b/libctru/source/services/soc/soc_bind.c index 94b2ac5..3a52622 100644 --- a/libctru/source/services/soc/soc_bind.c +++ b/libctru/source/services/soc/soc_bind.c @@ -8,7 +8,7 @@ int bind(int sockfd, const struct sockaddr *addr, socklen_t addrlen) int ret = 0; int tmp_addrlen = 0; u32 *cmdbuf = getThreadCommandBuffer(); - u8 tmpaddr[0x1c]; + u8 tmpaddr[ADDR_STORAGE_LEN]; sockfd = soc_get_fd(sockfd); if(sockfd < 0) { @@ -16,12 +16,12 @@ int bind(int sockfd, const struct sockaddr *addr, socklen_t addrlen) return -1; } - memset(tmpaddr, 0, 0x1c); + memset(tmpaddr, 0, ADDR_STORAGE_LEN); if(addr->sa_family == AF_INET) tmp_addrlen = 8; else - tmp_addrlen = 0x1c; + tmp_addrlen = ADDR_STORAGE_LEN; if(addrlen < tmp_addrlen) { errno = EINVAL; diff --git a/libctru/source/services/soc/soc_common.h b/libctru/source/services/soc/soc_common.h index e859ac5..85759eb 100644 --- a/libctru/source/services/soc/soc_common.h +++ b/libctru/source/services/soc/soc_common.h @@ -10,6 +10,7 @@ #include <3ds/services/soc.h> #define SYNC_ERROR ENODEV +#define ADDR_STORAGE_LEN sizeof(struct sockaddr_storage) extern Handle SOCU_handle; extern Handle socMemhandle; diff --git a/libctru/source/services/soc/soc_connect.c b/libctru/source/services/soc/soc_connect.c index e48f35d..19955f1 100644 --- a/libctru/source/services/soc/soc_connect.c +++ b/libctru/source/services/soc/soc_connect.c @@ -8,7 +8,7 @@ int connect(int sockfd, const struct sockaddr *addr, socklen_t addrlen) int ret = 0; int tmp_addrlen = 0; u32 *cmdbuf = getThreadCommandBuffer(); - u8 tmpaddr[0x1c]; + u8 tmpaddr[ADDR_STORAGE_LEN]; sockfd = soc_get_fd(sockfd); if(sockfd < 0) { @@ -16,12 +16,12 @@ int connect(int sockfd, const struct sockaddr *addr, socklen_t addrlen) return -1; } - memset(tmpaddr, 0, 0x1c); + memset(tmpaddr, 0, ADDR_STORAGE_LEN); if(addr->sa_family == AF_INET) tmp_addrlen = 8; else - tmp_addrlen = 0x1c; + tmp_addrlen = ADDR_STORAGE_LEN; if(addrlen < tmp_addrlen) { errno = EINVAL; diff --git a/libctru/source/services/soc/soc_getnameinfo.c b/libctru/source/services/soc/soc_getnameinfo.c index 08f37cb..e6cf111 100644 --- a/libctru/source/services/soc/soc_getnameinfo.c +++ b/libctru/source/services/soc/soc_getnameinfo.c @@ -8,7 +8,7 @@ int getnameinfo(const struct sockaddr *sa, socklen_t salen, char *host, socklen_ int i,tmp_addrlen; u32 *cmdbuf = getThreadCommandBuffer(); u32 saved_threadstorage[4]; - u8 tmpaddr[0x1c]; // sockaddr size for the kernel is 0x1C (sockaddr_in6?) + u8 tmpaddr[ADDR_STORAGE_LEN]; // sockaddr size for the kernel is 0x1C (sockaddr_in6?) if((host == NULL || hostlen == 0) && (serv == NULL || servlen == 0)) { @@ -18,7 +18,7 @@ int getnameinfo(const struct sockaddr *sa, socklen_t salen, char *host, socklen_ if(sa->sa_family == AF_INET) tmp_addrlen = 8; else - tmp_addrlen = 0x1c; + tmp_addrlen = ADDR_STORAGE_LEN; if(salen < tmp_addrlen) { errno = EINVAL; diff --git a/libctru/source/services/soc/soc_getpeername.c b/libctru/source/services/soc/soc_getpeername.c index 269a781..10ed709 100644 --- a/libctru/source/services/soc/soc_getpeername.c +++ b/libctru/source/services/soc/soc_getpeername.c @@ -8,7 +8,7 @@ int getpeername(int sockfd, struct sockaddr *addr, socklen_t *addrlen) int ret = 0; u32 *cmdbuf = getThreadCommandBuffer(); u32 saved_threadstorage[2]; - u8 tmpaddr[0x1c]; + u8 tmpaddr[ADDR_STORAGE_LEN]; sockfd = soc_get_fd(sockfd); if(sockfd < 0) { @@ -18,14 +18,14 @@ int getpeername(int sockfd, struct sockaddr *addr, socklen_t *addrlen) cmdbuf[0] = IPC_MakeHeader(0x18,2,2); // 0x180082 cmdbuf[1] = (u32)sockfd; - cmdbuf[2] = 0x1c; + cmdbuf[2] = ADDR_STORAGE_LEN; cmdbuf[3] = IPC_Desc_CurProcessId(); u32 * staticbufs = getThreadStaticBuffers(); saved_threadstorage[0] = staticbufs[0]; saved_threadstorage[1] = staticbufs[1]; - staticbufs[0] = IPC_Desc_StaticBuffer(0x1c,0); + staticbufs[0] = IPC_Desc_StaticBuffer(ADDR_STORAGE_LEN,0); staticbufs[1] = (u32)tmpaddr; ret = svcSendSyncRequest(SOCU_handle); diff --git a/libctru/source/services/soc/soc_getsockname.c b/libctru/source/services/soc/soc_getsockname.c index eef6bf4..87df82a 100644 --- a/libctru/source/services/soc/soc_getsockname.c +++ b/libctru/source/services/soc/soc_getsockname.c @@ -8,7 +8,7 @@ int getsockname(int sockfd, struct sockaddr *addr, socklen_t *addrlen) int ret = 0; u32 *cmdbuf = getThreadCommandBuffer(); u32 saved_threadstorage[2]; - u8 tmpaddr[0x1c]; + u8 tmpaddr[ADDR_STORAGE_LEN]; sockfd = soc_get_fd(sockfd); if(sockfd < 0) { @@ -18,14 +18,14 @@ int getsockname(int sockfd, struct sockaddr *addr, socklen_t *addrlen) cmdbuf[0] = IPC_MakeHeader(0x17,2,2); // 0x170082 cmdbuf[1] = (u32)sockfd; - cmdbuf[2] = 0x1c; + cmdbuf[2] = ADDR_STORAGE_LEN; cmdbuf[3] = IPC_Desc_CurProcessId(); u32 * staticbufs = getThreadStaticBuffers(); saved_threadstorage[0] = staticbufs[0]; saved_threadstorage[1] = staticbufs[1]; - staticbufs[0] = IPC_Desc_StaticBuffer(0x1c,0); + staticbufs[0] = IPC_Desc_StaticBuffer(ADDR_STORAGE_LEN,0); staticbufs[1] = (u32)tmpaddr; ret = svcSendSyncRequest(SOCU_handle); diff --git a/libctru/source/services/soc/soc_recvfrom.c b/libctru/source/services/soc/soc_recvfrom.c index 30ffd57..047a720 100644 --- a/libctru/source/services/soc/soc_recvfrom.c +++ b/libctru/source/services/soc/soc_recvfrom.c @@ -8,13 +8,13 @@ ssize_t socuipc_cmd7(int sockfd, void *buf, size_t len, int flags, struct sockad int ret = 0; u32 *cmdbuf = getThreadCommandBuffer(); u32 tmp_addrlen = 0; - u8 tmpaddr[0x1c]; + u8 tmpaddr[ADDR_STORAGE_LEN]; u32 saved_threadstorage[2]; - memset(tmpaddr, 0, 0x1c); + memset(tmpaddr, 0, ADDR_STORAGE_LEN); if(src_addr) - tmp_addrlen = 0x1c; + tmp_addrlen = ADDR_STORAGE_LEN; cmdbuf[0] = IPC_MakeHeader(0x7,4,4); // 0x70104 cmdbuf[1] = (u32)sockfd; @@ -66,13 +66,13 @@ ssize_t socuipc_cmd8(int sockfd, void *buf, size_t len, int flags, struct sockad int ret = 0; u32 *cmdbuf = getThreadCommandBuffer(); u32 tmp_addrlen = 0; - u8 tmpaddr[0x1c]; + u8 tmpaddr[ADDR_STORAGE_LEN]; u32 saved_threadstorage[4]; if(src_addr) - tmp_addrlen = 0x1c; + tmp_addrlen = ADDR_STORAGE_LEN; - memset(tmpaddr, 0, 0x1c); + memset(tmpaddr, 0, ADDR_STORAGE_LEN); cmdbuf[0] = 0x00080102; cmdbuf[1] = (u32)sockfd; diff --git a/libctru/source/services/soc/soc_sendto.c b/libctru/source/services/soc/soc_sendto.c index fe7b5ba..42fce5c 100644 --- a/libctru/source/services/soc/soc_sendto.c +++ b/libctru/source/services/soc/soc_sendto.c @@ -8,15 +8,15 @@ ssize_t socuipc_cmd9(int sockfd, const void *buf, size_t len, int flags, const s int ret = 0; u32 *cmdbuf = getThreadCommandBuffer(); u32 tmp_addrlen = 0; - u8 tmpaddr[0x1c]; + u8 tmpaddr[ADDR_STORAGE_LEN]; - memset(tmpaddr, 0, 0x1c); + memset(tmpaddr, 0, ADDR_STORAGE_LEN); if(dest_addr) { if(dest_addr->sa_family == AF_INET) tmp_addrlen = 8; else - tmp_addrlen = 0x1c; + tmp_addrlen = ADDR_STORAGE_LEN; if(addrlen < tmp_addrlen) { errno = EINVAL; @@ -62,15 +62,15 @@ ssize_t socuipc_cmda(int sockfd, const void *buf, size_t len, int flags, const s int ret = 0; u32 *cmdbuf = getThreadCommandBuffer(); u32 tmp_addrlen = 0; - u8 tmpaddr[0x1c]; + u8 tmpaddr[ADDR_STORAGE_LEN]; - memset(tmpaddr, 0, 0x1c); + memset(tmpaddr, 0, ADDR_STORAGE_LEN); if(dest_addr) { if(dest_addr->sa_family == AF_INET) tmp_addrlen = 8; else - tmp_addrlen = 0x1c; + tmp_addrlen = ADDR_STORAGE_LEN; if(addrlen < tmp_addrlen) { errno = EINVAL;