[PATCH] D27729: [libFuzzer] Diff 20 - Update tests to use more general functions instead of posix specific.
Marcos Pividori via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Dec 13 14:44:12 PST 2016
mpividori added inline comments.
================
Comment at: lib/Fuzzer/FuzzerUtilWindows.cpp:181-182
+const char *StrStr(const char* Ref, const char* Patt) {
+ return (const char*) fuzzer::SearchMemory(Ref, ::strlen(Ref), Patt,
+ ::strlen(Patt));
+}
----------------
zturner wrote:
> Windows has `strstr`, it just doesn't have `strcasestr`. I would implement this one using just `strstr`.
@zturner , You are right. I confused because I found the function `StrStr()` from Shlwapi.lib . I didn't realized that `strstr()` is part of the std library.
I think it is better to continue with actual implementation of `StrCaseStr()`. By using `std::search()`, we don't have no copy the arrays.
If we want to use `strstr()`, we should copy both arrays, then apply `tolower()`, and then use `strstr()`. This can be slower with large arrays.
Repository:
rL LLVM
https://reviews.llvm.org/D27729
More information about the llvm-commits
mailing list