resource/aws_iam_saml_provider: Fixes for tfproviderlint R002 (#12027)

Reference: #9952

Remove pointer value dereferences, which can cause potential panics and are extraneous as `Set()` automatically handles pointer types including when `nil`.

Previously:

```
aws/resource_aws_iam_saml_provider.go:91:34: R002: ResourceData.Set() pointer value dereference is extraneous
```

Output from acceptance testing:

```
--- PASS: TestAccAWSIAMSamlProvider_basic (22.26s)
```

aws/resource_aws_iam_saml_provider.go

@@ -59,7 +59,7 @@ func resourceAwsIamSamlProviderCreate(d *schema.ResourceData, meta interface{})
 		return err
 	}
 
-	d.SetId(*out.SAMLProviderArn)
+	d.SetId(aws.StringValue(out.SAMLProviderArn))
 
 	return resourceAwsIamSamlProviderRead(d, meta)
 }
@@ -80,15 +80,14 @@ func resourceAwsIamSamlProviderRead(d *schema.ResourceData, meta interface{}) er
 		return err
 	}
 
-	validUntil := out.ValidUntil.Format(time.RFC1123)
 	d.Set("arn", d.Id())
 	name, err := extractNameFromIAMSamlProviderArn(d.Id(), meta.(*AWSClient).partition)
 	if err != nil {
 		return err
 	}
 	d.Set("name", name)
-	d.Set("valid_until", validUntil)
-	d.Set("saml_metadata_document", *out.SAMLMetadataDocument)
+	d.Set("valid_until", out.ValidUntil.Format(time.RFC1123))
+	d.Set("saml_metadata_document", out.SAMLMetadataDocument)
 
 	return nil
 }