Gowrima commented on PR #2209: URL: https://github.com/apache/zookeeper/pull/2209#issuecomment-2460700581
@kezhuw Isn't zcert.certstr supposed to hold the entire certificate and zcert.ca, zcert.key, zcert.password hold the appropriate tokens? With the suggested change, zcert.certstr will no longer hold the whole certificate, but only the first token (both zcert.certstr and zcert.ca will point to the first token - duplicates). zcert.ca = strtok(strdup(cert), ","); -- this will point to the first token (i.e., 'ca') in the allocated memory. zcert.certstr = strdup(cert) - this will hold the entire certificate as a string. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@zookeeper.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
