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