diff options
author | Yaniv Kaul <ykaul@redhat.com> | 2020-01-14 16:55:32 +0200 |
---|---|---|
committer | gluster-ant <bugzilla-bot@gluster.org> | 2020-01-14 16:55:32 +0200 |
commit | 11c3113db4600bcafadcd58e6529f293ca355418 (patch) | |
tree | 7716fd5f0764abc45e7e23faa999b86551be3452 /rpc | |
parent | e651a2a624c09bc6d793a24bf218f494cc82e2e7 (diff) | |
download | glusterfs-11c3113db4600bcafadcd58e6529f293ca355418.tar.gz glusterfs-11c3113db4600bcafadcd58e6529f293ca355418.tar.xz glusterfs-11c3113db4600bcafadcd58e6529f293ca355418.zip |
name.c: fix Coverity issues 1412332/3 - strcat into uninitialized value
Check limit to 108 bytes before strcpy().
fixes: CID#1412332
updates: bz#1193929
Signed-off-by: Yaniv Kaul <ykaul@redhat.com>
Change-Id: I8b26b1e1d2daca98ff36db531539bec0a405769c
Diffstat (limited to 'rpc')
-rw-r--r-- | rpc/rpc-transport/socket/src/name.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/rpc/rpc-transport/socket/src/name.c b/rpc/rpc-transport/socket/src/name.c index e374de6021..9286bbb236 100644 --- a/rpc/rpc-transport/socket/src/name.c +++ b/rpc/rpc-transport/socket/src/name.c @@ -108,7 +108,7 @@ af_unix_client_bind(rpc_transport_t *this, struct sockaddr *sockaddr, path_data = dict_get_sizen(this->options, "transport.socket.bind-path"); if (path_data) { char *path = data_to_str(path_data); - if (!path || path_data->len > UNIX_PATH_MAX) { + if (!path || path_data->len > 108) { /* 108 = addr->sun_path length */ gf_log(this->name, GF_LOG_TRACE, "bind-path not specified for unix socket, " "letting connect to assign default value"); @@ -292,7 +292,8 @@ af_unix_client_get_remote_sockaddr(rpc_transport_t *this, goto err; } - if ((connect_path_data->len + 1) > UNIX_PATH_MAX) { + /* 108 = sockaddr_un->sun_path length */ + if ((connect_path_data->len + 1) > 108) { gf_log(this->name, GF_LOG_ERROR, "connect-path value length %d > %d octets", connect_path_data->len + 1, UNIX_PATH_MAX); |