zookeeper.c:prepend_string func can dereference null ptr
--------------------------------------------------------
Key: ZOOKEEPER-1305
URL: https://issues.apache.org/jira/browse/ZOOKEEPER-1305
Project: ZooKeeper
Issue Type: Bug
Components: c client
Affects Versions: 3.3.3
Environment: All
Reporter: Daniel Lescohier
All the callers of the function prepend_string make a call to prepend_string
before checking that zhandle_t *zh is not null. At the top of prepend_string,
zh is dereferenced without checking for a null ptr:
static char* prepend_string(zhandle_t *zh, const char* client_path) {
char *ret_str;
if (zh->chroot == NULL)
return (char *) client_path;
I propose fixing this by adding the check here in prepend_string:
static char* prepend_string(zhandle_t *zh, const char* client_path) {
char *ret_str;
if (zh==NULL || zh->chroot == NULL)
return (char *) client_path;
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira