kezhuw commented on PR #2209: URL: https://github.com/apache/zookeeper/pull/2209#issuecomment-2484562704
> The problem of current aproach is that it will corrupt program only after zookeeper_close with cert str ",,,/tmp/certs/server.crt,/tmp/certs/client.crt,/tmp/certs/clientkey.pem,password"(be aware of the leading ",") as ca will point to address not from malloc. My best wish is crash. -- https://github.com/apache/zookeeper/pull/2209#issuecomment-2466111427 Could you test it with cert str ",,,/tmp/certs/server.crt,/tmp/certs/client.crt,/tmp/certs/clientkey.pem,password" ? I tested with an online repl, it crashed. Though I think crash is ok sometimes, but I don't think this is the case: 1. The crash is far from the cause. 2. There is no guarantee that `free` will crash in this case. -- 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
