Skip to content

TS-4697: Free MIOBuffer if the ipallow check fails. (backport for 6.2.x)#1200

Merged
PSUdaemon merged 1 commit intoapache:6.2.xfrom
oknet:TS-4697-6.2.x
Nov 7, 2016
Merged

TS-4697: Free MIOBuffer if the ipallow check fails. (backport for 6.2.x)#1200
PSUdaemon merged 1 commit intoapache:6.2.xfrom
oknet:TS-4697-6.2.x

Conversation

@oknet
Copy link
Copy Markdown
Member

@oknet oknet commented Nov 5, 2016

(cherry picked from commit 40b7b48)

@atsci
Copy link
Copy Markdown

atsci commented Nov 5, 2016

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

@atsci
Copy link
Copy Markdown

atsci commented Nov 5, 2016

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

@PSUdaemon
Copy link
Copy Markdown
Contributor

When you do the backports can you leave the commit messages the same except for the cherry pick comment and the conflicts?

@PSUdaemon PSUdaemon added Core Backport Marked for backport for an LTS patch release labels Nov 6, 2016
@PSUdaemon PSUdaemon added this to the 6.2.1 milestone Nov 6, 2016
@oknet
Copy link
Copy Markdown
Member Author

oknet commented Nov 7, 2016

Do you means “This closes #823.“ ?

2016-11-06 12:27 GMT+08:00 Phil Sorber notifications@github.com:

When you do the backports can you leave the commit messages the same
except for the cherry pick comment and the conflicts?


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#1200 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE_9sVkUa_ydpksX6BRd1GonVrRRBZDkks5q7VdPgaJpZM4KqMNo
.

@PSUdaemon
Copy link
Copy Markdown
Contributor

I mean don't add (backport for 6.2.x) to the existing commit message. It's superfluous.

@oknet
Copy link
Copy Markdown
Member Author

oknet commented Nov 7, 2016

@PSUdaemon I'm just leave the same commit message and the change log only for 6.2.x branch. please check the new commit message, please leave you advice, thanks.

@atsci
Copy link
Copy Markdown

atsci commented Nov 7, 2016

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

@atsci
Copy link
Copy Markdown

atsci commented Nov 7, 2016

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

(cherry picked from commit 40b7b48)

With minor revision of SpdySessionAccept.
@atsci
Copy link
Copy Markdown

atsci commented Nov 7, 2016

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

@atsci
Copy link
Copy Markdown

atsci commented Nov 7, 2016

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

@PSUdaemon PSUdaemon merged commit f82a44c into apache:6.2.x Nov 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Backport Marked for backport for an LTS patch release Core

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants