libcxxrt: Correct off-by-one in free_exception of emergency buffer
ClosedPublic
Actions

Authored by cem on Sep 25 2015, 8:38 PM.

Details

Reviewers

dim
emaste
markj
theraven

Commits

rS288238: MFV c3ccd112: Correct off-by-ones in free_exception of emergency buffer

Test Plan

Note, this has been broken since import in r227825.

https://github.com/pathscale/libcxxrt/issues/29

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

No Lint Coverage

Unit

No Test Coverage

Build Status

Buildable 608
Build 608: arc lint + arc unit

Event Timeline

cem updated this revision to Diff 8936.Sep 25 2015, 8:38 PM

cem retitled this revision from to libcxxrt: Correct off-by-one in free_exception of emergency buffer.

cem updated this object.

cem edited the test plan for this revision. (Show Details)

cem added reviewers: theraven, dim, markj.

Herald added a subscriber: imp. · View Herald TranscriptSep 25 2015, 8:38 PM

cem edited the test plan for this revision. (Show Details)Sep 25 2015, 8:41 PM

emaste accepted this revision.Sep 25 2015, 8:46 PM

emaste added a reviewer: emaste.

This revision is now accepted and ready to land.Sep 25 2015, 8:46 PM

Per kan, also need the same fix in emergency_malloc_free.

https://github.com/pathscale/libcxxrt/pull/31

This revision now requires review to proceed.Sep 25 2015, 8:57 PM

Closed by commit rS288238: MFV c3ccd112: Correct off-by-ones in free_exception of emergency buffer (authored by cem). · Explain WhySep 25 2015, 10:29 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

contrib/

libcxxrt/

exception.cc

4 lines

Diff 8937

View Options

contrib/libcxxrt/exception.cc

libcxxrt: Correct off-by-one in free_exception of emergency bufferClosedPublicActions