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

        

Reply via email to