[incubator-nuttx] branch master updated: net: socket: Enable parameter check (addr and addrlen) in getsockname.c

[xi...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

xiaoxiang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git


The following commit(s) were added to refs/heads/master by this push:
     new e9fe1f8  net: socket: Enable parameter check (addr and addrlen) in getsockname.c
e9fe1f8 is described below

commit e9fe1f84af469568868c6e403b526b76e993f70f
Author: Masayuki Ishikawa <ma...@gmail.com>
AuthorDate: Fri Feb 21 15:37:25 2020 +0900

    net: socket: Enable parameter check (addr and addrlen) in getsockname.c
    
    Signed-off-by: Masayuki Ishikawa <Ma...@jp.sony.com>
---
 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 bed6217..ea314a1 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 */