From 58d9018a7b528e22fad67053a722184cb6569100 Mon Sep 17 00:00:00 2001 From: Chris Rummel Date: Mon, 22 Aug 2022 22:40:20 -0500 Subject: [PATCH] Update poison baseline. (#14345) * Update poison baseline. * Fix RIDs by replacing with placeholders. * Exclude a few more non-binary file types from poison checking. * Ignore empty files when checking for poison. * Update baseline with new exclusions. * Poison report can have both portable and non-portable RIDs in it, remove both. --- .../PoisonTests.cs | 1 + .../assets/baselines/PoisonUsage.txt | 1281 +++++++++-------- .../CheckForPoison.cs | 14 +- 3 files changed, 694 insertions(+), 602 deletions(-) diff --git a/src/SourceBuild/tarball/content/test/Microsoft.DotNet.SourceBuild.SmokeTests/PoisonTests.cs b/src/SourceBuild/tarball/content/test/Microsoft.DotNet.SourceBuild.SmokeTests/PoisonTests.cs index 11ea0667f..f600d558d 100644 --- a/src/SourceBuild/tarball/content/test/Microsoft.DotNet.SourceBuild.SmokeTests/PoisonTests.cs +++ b/src/SourceBuild/tarball/content/test/Microsoft.DotNet.SourceBuild.SmokeTests/PoisonTests.cs @@ -24,6 +24,7 @@ namespace Microsoft.DotNet.SourceBuild.SmokeTests string currentPoisonReport = File.ReadAllText(Config.PoisonReportPath); currentPoisonReport = RemoveHashes(currentPoisonReport); currentPoisonReport = BaselineHelper.RemoveRids(currentPoisonReport); + currentPoisonReport = BaselineHelper.RemoveRids(currentPoisonReport, true); currentPoisonReport = BaselineHelper.RemoveVersions(currentPoisonReport); BaselineHelper.CompareContents("PoisonUsage.txt", currentPoisonReport, OutputHelper, Config.WarnOnPoisonDiffs); diff --git a/src/SourceBuild/tarball/content/test/Microsoft.DotNet.SourceBuild.SmokeTests/assets/baselines/PoisonUsage.txt b/src/SourceBuild/tarball/content/test/Microsoft.DotNet.SourceBuild.SmokeTests/assets/baselines/PoisonUsage.txt index 25f79024b..fcac02fbd 100644 --- a/src/SourceBuild/tarball/content/test/Microsoft.DotNet.SourceBuild.SmokeTests/assets/baselines/PoisonUsage.txt +++ b/src/SourceBuild/tarball/content/test/Microsoft.DotNet.SourceBuild.SmokeTests/assets/baselines/PoisonUsage.txt @@ -1,77 +1,218 @@ - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + AssemblyAttribute @@ -80,70 +221,130 @@ AssemblyAttribute - - AssemblyAttribute - AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - AssemblyAttribute AssemblyAttribute - + + AssemblyAttribute + + AssemblyAttribute AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + AssemblyAttribute AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + AssemblyAttribute @@ -152,28 +353,16 @@ AssemblyAttribute - - AssemblyAttribute - AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - AssemblyAttribute AssemblyAttribute - - AssemblyAttribute - - + AssemblyAttribute @@ -182,142 +371,115 @@ AssemblyAttribute - - AssemblyAttribute - AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - AssemblyAttribute AssemblyAttribute - - AssemblyAttribute - AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - AssemblyAttribute AssemblyAttribute - - AssemblyAttribute - AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - AssemblyAttribute AssemblyAttribute - - AssemblyAttribute - AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + + AssemblyAttribute + + AssemblyAttribute @@ -326,202 +488,94 @@ AssemblyAttribute - - AssemblyAttribute - AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - AssemblyAttribute AssemblyAttribute - - AssemblyAttribute - AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - AssemblyAttribute AssemblyAttribute - - AssemblyAttribute - AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - AssemblyAttribute AssemblyAttribute - - AssemblyAttribute - AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - AssemblyAttribute AssemblyAttribute - - AssemblyAttribute - AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - AssemblyAttribute AssemblyAttribute - - AssemblyAttribute - AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - AssemblyAttribute AssemblyAttribute - - AssemblyAttribute - AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - AssemblyAttribute AssemblyAttribute - - AssemblyAttribute - AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - AssemblyAttribute AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - AssemblyAttribute AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - - - AssemblyAttribute - - - AssemblyAttribute - - - AssemblyAttribute - - - AssemblyAttribute - - - AssemblyAttribute - - - AssemblyAttribute - - - AssemblyAttribute - - - AssemblyAttribute - - + AssemblyAttribute @@ -530,226 +584,154 @@ AssemblyAttribute - - AssemblyAttribute - AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - AssemblyAttribute AssemblyAttribute - - AssemblyAttribute - AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - - - AssemblyAttribute - - - AssemblyAttribute - - - AssemblyAttribute - - - AssemblyAttribute - - - AssemblyAttribute - - - AssemblyAttribute - - - AssemblyAttribute - - - AssemblyAttribute - - - AssemblyAttribute - - - AssemblyAttribute - - - AssemblyAttribute - - - AssemblyAttribute - - - AssemblyAttribute - - - AssemblyAttribute - - - AssemblyAttribute - - - AssemblyAttribute - - - AssemblyAttribute - - - AssemblyAttribute - - + AssemblyAttribute @@ -758,190 +740,91 @@ AssemblyAttribute - - AssemblyAttribute - AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - AssemblyAttribute AssemblyAttribute - - AssemblyAttribute - AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - AssemblyAttribute AssemblyAttribute - - AssemblyAttribute - AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - AssemblyAttribute AssemblyAttribute - - AssemblyAttribute - AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - AssemblyAttribute AssemblyAttribute - - AssemblyAttribute - AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - AssemblyAttribute AssemblyAttribute - - AssemblyAttribute - AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - AssemblyAttribute AssemblyAttribute - - AssemblyAttribute - AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - AssemblyAttribute AssemblyAttribute - - AssemblyAttribute - AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - AssemblyAttribute AssemblyAttribute - - AssemblyAttribute - AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - - - AssemblyAttribute - - - AssemblyAttribute - - - AssemblyAttribute - - - AssemblyAttribute - - - AssemblyAttribute - - - AssemblyAttribute - - + AssemblyAttribute @@ -950,34 +833,64 @@ AssemblyAttribute - - AssemblyAttribute - AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - AssemblyAttribute AssemblyAttribute - - AssemblyAttribute - AssemblyAttribute - + AssemblyAttribute - + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + AssemblyAttribute @@ -986,100 +899,67 @@ AssemblyAttribute - - AssemblyAttribute - AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - AssemblyAttribute AssemblyAttribute - - AssemblyAttribute - AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - AssemblyAttribute AssemblyAttribute - - AssemblyAttribute - AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - AssemblyAttribute AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - - + AssemblyAttribute @@ -1088,52 +968,55 @@ AssemblyAttribute - - AssemblyAttribute - AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - AssemblyAttribute AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - AssemblyAttribute AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + AssemblyAttribute @@ -1142,52 +1025,55 @@ AssemblyAttribute - - AssemblyAttribute - AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - AssemblyAttribute AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - AssemblyAttribute AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + AssemblyAttribute @@ -1196,88 +1082,76 @@ AssemblyAttribute - - AssemblyAttribute - AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - AssemblyAttribute AssemblyAttribute - - AssemblyAttribute - AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - AssemblyAttribute AssemblyAttribute - - AssemblyAttribute - - - AssemblyAttribute - AssemblyAttribute AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + AssemblyAttribute @@ -1286,43 +1160,250 @@ AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + AssemblyAttribute - + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + + AssemblyAttribute + + AssemblyAttribute diff --git a/src/SourceBuild/tarball/content/tools-local/tasks/Microsoft.DotNet.SourceBuild.Tasks.LeakDetection/CheckForPoison.cs b/src/SourceBuild/tarball/content/tools-local/tasks/Microsoft.DotNet.SourceBuild.Tasks.LeakDetection/CheckForPoison.cs index d3c683c56..b8d3ffe24 100644 --- a/src/SourceBuild/tarball/content/tools-local/tasks/Microsoft.DotNet.SourceBuild.Tasks.LeakDetection/CheckForPoison.cs +++ b/src/SourceBuild/tarball/content/tools-local/tasks/Microsoft.DotNet.SourceBuild.Tasks.LeakDetection/CheckForPoison.cs @@ -79,7 +79,6 @@ namespace Microsoft.DotNet.SourceBuild.Tasks.LeakDetection "_._", "-.-", ".bowerrc", - ".editorconfig", ".gitignore", ".gitkeep", ".rels", @@ -97,9 +96,11 @@ namespace Microsoft.DotNet.SourceBuild.Tasks.LeakDetection ".csproj", ".css", ".db", + ".editorconfig", ".eot", ".fs", ".fsproj", + ".h", ".html", ".ico", ".js", @@ -107,18 +108,26 @@ namespace Microsoft.DotNet.SourceBuild.Tasks.LeakDetection ".map", ".md", ".nuspec", + ".otf", ".png", ".props", + ".proto", + ".proj", ".psmdcp", + ".pubxml", + ".razor", ".rtf", ".scss", + ".sln", ".svg", ".targets", + ".transform", ".ts", ".ttf", ".txt", ".vb", ".vbproj", + ".win32manifest", ".woff", ".woff2", ".xaml", @@ -209,7 +218,8 @@ namespace Microsoft.DotNet.SourceBuild.Tasks.LeakDetection // skip some common files that get copied verbatim from nupkgs - LICENSE, _._, etc as well as // file types that we never care about - text files, .gitconfig, etc. if (FileNamesToSkip.Any(f => Path.GetFileName(fileToCheck).ToLowerInvariant() == f.ToLowerInvariant()) || - FileExtensionsToSkip.Any(e => Path.GetExtension(fileToCheck).ToLowerInvariant() == e.ToLowerInvariant())) + FileExtensionsToSkip.Any(e => Path.GetExtension(fileToCheck).ToLowerInvariant() == e.ToLowerInvariant()) || + (new FileInfo(fileToCheck).Length == 0)) { return null; }