Bugzilla – Bug 5816
type frees wrong memory
Last modified: 2008-08-11 15:04:03
You need to log in before you can comment on or make changes to this bug.
In the function globus_gsi_proxy_handle_set_req, the second if block checks to see if the handle already contains a request. In this case, you probably want to free the old request rather than the new one. That is, X509_REQ_free(req); should be: X509_REQ_free(handle->req); This would match the behavior in globus_gsi_proxy_handle_set_private_key Full disclosure: I just noticed this reading the code; I haven't ever tried to call this function twice or observed any code that does so and has problems. Thanks, -Ian
Thanks for the bug report. I don't think anything uses that code path, but it's clearly wrong. The problem is patched in the 4.0 branch and CVS trunk now.