bug in pg.py and the fix - Mailing list pgsql-patches

From Chih-Hao Huang
Subject bug in pg.py and the fix
Date
Msg-id 200306132132.h5DLWhI03049@woodlawn.fnal.gov
Whole thread Raw
Responses Re: [BUGS] bug in pg.py and the fix  (Bruce Momjian <pgman@candle.pha.pa.us>)
Re: [BUGS] bug in pg.py and the fix  (Bruce Momjian <pgman@candle.pha.pa.us>)
List pgsql-patches
If PostgreSQL failed to compile on your computer or you found a bug that
is likely to be specific to one platform then please fill out this form
and e-mail it to pgsql-ports@postgresql.org.

To report any other bug, fill out the form below and e-mail it to
pgsql-bugs@postgresql.org.

If you not only found the problem but solved it and generated a patch
then e-mail it to pgsql-patches@postgresql.org instead.  Please use the
command "diff -c" to generate the patch.

You may also enter a bug report at http://www.postgresql.org/ instead of
e-mail-ing this form.

============================================================================
                        POSTGRESQL BUG REPORT TEMPLATE
============================================================================


Your name        :    Chih-Hao Huang
Your email address    :    huangch@fnal.gov


System Configuration
---------------------
  Architecture (example: Intel Pentium)      : Intel Pentium

  Operating System (example: Linux 2.0.26 ELF)     : Linux 2.4.3-12

  PostgreSQL version (example: PostgreSQL-7.3.3):   PostgreSQL-7.3.3

  Compiler used (example:  gcc 2.95.2)        : gcc version 2.96 20000731 (Red Hat Linux 7.1 2.96-85)


Please enter a FULL description of your problem:
------------------------------------------------

This is a bug in python interface module,
postgresql-7.3.3/src/interfaces/python/pg.py.

_quote() function fails due to integer overflow if input d is larger
than max integer.

In the case where the column type is "BIGINT", the input d may very well
be larger than max integer while its type, t, is labeled 'int'.
The conversion on line 19, return "%d" % int(d), will fail due to
"OverflowError: long int too large to convert to int".



Please describe a way to repeat the problem.   Please try to provide a
concise reproducible example, if at all possible:
----------------------------------------------------------------------

[1] create a table with a column type 'BIGINT'.
[2] use pg.DB.insert() to insert a value that is larger than max integer

If you know how this problem might be fixed, list the solution below:
---------------------------------------------------------------------

Just changing the conversion at line 19 of pg.py to long(d) instead of
int(d) should fix it. The following is a patch:

*** pg.py    Fri Jun 13 15:15:57 2003
--- pg.py.new    Fri Jun 13 15:16:59 2003
***************
*** 16,22 ****

      if t in ['int', 'seq']:
          if d == "": return "NULL"
!         return "%d" % int(d)

      if t == 'decimal':
          if d == "": return "NULL"
--- 16,22 ----

      if t in ['int', 'seq']:
          if d == "": return "NULL"
!         return "%d" % long(d)

      if t == 'decimal':
          if d == "": return "NULL"

pgsql-patches by date:

Previous
From: deststar
Date:
Subject: ecpg compile (compatlib)
Next
From: Bruce Momjian
Date:
Subject: Re: [BUGS] bug in pg.py and the fix