Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue #1367: HdrHeap potential corruption #1368

Merged
merged 1 commit into from
Jan 25, 2017

Conversation

shinrich
Copy link
Member

No description provided.

@atsci
Copy link

atsci commented Jan 24, 2017

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1375/ for details.

@atsci
Copy link

atsci commented Jan 24, 2017

Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/1268/ for details.

Copy link
Member

@maskit maskit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The change looks reasonable.

@shinrich shinrich merged commit fa12a49 into apache:master Jan 25, 2017
@zwoop
Copy link
Contributor

zwoop commented Jan 26, 2017

@shinrich Should we cherry-pick this back to 7.1.x ?

@zwoop zwoop added the Core label Jan 26, 2017
@zwoop zwoop added this to the 7.2.0 milestone Jan 26, 2017
@zwoop
Copy link
Contributor

zwoop commented Jan 26, 2017

@maskit @shinrich ping? Should we back port to 7.1.x ?

@maskit
Copy link
Member

maskit commented Jan 26, 2017

I'm +1 on backporting to 7.1.x.
I took a quick look and it seems to be easy, git apply fails though.

@zwoop zwoop modified the milestones: 7.1.0, 7.2.0 Jan 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants