Skip to content

TSStringPercentDecode null-termination clipping on overflow. #1376

Merged
shinrich merged 1 commit intoapache:masterfrom
shinrich:tsstringpercentdecode_overflow
Jan 26, 2017
Merged

TSStringPercentDecode null-termination clipping on overflow. #1376
shinrich merged 1 commit intoapache:masterfrom
shinrich:tsstringpercentdecode_overflow

Conversation

@shinrich
Copy link
Copy Markdown
Member

Actually fixed by @petar last summer, but forgot to get it pushed to up.

Long URL's would cause buffer overflow write.

@atsci
Copy link
Copy Markdown

atsci commented Jan 25, 2017

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

@atsci
Copy link
Copy Markdown

atsci commented Jan 25, 2017

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

Copy link
Copy Markdown
Contributor

@zwoop zwoop left a comment

Choose a reason for hiding this comment

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

+1. I think this should go to 7.1.0

@zwoop zwoop added this to the 7.1.0 milestone Jan 26, 2017
@zwoop zwoop added TS API Backport Marked for backport for an LTS patch release labels Jan 26, 2017
@shinrich shinrich merged commit 8e8a4b4 into apache:master Jan 26, 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 TS API

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants