From 5f50191de837d02d56d6a3eb633e193ff848350d Mon Sep 17 00:00:00 2001
From: Mikel Blanchard <mblanchard@macrosssoftware.com>
Date: Mon, 27 Sep 2021 09:42:03 -0700
Subject: [PATCH] Code review.

---
 .../Implementation/MeterProviderBuilderHosting.cs            | 5 +++++
 .../Implementation/TracerProviderBuilderHosting.cs           | 5 +++++
 2 files changed, 10 insertions(+)

diff --git a/src/OpenTelemetry.Extensions.Hosting/Implementation/MeterProviderBuilderHosting.cs b/src/OpenTelemetry.Extensions.Hosting/Implementation/MeterProviderBuilderHosting.cs
index 08379ac66be..e5529bc1b2b 100644
--- a/src/OpenTelemetry.Extensions.Hosting/Implementation/MeterProviderBuilderHosting.cs
+++ b/src/OpenTelemetry.Extensions.Hosting/Implementation/MeterProviderBuilderHosting.cs
@@ -47,6 +47,11 @@ public MeterProviderBuilder Configure(Action<IServiceProvider, MeterProviderBuil
 
         public MeterProvider Build(IServiceProvider serviceProvider)
         {
+            if (serviceProvider == null)
+            {
+                throw new ArgumentNullException(nameof(serviceProvider));
+            }
+
             // Note: Not using a foreach loop because additional actions can be
             // added during each call.
             for (int i = 0; i < this.configurationActions.Count; i++)
diff --git a/src/OpenTelemetry.Extensions.Hosting/Implementation/TracerProviderBuilderHosting.cs b/src/OpenTelemetry.Extensions.Hosting/Implementation/TracerProviderBuilderHosting.cs
index c48ef0cd42a..f56e9d12aea 100644
--- a/src/OpenTelemetry.Extensions.Hosting/Implementation/TracerProviderBuilderHosting.cs
+++ b/src/OpenTelemetry.Extensions.Hosting/Implementation/TracerProviderBuilderHosting.cs
@@ -47,6 +47,11 @@ public TracerProviderBuilder Configure(Action<IServiceProvider, TracerProviderBu
 
         public TracerProvider Build(IServiceProvider serviceProvider)
         {
+            if (serviceProvider == null)
+            {
+                throw new ArgumentNullException(nameof(serviceProvider));
+            }
+
             // Note: Not using a foreach loop because additional actions can be
             // added during each call.
             for (int i = 0; i < this.configurationActions.Count; i++)