diff options
| author | Shwetha Acharya <sacharya@redhat.com> | 2018-10-23 12:57:45 +0530 |
|---|---|---|
| committer | Amar Tumballi <amarts@redhat.com> | 2018-10-30 05:08:04 +0000 |
| commit | 0c835893fdce59872103253985d152f4592eea89 (patch) | |
| tree | 2f5faa10b212644ccde3ee0f840811d0d8eb5d17 /rpc/rpc-transport/socket/src/name.c | |
| parent | d7fa1a85b1d5213128ec1f58d1e22af2495b9a3e (diff) | |
rpc-transport/socket: NULL pointer dereferencing clang fix
Problem: ctx and res can be NULL.
Solution: introduced a VALIDATE_OR_GOTO statement, hence removed
the null check for ctx; added a check for res.
Updates: bz#1622665
Change-Id: Ifee4c73e260530ab44c0a34c5ff5568f38f92c94
Signed-off-by: Shwetha Acharya <sacharya@redhat.com>
Diffstat (limited to 'rpc/rpc-transport/socket/src/name.c')
| -rw-r--r-- | rpc/rpc-transport/socket/src/name.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/rpc/rpc-transport/socket/src/name.c b/rpc/rpc-transport/socket/src/name.c index 39c0530b08e..3d3d108f135 100644 --- a/rpc/rpc-transport/socket/src/name.c +++ b/rpc/rpc-transport/socket/src/name.c @@ -413,7 +413,7 @@ af_inet_server_get_local_sockaddr(rpc_transport_t *this, struct sockaddr *addr, } } - if (!(*addr_len)) { + if (!(*addr_len) && res) { memcpy(addr, res->ai_addr, res->ai_addrlen); *addr_len = res->ai_addrlen; } |