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 2021/07/21 22:24:50 UTC

[GitHub] [trafficserver] ywkaras commented on pull request #8106: Make the rest of InkAPI allocators Proxy Allocated

ywkaras commented on pull request #8106:
URL: https://github.com/apache/trafficserver/pull/8106#issuecomment-884538017


   The test_hooks Au test is causing a segment violation in TSContDestroy():  https://ci.trafficserver.apache.org/autest/ausb-8106.16267/test_hooks/_output/test_hooks-ts/stream.stderr.txt
   
   I think what's happening is that INKContAllocator or it's proxy allocator is getting destroyed before `cleanup`:
   https://github.com/apache/trafficserver/blob/c214c9f40096afbe580c0ee90647933ad8b21bcd/tests/tools/plugins/test_hooks.cc#L371
   
   I think you should just delete the lines https://github.com/apache/trafficserver/blob/c214c9f40096afbe580c0ee90647933ad8b21bcd/tests/tools/plugins/test_hooks.cc#L348-L372 .
   
   There are tricky ways to control the order of destruction at program exit, but I don't think it's worth it.  The only problem is that it will probably look like a memory leak to ASAN.


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