Merge lp:~jtv/gwacl/missing-error-check into lp:gwacl

[Jeroen T. Vermeulen]

Missing error check in CreateAffinityGroup.

Another case of making error handling broken by default. Another nail in the coffin of the "Go forces you to think about error handling" myth. A traceback would have been perfectly acceptable, but continuing on with a nil pointer was not.

Jeroen

[Gavin Panella (allenap)]

Test?

review: Approve

=== modified file 'management_base.go'
--- management_base.go	2013-07-18 14:08:17 +0000
+++ management_base.go	2013-07-19 13:22:29 +0000
@@ -448,6 +448,9 @@
         return err
     }
     response, err := api.session.post(url, "2012-03-01", []byte(body), "application/xml")
+    if err != nil {
+        return err
+    }
     return api.blockUntilCompleted(response)
 }