Fix uncaught code analysis warnings causing a build break

dotnet · Dec 28, 2022 · e9242d6 · e9242d6
1 parent 52cb28d
commit e9242d6
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/src/AspNet/OData/src/Asp.Versioning.WebApi.OData.ApiExplorer/ApiExplorer/ODataApiExplorer.cs b/src/AspNet/OData/src/Asp.Versioning.WebApi.OData.ApiExplorer/ApiExplorer/ODataApiExplorer.cs
@@ -54,7 +54,7 @@ public ODataApiExplorer( HttpConfiguration configuration )
     public ODataApiExplorer( HttpConfiguration configuration, ODataApiExplorerOptions options )
         : base( configuration, options )
     {
-        this.options = options;
+        this.options = options ?? throw new ArgumentNullException( nameof( options ) );
         options.AdHocModelBuilder.OnModelCreated += MarkAsAdHoc;
     }
 

diff --git a/...p.Versioning.WebApi.OData.ApiExplorer/Conventions/ImplicitModelBoundSettingsConvention.cs b/...p.Versioning.WebApi.OData.ApiExplorer/Conventions/ImplicitModelBoundSettingsConvention.cs
@@ -13,6 +13,11 @@ public partial class ImplicitModelBoundSettingsConvention : IModelConfiguration,
     /// <inheritdoc />
     public void ApplyTo( ApiDescription apiDescription )
     {
+        if ( apiDescription == null )
+        {
+            throw new ArgumentNullException( nameof( apiDescription ) );
+        }
+
         var response = apiDescription.ResponseDescription;
         var type = response.ResponseType ?? response.DeclaredType;