repository: prevent nil pointer dereference in error detection

tsuru · Feb 13, 2015 · 47ecc5d · 47ecc5d
1 parent 0b591b0
commit 47ecc5d
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/repository/repository.go b/repository/repository.go
@@ -312,10 +312,13 @@ func GrantAccess(rNames, uNames []string, readOnly bool) error {
 	} else {
 		info, err = conn.Repository().UpdateAll(bson.M{"_id": bson.M{"$in": rNames}}, bson.M{"$addToSet": bson.M{"users": bson.M{"$each": uNames}}})
 	}
+	if err != nil {
+		return err
+	}
 	if info.Updated < 1 {
 		return mgo.ErrNotFound
 	}
-	return err
+	return nil
 }
 
 // RevokeAccess revokes write permission from users in all specified