2011/11/12 Ted Kremenek <[email protected]>:
> I think we should instead fix CXLoadedDiagnosticSetImpl::makeString() to add 
> a null terminator.  Your patch actually introduces a memory leak since we 
> don't ever explicitly release these CXStrings (and expect them to get 
> released when the CXLoadedDiagnosticSetImpl object is destroyed).

Ted, I see. Thanks to point me out.

Explanation: I took simpler patch. I should've expanded makeString() ;)

...Takumi

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to