Skip to content

fixes for crash on HdrHeap::attach_block#501

Closed
dmorilha wants to merge 1 commit intoapache:masterfrom
dmorilha:HdrHeap_attach_block
Closed

fixes for crash on HdrHeap::attach_block#501
dmorilha wants to merge 1 commit intoapache:masterfrom
dmorilha:HdrHeap_attach_block

Conversation

@dmorilha
Copy link

@SolidWallOfCode here is the proposal for our problems with HdrHeap::attach_block

@SolidWallOfCode
Copy link
Member

I'm going to -1 this patch because I think it treats the symptoms, not the problem. The latter seems to be TS-4256.

@dmorilha
Copy link
Author

dmorilha commented Mar 6, 2016

That's fine, thanks Alan for all investigation. I think we later identified a possible fix with the plug-in: #511

@dmorilha dmorilha closed this Mar 6, 2016
@dmorilha dmorilha deleted the HdrHeap_attach_block branch March 6, 2016 15:38
shinrich added a commit to shinrich/trafficserver that referenced this pull request Jul 10, 2017
Remove accidental file from rebase.
SolidWallOfCode added a commit to SolidWallOfCode/trafficserver that referenced this pull request Jul 23, 2021
* HostDB restructuring.

* Add method to write an IpAddr value to a sockaddr. (apache#7821)

(cherry picked from commit fbdbb5b)

* DNS: Clean up argument passing to DNS queries. (apache#7778)

(cherry picked from commit 6009f46)
(cherry picked from commit bc546e4)

* Fix: avoid reusing null hostent if the DNS query failed. (apache#485)

(cherry picked from commit 8b260f1)
(cherry picked from commit 85be058)

* Fix brokenness in Lua do to API change and reversion.

(cherry picked from commit 6266b4f)

* HostDB: Fix plugin API port error.

(cherry picked from commit 34518d1)

* Revert API change - maintain 9.x compatibility.

(cherry picked from commit 330a203)

Co-authored-by: Alan M. Carroll <amc@apache.org>
masaori335 pushed a commit to masaori335/trafficserver that referenced this pull request Jul 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants