[cfe-commits] r166188 - /cfe/trunk/lib/Sema/SemaInit.cpp
dgregor at apple.com
Thu Oct 18 13:50:59 PDT 2012
On Oct 18, 2012, at 1:48 PM, David Blaikie <dblaikie at gmail.com> wrote:
> On Thu, Oct 18, 2012 at 1:45 PM, Douglas Gregor <dgregor at apple.com> wrote:
>> On Oct 18, 2012, at 9:57 AM, David Blaikie <dblaikie at gmail.com> wrote:
>>> Author: dblaikie
>>> Date: Thu Oct 18 11:57:32 2012
>>> New Revision: 166188
>>> URL: http://llvm.org/viewvc/llvm-project?rev=166188&view=rev
>>> PR14021: Copy lookup results to ensure safe iteration.
>>> Within the body of the loop the underlying map may be modified via
>>> -> Sema::CompareReferenceRelationship
>>> -> Sema::RequireCompleteType
>>> to avoid the use of invalid iterators the sequence is copied first.
>> Did you audit other uses of LookupConstructors to ensure that this is the only ticking time bomb in this area?
> [+Robert Muth]
> I've not performed any such audit, no. With the hack debug check
> inserted we could just run the whole test suite to see if anything
> pops up.
That seems unlikely to help; this is only a problem when the underlying StoredDeclsList has to reallocate.
More information about the cfe-commits