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;