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

Fix Assertion failure in the regex_revalidate plugin. #1406

Merged
merged 1 commit into from
Feb 9, 2017

Conversation

jrushford
Copy link
Contributor

When using the regex_revalidate plugin, we see an assertion failure in TSContSchedule() at InkAPI.cc:4232

Since TS-4387, Calls to TSContSchedule/TSContScheduleEvery(), require
that the continuation associated with the TSCont parameter must have a mutex.

@jrushford jrushford added Backport Marked for backport for an LTS patch release Plugins labels Feb 1, 2017
@jrushford jrushford self-assigned this Feb 1, 2017
@atsci
Copy link

atsci commented Feb 1, 2017

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

@atsci
Copy link

atsci commented Feb 1, 2017

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

@atsci
Copy link

atsci commented Feb 1, 2017

clang-analyzer build failed! See https://ci.trafficserver.apache.org/job/Github-Clang/14/ for details.

@jrushford jrushford added this to the 6.2.1 milestone Feb 1, 2017
@jrushford
Copy link
Contributor Author

[approve ci]

@atsci
Copy link

atsci commented Feb 1, 2017

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

@atsci
Copy link

atsci commented Feb 1, 2017

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

@atsci
Copy link

atsci commented Feb 1, 2017

clang-analyzer build failed! See https://ci.trafficserver.apache.org/job/Github-Clang/17/ for details.

@jrushford
Copy link
Contributor Author

[approve ci]

@atsci
Copy link

atsci commented Feb 2, 2017

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

@atsci
Copy link

atsci commented Feb 2, 2017

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

@atsci
Copy link

atsci commented Feb 2, 2017

clang-analyzer build failed! See https://ci.trafficserver.apache.org/job/Github-Clang/24/ for details.

@jrushford
Copy link
Contributor Author

[approve ci]

@atsci
Copy link

atsci commented Feb 6, 2017

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/freebsd-github/1476/ for details.

@atsci
Copy link

atsci commented Feb 6, 2017

Linux build successful! See https://ci.trafficserver.apache.org/job/linux-github/1369/ for details.

@atsci
Copy link

atsci commented Feb 6, 2017

clang-analyzer build failed! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/40/ for details.

@PSUdaemon
Copy link
Contributor

[approve ci]

@atsci
Copy link

atsci commented Feb 9, 2017

Linux build successful! See https://ci.trafficserver.apache.org/job/linux-github/1390/ for details.

@atsci
Copy link

atsci commented Feb 9, 2017

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/freebsd-github/1497/ for details.

@atsci
Copy link

atsci commented Feb 9, 2017

clang-analyzer build failed! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/61/ for details.

@PSUdaemon
Copy link
Contributor

@jrushford Can you rebase this to 6.2.x?

Since TS-4387, Calls to TSContSchedule/TSContScheduleEvery(), require
that the continuation associated with the TSCont parameter must have a mutex.
@atsci
Copy link

atsci commented Feb 9, 2017

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/freebsd-github/1503/ for details.

@atsci
Copy link

atsci commented Feb 9, 2017

Linux build successful! See https://ci.trafficserver.apache.org/job/linux-github/1396/ for details.

@atsci
Copy link

atsci commented Feb 9, 2017

clang-analyzer build successful! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/67/ for details.

@PSUdaemon PSUdaemon merged commit 0b1f28b into apache:6.2.x Feb 9, 2017
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 Plugins
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants