You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@trafficserver.apache.org by GitBox <gi...@apache.org> on 2020/04/24 22:22:14 UTC

[GitHub] [trafficserver] shinrich opened a new pull request #6709: Fix use-after free

shinrich opened a new pull request #6709:
URL: https://github.com/apache/trafficserver/pull/6709


   @bneradt found this use-after free when running prod sim.
   
   The problem was introduced with https://github.com/apache/trafficserver/pull/6469
   
   The do_io_close may close the session object.


----------------------------------------------------------------
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.

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



[GitHub] [trafficserver] shinrich commented on pull request #6709: Fix use-after free

Posted by GitBox <gi...@apache.org>.
shinrich commented on pull request #6709:
URL: https://github.com/apache/trafficserver/pull/6709#issuecomment-620070539


   Seems likely that this is the same as PR #6574 


----------------------------------------------------------------
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.

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



[GitHub] [trafficserver] shinrich commented on pull request #6709: Fix use-after free

Posted by GitBox <gi...@apache.org>.
shinrich commented on pull request #6709:
URL: https://github.com/apache/trafficserver/pull/6709#issuecomment-620067671


   I'm reconsidering the original fix to  #6469.  Will do some more experiments locally.


----------------------------------------------------------------
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.

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



[GitHub] [trafficserver] randall commented on pull request #6709: Fix use-after free

Posted by GitBox <gi...@apache.org>.
randall commented on pull request #6709:
URL: https://github.com/apache/trafficserver/pull/6709#issuecomment-619283756


   [approve ci autest]


----------------------------------------------------------------
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.

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



[GitHub] [trafficserver] zwoop commented on pull request #6709: Fix use-after free

Posted by GitBox <gi...@apache.org>.
zwoop commented on pull request #6709:
URL: https://github.com/apache/trafficserver/pull/6709#issuecomment-623849011


   Please remember to remove Milestone and Projects from PRs that are closed without merging.


----------------------------------------------------------------
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.

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



[GitHub] [trafficserver] shinrich commented on pull request #6709: Fix use-after free

Posted by GitBox <gi...@apache.org>.
shinrich commented on pull request #6709:
URL: https://github.com/apache/trafficserver/pull/6709#issuecomment-622411847


   There are better ways to do this.  Will put up new PR soon.


----------------------------------------------------------------
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.

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