[GitHub] [trafficserver] maskit commented on pull request #7883: Origin session cache mem leak fix

[GitBox <gi...@apache.org>]

maskit commented on pull request #7883:
URL: https://github.com/apache/trafficserver/pull/7883#issuecomment-868796977


   My understanding of `i2d_SSL_SESSION` is just a serialization mechanism. So I think it should be possible to use i2d_SSL_SESSION without memory leak unless we are missing something. Is this correct?
   
   Removing the serialization part is fine if it's unnecessary, but is it only possible for origin sessions?
   
   > As for the exdata, as we only use it to store a single integer value, I see no reason to pass it as a pointer to a structure.
   
   Is this only true for origin sessions and we still need to pass it as a pointer to a structure for server sessions?


-- 
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: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org