Dutch T. Meyer created ZOOKEEPER-1836:
-----------------------------------------

             Summary: addrvec_next() fails to set next parameter if 
addrvec_hasnext() returns false
                 Key: ZOOKEEPER-1836
                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-1836
             Project: ZooKeeper
          Issue Type: Bug
          Components: c client
            Reporter: Dutch T. Meyer
            Priority: Trivial


There is a relatively innocuous but useless pointer assignment in
addrvec_next():

195     void addrvec_next(addrvec_t *avec, struct sockaddr_storage *next)
....
203         if (!addrvec_hasnext(avec))
204         {
205             next = NULL;
206             return;

That assignment on (205) has no point, as next is a local variable lost upon 
function return.  Likely this should be a memset to zero out the actual 
parameter.



--
This message was sent by Atlassian JIRA
(v6.1.4#6159)

Reply via email to