Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Fuzz] Add Fuzz testing for RegistryPreview #37607

Open
wants to merge 18 commits into
base: main
Choose a base branch
from
Open

[Fuzz] Add Fuzz testing for RegistryPreview #37607

Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
b205794
Add fuzz test cases
chenmy77 Jan 28, 2025
8aefd45
Merge remote-tracking branch 'origin/main' into dev/mengyuanchen/add_…
chenmy77 Feb 10, 2025
88f5e65
add fuzz tests framework in registrypreview
chenmy77 Feb 11, 2025
5e28035
update code
chenmy77 Feb 21, 2025
330e292
add registrypreview fuzzing code
chenmy77 Feb 24, 2025
6ee7dba
merge main into code
chenmy77 Feb 24, 2025
d1b4fee
add annotations and change net7.0--net8.0
chenmy77 Feb 24, 2025
63724c8
Merge remote-tracking branch 'origin/main' into dev/mengyuanchen/add_…
chenmy77 Feb 25, 2025
a6a9f92
merge main into code
chenmy77 Feb 25, 2025
16c2c2b
add registry fuzz sln
chenmy77 Feb 25, 2025
f0951bb
change fuzzing tests scope
chenmy77 Feb 25, 2025
11471bd
remove unuse annotations
chenmy77 Feb 25, 2025
8ca1c2f
fix typos
chenmy77 Feb 25, 2025
a7b8a27
change public parser to internel and private
chenmy77 Feb 26, 2025
ad15dd1
change public to internal
chenmy77 Feb 28, 2025
136b7ca
Merge remote-tracking branch 'origin/main' into dev/mengyuanchen/add_…
chenmy77 Mar 3, 2025
e8269b8
fix linelower error and modify filenametext to registryContent
chenmy77 Mar 3, 2025
0438b20
Revert "fix linelower error and modify filenametext to registryContent"
chenmy77 Mar 3, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix linelower error and modify filenametext to registryContent
  • Loading branch information
