r253418 - Try to fix leak in CXStringSet from r252853

Saleem Abdulrasool via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 18 08:48:49 PST 2015


On Tue, Nov 17, 2015 at 5:06 PM, Reid Kleckner via cfe-commits <
cfe-commits at lists.llvm.org> wrote:

> Author: rnk
> Date: Tue Nov 17 19:06:39 2015
> New Revision: 253418
>
> URL: http://llvm.org/viewvc/llvm-project?rev=253418&view=rev
> Log:
> Try to fix leak in CXStringSet from r252853
>
> Modified:
>     cfe/trunk/tools/libclang/CXString.cpp
>
> Modified: cfe/trunk/tools/libclang/CXString.cpp
> URL:
> http://llvm.org/viewvc/llvm-project/cfe/trunk/tools/libclang/CXString.cpp?rev=253418&r1=253417&r2=253418&view=diff
>
> ==============================================================================
> --- cfe/trunk/tools/libclang/CXString.cpp (original)
> +++ cfe/trunk/tools/libclang/CXString.cpp Tue Nov 17 19:06:39 2015
> @@ -186,6 +186,8 @@ void clang_disposeString(CXString string
>  }
>
>  void clang_disposeStringSet(CXStringSet *set) {
> +  for (unsigned SI = 0, SE = set->Count; SI < SE; ++SI)
> +    clang_disposeString(set->Strings[SI]);
>    delete[] set->Strings;
>    delete set;
>  }
>
>
Thanks for fixing this!


> _______________________________________________
> cfe-commits mailing list
> cfe-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
>



-- 
Saleem Abdulrasool
compnerd (at) compnerd (dot) org
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20151118/944f8b8c/attachment.html>


More information about the cfe-commits mailing list