Sanity checks, errors out with nil ptr if API call fails

Author: godinezj

modules/post/multi/escalate/aws_create_iam_user.rb

@@ -118,8 +118,10 @@ def run
       action = 'CreateAccessKey'
       response = call_iam(creds, 'Action' => action, 'UserName' => username)
       doc = print_results(response, action)
-      results['SecretAccessKey'] = doc['SecretAccessKey']
-      results['AccessKeyId'] = doc['AccessKeyId']
+      if doc
+        results['SecretAccessKey'] = doc['SecretAccessKey']
+        results['AccessKeyId'] = doc['AccessKeyId']
+      end
     end
 
     if datastore['CREATE_CONSOLE']
@@ -128,14 +130,15 @@ def run
       action = 'CreateLoginProfile'
       response = call_iam(creds, 'Action' => action, 'UserName' => username, 'Password' => password)
       doc = print_results(response, action)
-      results['Password'] = password
+      results['Password'] = password if doc
     end
 
     action = 'GetUser'
     response = call_iam(creds, 'Action' => action, 'UserName' => username)
     doc = print_results(response, action)
+    return if doc.nil?
     arn = doc['Arn']
-    results['AccountId'] = arn[/^arn:aws:iam::(\d+):/,1]
+    results['AccountId'] = arn[/^arn:aws:iam::(\d+):/, 1]
 
     keys = results.keys
     table = Rex::Text::Table.new(