@chenmy77
chenmy77 committed Mar 3, 2025
commit e8269b8af275dc0487cb6e1f630b45e6a2086727
20 changes: 11 additions & 9 deletions PowerToys.sln
View file
Original file line number Diff line number Diff line change
@@ -646,6 +646,8 @@ Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Hosts.FuzzTests", "src\modu
EndProject
Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Hosts.UITests", "src\modules\Hosts\Hosts.UITests\Hosts.UITests.csproj", "{4E0AE3A4-2EE0-44D7-A2D0-8769977254A0}"
EndProject
Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "RegistryPreview.FuzzTests", "src\modules\registrypreview\RegistryPreview.FuzzTests\RegistryPreview.FuzzTests.csproj", "{F99690AC-D81C-4371-81F2-E2E1A62A28F1}"
EndProject
Global
GlobalSection(SolutionConfigurationPlatforms) = preSolution
Debug|ARM64 = Debug|ARM64
@@ -2294,14 +2296,14 @@ Global
{4E0AE3A4-2EE0-44D7-A2D0-8769977254A0}.Release|ARM64.Build.0 = Release|ARM64
{4E0AE3A4-2EE0-44D7-A2D0-8769977254A0}.Release|x64.ActiveCfg = Release|x64
{4E0AE3A4-2EE0-44D7-A2D0-8769977254A0}.Release|x64.Build.0 = Release|x64
{50A9F3DE-CF0B-4CF0-AFDE-3A3E245D7734}.Debug|ARM64.ActiveCfg = Debug|ARM64
{50A9F3DE-CF0B-4CF0-AFDE-3A3E245D7734}.Debug|ARM64.Build.0 = Debug|ARM64
{50A9F3DE-CF0B-4CF0-AFDE-3A3E245D7734}.Debug|x64.ActiveCfg = Debug|x64
{50A9F3DE-CF0B-4CF0-AFDE-3A3E245D7734}.Debug|x64.Build.0 = Debug|x64
{50A9F3DE-CF0B-4CF0-AFDE-3A3E245D7734}.Release|ARM64.ActiveCfg = Release|ARM64
{50A9F3DE-CF0B-4CF0-AFDE-3A3E245D7734}.Release|ARM64.Build.0 = Release|ARM64
{50A9F3DE-CF0B-4CF0-AFDE-3A3E245D7734}.Release|x64.ActiveCfg = Release|x64
{50A9F3DE-CF0B-4CF0-AFDE-3A3E245D7734}.Release|x64.Build.0 = Release|x64
{F99690AC-D81C-4371-81F2-E2E1A62A28F1}.Debug|ARM64.ActiveCfg = Debug|ARM64
{F99690AC-D81C-4371-81F2-E2E1A62A28F1}.Debug|ARM64.Build.0 = Debug|ARM64
{F99690AC-D81C-4371-81F2-E2E1A62A28F1}.Debug|x64.ActiveCfg = Debug|x64
{F99690AC-D81C-4371-81F2-E2E1A62A28F1}.Debug|x64.Build.0 = Debug|x64
{F99690AC-D81C-4371-81F2-E2E1A62A28F1}.Release|ARM64.ActiveCfg = Release|ARM64
{F99690AC-D81C-4371-81F2-E2E1A62A28F1}.Release|ARM64.Build.0 = Release|ARM64
{F99690AC-D81C-4371-81F2-E2E1A62A28F1}.Release|x64.ActiveCfg = Release|x64
{F99690AC-D81C-4371-81F2-E2E1A62A28F1}.Release|x64.Build.0 = Release|x64
EndGlobalSection
GlobalSection(SolutionProperties) = preSolution
HideSolutionNode = FALSE
@@ -2542,7 +2544,7 @@ Global
{4382A954-179A-4078-92AF-715187DFFF50} = {38BDB927-829B-4C65-9CD9-93FB05D66D65}
{EBED240C-8702-452D-B764-6DB9DA9179AF} = {F05E590D-AD46-42BE-9C25-6A63ADD2E3EA}
{4E0AE3A4-2EE0-44D7-A2D0-8769977254A0} = {F05E590D-AD46-42BE-9C25-6A63ADD2E3EA}
{50A9F3DE-CF0B-4CF0-AFDE-3A3E245D7734} = {929C1324-22E8-4412-A9A8-80E85F3985A5}
{F99690AC-D81C-4371-81F2-E2E1A62A28F1} = {929C1324-22E8-4412-A9A8-80E85F3985A5}
EndGlobalSection
GlobalSection(ExtensibilityGlobals) = postSolution
SolutionGuid = {C3A2F9D1-7930-4EF4-A6FC-7EE0A99821D0}
1 change: 1 addition & 0 deletions deps/cxxopts
Submodule cxxopts added at 12e496
2 changes: 1 addition & 1 deletion deps/cziplib
Submodule cziplib updated 18 files
+0 −1 .gitattributes
+19 −36 .github/workflows/build.yml
+0 −2 .gitignore
+23 −43 CMakeLists.txt
+0 −1 CONTRIBUTING.md
+5 −61 README.md
+4,302 −7,546 src/miniz.h
+1,062 −1,374 src/zip.c
+65 −182 src/zip.h
+7 −36 test/CMakeLists.txt
+0 −370 test/minunit.h
+671 −0 test/test.c
+0 −96 test/test_append.c
+0 −297 test/test_entry.c
+0 −153 test/test_extract.c
+0 −183 test/test_permissions.c
+0 −140 test/test_read.c
+0 −86 test/test_write.c
2 changes: 1 addition & 1 deletion deps/expected-lite
Submodule expected-lite updated 15 files
+0 −97 .github/workflows/ci.yml
+1 −7 CMakeLists.txt
+30 −32 README.md
+1 −1 conanfile.py
+3 −12 example/CMakeLists.txt
+20 −23 include/nonstd/expected.hpp
+0 −1 test/CMakeLists.txt
+0 −6 test/expected-noexcept.t.cpp
+0 −42 test/expected.t.cpp
+0 −0 test/lest.hpp
+1 −2 test/t-noexcept.bat
+1 −1 test/t.bat
+0 −60 test/tc-cl.bat
+1 −1 test/tc.bat
+1 −1 test/tg.bat
2 changes: 1 addition & 1 deletion deps/spdlog
Submodule spdlog updated 74 files
+1 −1 .clang-tidy
+4 −47 CMakeLists.txt
+2 −6 README.md
+7 −27 appveyor.yml
+1 −18 bench/CMakeLists.txt
+0 −5 bench/async_bench.cpp
+0 −5 bench/bench.cpp
+5 −5 bench/formatter-bench.cpp
+0 −14 cmake/spdlogCPack.cmake
+0 −1 cmake/utils.cmake
+2 −2 example/example.cpp
+0 −1 include/spdlog/cfg/helpers-inl.h
+10 −12 include/spdlog/common-inl.h
+7 −40 include/spdlog/common.h
+1 −2 include/spdlog/details/backtracer.h
+2 −17 include/spdlog/details/file_helper-inl.h
+1 −3 include/spdlog/details/log_msg_buffer-inl.h
+0 −6 include/spdlog/details/mpmc_blocking_q.h
+22 −67 include/spdlog/details/os-inl.h
+2 −9 include/spdlog/details/os.h
+1 −1 include/spdlog/details/registry.h
+1 −2 include/spdlog/details/synchronous_factory.h
+1 −1 include/spdlog/details/tcp_client-windows.h
+2 −2 include/spdlog/details/tcp_client.h
+1 −6 include/spdlog/details/thread_pool-inl.h
+0 −1 include/spdlog/details/thread_pool.h
+42 −37 include/spdlog/fmt/bundled/chrono.h
+10 −47 include/spdlog/fmt/bundled/color.h
+23 −59 include/spdlog/fmt/bundled/compile.h
+104 −344 include/spdlog/fmt/bundled/core.h
+273 −1,621 include/spdlog/fmt/bundled/format-inl.h
+463 −694 include/spdlog/fmt/bundled/format.h
+29 −15 include/spdlog/fmt/bundled/locale.h
+37 −67 include/spdlog/fmt/bundled/os.h
+9 −19 include/spdlog/fmt/bundled/ostream.h
+2 −2 include/spdlog/fmt/bundled/printf.h
+6 −16 include/spdlog/fmt/bundled/ranges.h
+1 −2 include/spdlog/fmt/fmt.h
+23 −24 include/spdlog/logger.h
+4 −26 include/spdlog/pattern_formatter-inl.h
+2 −2 include/spdlog/pattern_formatter.h
+0 −2 include/spdlog/sinks/ansicolor_sink-inl.h
+1 −1 include/spdlog/sinks/base_sink.h
+1 −2 include/spdlog/sinks/basic_file_sink.h
+0 −34 include/spdlog/sinks/daily_file_sink.h
+1 −1 include/spdlog/sinks/dup_filter_sink.h
+0 −194 include/spdlog/sinks/hourly_file_sink.h
+1 −2 include/spdlog/sinks/stdout_color_sinks-inl.h
+3 −19 include/spdlog/sinks/stdout_sinks-inl.h
+5 −15 include/spdlog/sinks/win_eventlog_sink.h
+54 −59 include/spdlog/sinks/wincolor_sink-inl.h
+19 −10 include/spdlog/sinks/wincolor_sink.h
+17 −17 include/spdlog/spdlog.h
+1 −51 include/spdlog/tweakme.h
+1 −1 include/spdlog/version.h
+8 −8 scripts/extract_version.py
+1 −2 src/async.cpp
+1 −2 src/cfg.cpp
+0 −1 src/color_sinks.cpp
+1 −2 src/file_sinks.cpp
+21 −61 src/fmt.cpp
+1 −2 src/spdlog.cpp
+1 −2 src/stdout_sinks.cpp
+6 −8 tests/test_async.cpp
+41 −35 tests/test_create_dir.cpp
+18 −32 tests/test_daily_logger.cpp
+8 −11 tests/test_errors.cpp
+3 −3 tests/test_eventlog.cpp
+28 −26 tests/test_file_helper.cpp
+14 −16 tests/test_file_logging.cpp
+5 −7 tests/test_macros.cpp
+1 −1 tests/test_pattern_formatter.cpp
+1 −1 tests/test_registry.cpp
+3 −5 tests/utils.cpp
15 changes: 6 additions & 9 deletions src/modules/registrypreview/RegistryPreview.FuzzTests/FuzzTests.cs
View file
Original file line number Diff line number Diff line change
@@ -24,9 +24,9 @@
string registryLine;

// Simulate registry file content as filenameText
var filenameText = GenerateRegistryHeader(input);
var registryContent = GenerateRegistryHeader(input);

string[] registryLines = filenameText.Split("\r");
string[] registryLines = registryContent.Split("\r");

if (registryLines.Length <= 1)
{
@@ -75,10 +75,9 @@
{
string registryLine;

var filenameText = GenerateRegistryHeader(input);
var regisrtyContent = GenerateRegistryHeader(input);

filenameText = filenameText.Replace("\r\n", "\r");
string[] registryLines = filenameText.Split("\r");
string[] registryLines = regisrtyContent.Split("\r");

if (registryLines.Length <= 1)
{
@@ -178,16 +177,14 @@
string header = new Random().Next(2) == 0 ? REGISTRYHEADER4 : REGISTRYHEADER5;

string inputText = System.Text.Encoding.UTF8.GetString(input);
string filenameText = header + "\r\n" + inputText;
string registryContent = header + "\r" + inputText;

return filenameText.Replace("\r\n", "\r");
return registryContent;
}

private static bool IsValidRegistryHeader(string line)
{
// Convert the line to lowercase once for comparison
var lineLower = line.ToLowerInvariant();

switch (line)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I noticed that you're using switch (line), which relies on the original line, but you convert lineLower to lowercase for comparison. Which one do you actually intend to use?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I intend to use the line, and the lineLower needs to be removed.

{
case REGISTRYHEADER4:
Loading
Oops, something went wrong.