always return after errors.

pkg/api/dashboard.go

@@ -91,6 +91,7 @@ func PostDashboard(c *middleware.Context, cmd m.SaveDashboardCommand) {
 		limitReached, err := m.QuotaReached(cmd.OrgId, m.QUOTA_DASHBOARD)
 		if err != nil {
 			c.JsonApiErr(500, "failed to get quota", err)
+			return
 		}
 		if limitReached {
 			c.JsonApiErr(403, "Quota reached", nil)

pkg/middleware/middleware.go

@@ -259,6 +259,7 @@ func LimitQuota(target m.QuotaTarget) macaron.Handler {
 		limitReached, err := m.QuotaReached(c.OrgId, target)
 		if err != nil {
 			c.JsonApiErr(500, "failed to get quota", err)
+			return
 		}
 		if limitReached {
 			c.JsonApiErr(403, "Quota reached", nil)