Fix the NPE issue in provisioner resource/datasource

Added the fix to return without trying to parse the nil response in case of any error. Signed-off-by: Ramya Bangera <[email protected]>
vmware · Jan 19, 2024 · 25c9eca · 25c9eca
1 parent ee9e28b
commit 25c9eca
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/internal/resources/provisioner/data_source_provisioner.go b/internal/resources/provisioner/data_source_provisioner.go
@@ -62,6 +62,8 @@ func dataSourceProvisionerRead(ctx context.Context, d *schema.ResourceData, m in
 				_ = schema.RemoveFromState(d, m)
 				return
 			}
+
+			return
 		}
 
 		for i := range resp.Provisioners {
@@ -78,6 +80,7 @@ func dataSourceProvisionerRead(ctx context.Context, d *schema.ResourceData, m in
 				_ = schema.RemoveFromState(d, m)
 				return
 			}
+			return
 		}
 
 		d.SetId(resp.Provisioner.Meta.UID)

diff --git a/internal/resources/provisioner/resource_provisioner.go b/internal/resources/provisioner/resource_provisioner.go
@@ -147,6 +147,8 @@ func resourceProvisionerRead(ctx context.Context, d *schema.ResourceData, m inte
 			_ = schema.RemoveFromState(d, m)
 			return
 		}
+
+		return
 	}
 
 	d.SetId(resp.Provisioner.Meta.UID)