Fznamznon created this revision. Herald added subscribers: manas, ASDenysPetrov, dkrupp, donat.nagy, Szelethus, a.sidorin, baloghadamsoftware. Herald added a project: All. Fznamznon requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits.
TypeLocBuilder frees resources in the destructor but doesn't have user-written copy c'tor or assignment operator, so copying it using default ones can cause double free. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D156222 Files: clang/lib/Sema/TypeLocBuilder.h Index: clang/lib/Sema/TypeLocBuilder.h =================================================================== --- clang/lib/Sema/TypeLocBuilder.h +++ clang/lib/Sema/TypeLocBuilder.h @@ -53,6 +53,9 @@ delete[] Buffer; } + TypeLocBuilder(const TypeLocBuilder &) = delete; + TypeLocBuilder &operator=(const TypeLocBuilder &) = delete; + /// Ensures that this buffer has at least as much capacity as described. void reserve(size_t Requested) { if (Requested > Capacity)
Index: clang/lib/Sema/TypeLocBuilder.h =================================================================== --- clang/lib/Sema/TypeLocBuilder.h +++ clang/lib/Sema/TypeLocBuilder.h @@ -53,6 +53,9 @@ delete[] Buffer; } + TypeLocBuilder(const TypeLocBuilder &) = delete; + TypeLocBuilder &operator=(const TypeLocBuilder &) = delete; + /// Ensures that this buffer has at least as much capacity as described. void reserve(size_t Requested) { if (Requested > Capacity)
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits