Convert pool arguments to int

Failing to do so may cause dangerous misbehaviours such as an unbounded
pool (because of lame comparison operators in Python 2).

Fix ticket #220.
This commit is contained in:
Daniele Varrazzo 2014-08-13 02:37:00 +01:00
parent f9518e42b9
commit d232f25a68
2 changed files with 4 additions and 2 deletions

2
NEWS
View File

@ -7,6 +7,8 @@ What's new in psycopg 2.5.4
- Added :sql:`jsonb` support for PostgreSQL 9.4. - Added :sql:`jsonb` support for PostgreSQL 9.4.
- Fixed segfault if COPY statements are executed instead of using the - Fixed segfault if COPY statements are executed instead of using the
proper methods (:ticket:`#219`). proper methods (:ticket:`#219`).
- Force conversion of pool arguments to integer to avoid potentially unbounded
pools (:ticket:`#220`).
- Don't ignore silently the `cursor.callproc` argument without a length. - Don't ignore silently the `cursor.callproc` argument without a length.
- Added a few errors missing from `~psycopg2.errorcodes`, defined by - Added a few errors missing from `~psycopg2.errorcodes`, defined by
PostgreSQL but not documented. PostgreSQL but not documented.

View File

@ -42,8 +42,8 @@ class AbstractConnectionPool(object):
with given parameters. The connection pool will support a maximum of with given parameters. The connection pool will support a maximum of
about 'maxconn' connections. about 'maxconn' connections.
""" """
self.minconn = minconn self.minconn = int(minconn)
self.maxconn = maxconn self.maxconn = int(maxconn)
self.closed = False self.closed = False
self._args = args self._args = args