From 78399e2a665fec41edcdd91fa85b372fbaf1d61f Mon Sep 17 00:00:00 2001 From: Masayuki Ishikawa Date: Fri, 21 Feb 2020 15:37:25 +0900 Subject: [PATCH] net: socket: Enable parameter check (addr and addrlen) in getsockname.c Signed-off-by: Masayuki Ishikawa --- net/socket/getsockname.c | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/net/socket/getsockname.c b/net/socket/getsockname.c index bed6217bccb49..ea314a1a4fc09 100644 --- a/net/socket/getsockname.c +++ b/net/socket/getsockname.c @@ -101,16 +101,12 @@ int psock_getsockname(FAR struct socket *psock, FAR struct sockaddr *addr, return -EBADF; } - /* Some sanity checking... Shouldn't need this on a buckled up embedded - * system (?) - */ + /* Some sanity checking... */ -#ifdef CONFIG_DEBUG_FEATURES - if (addr == NULL || *addrlen <= 0) + if (addr == NULL || addrlen == NULL) { return -EINVAL; } -#endif /* Let the address family's send() method handle the operation */