diff options
| author | Sheetal Pamecha <sheetal.pamecha08@gmail.com> | 2018-11-30 22:01:53 +0530 |
|---|---|---|
| committer | Amar Tumballi <amarts@redhat.com> | 2018-12-05 09:14:27 +0000 |
| commit | 65dc176e7c3e3d598605b75f06b796fd0ade2c7e (patch) | |
| tree | 55c45a9f0ec70275bc081a1db67786fdf3e1241d /rpc/rpc-transport/socket | |
| parent | 785106e13864b84f35ca3394518d05d69a4e755f (diff) | |
rpc-transport/socket: NULL pointer dereferencing clang fix
Problem: res->ai_addr could be NULL
Added a check to address this issue
Change-Id: Iac88a8d6dc1f009836554448afbc228df93decd6
Updates: bz#1622665
Signed-off-by: Sheetal Pamecha <sheetal.pamecha08@gmail.com>
Diffstat (limited to 'rpc/rpc-transport/socket')
| -rw-r--r-- | rpc/rpc-transport/socket/src/name.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/rpc/rpc-transport/socket/src/name.c b/rpc/rpc-transport/socket/src/name.c index 3d3d108f135..1f866ccdecf 100644 --- a/rpc/rpc-transport/socket/src/name.c +++ b/rpc/rpc-transport/socket/src/name.c @@ -413,9 +413,11 @@ af_inet_server_get_local_sockaddr(rpc_transport_t *this, struct sockaddr *addr, } } - if (!(*addr_len) && res) { + if (!(*addr_len) && res && res->ai_addr) { memcpy(addr, res->ai_addr, res->ai_addrlen); *addr_len = res->ai_addrlen; + } else { + ret = -1; } freeaddrinfo(res); |