[PATCH] D45050: [clang-tidy] New checker for not null-terminated result caused by strlen or wcslen

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Jun 2 09:15:09 PDT 2018


lebedev.ri added a comment.

In https://reviews.llvm.org/D45050#1119973, @Charusso wrote:

> In https://reviews.llvm.org/D45050#1116446, @lebedev.ri wrote:
>
> > I would like to see more negative tests.
> >  What does it do if the len/size is a constant?
> >  Variable that wasn't just assigned with `strlen()` return?
>
>
> Thanks for the comment! What do you mean by negative test?


The tests where the check matches and issues a warning is a positive test.
The tests where the check does not match and/or does not issue a warning is a negative test.


https://reviews.llvm.org/D45050





More information about the cfe-commits mailing list