From 3c346867f4dd42d560e0736917dc6d276535396a Mon Sep 17 00:00:00 2001 From: Megan Marsh Date: Mon, 7 Dec 2020 15:41:40 -0800 Subject: [PATCH] fix nil pointer dereference --- builder/amazon/common/step_run_source_instance.go | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/builder/amazon/common/step_run_source_instance.go b/builder/amazon/common/step_run_source_instance.go index 7b07e23e8..4543b520b 100644 --- a/builder/amazon/common/step_run_source_instance.go +++ b/builder/amazon/common/step_run_source_instance.go @@ -251,8 +251,10 @@ func (s *StepRunSourceInstance) Run(ctx context.Context, state multistep.StateBa if resp, e := ec2conn.DescribeInstances(describeInstance); e == nil { if len(resp.Reservations) > 0 && len(resp.Reservations[0].Instances) > 0 { instance := resp.Reservations[0].Instances[0] - ui.Error(fmt.Sprintf("Instance state change details: %s: %s", - *instance.StateTransitionReason, *instance.StateReason.Message)) + if instance.StateTransitionReason != nil && instance.StateReason.Message != nil { + ui.Error(fmt.Sprintf("Instance state change details: %s: %s", + *instance.StateTransitionReason, *instance.StateReason.Message)) + } } } return multistep.ActionHalt