Re: BUG #6700: Potential Bug in numeric.c - Mailing list pgsql-bugs

From Michael Meskes
Subject Re: BUG #6700: Potential Bug in numeric.c
Date
Msg-id 20120620160242.GA12024@feivel.credativ.lan
Whole thread Raw
In response to BUG #6700: Potential Bug in numeric.c  (msrbugzilla@gmail.com)
List pgsql-bugs
On Wed, Jun 20, 2012 at 05:53:14AM +0000, msrbugzilla@gmail.com wrote:
> I observed a code clone in the following files. In the function
> PGTYPESnumeric_copy, the variable dst is ensured to be not NULL before
> passing to zero_var(). However, the function PGTYPESnumeric_from_decimal
> does not perform this checking. It will cause an exception when dst is NU=
LL
> and passed to zero_var().

To be honest most of the functions do not check if the parameters are indeed
set. Of course we could easily add the chekc for variable !=3D NULL but that
doesn't ensure that variable points to a valid numeric datatype. Suggestion=
s?

Michael
--=20
Michael Meskes
Michael at Fam-Meskes dot De, Michael at Meskes dot (De|Com|Net|Org)
Michael at BorussiaFan dot De, Meskes at (Debian|Postgresql) dot Org
Jabber: michael.meskes at googlemail dot com
VfL Borussia! For=C3=A7a Bar=C3=A7a! Go SF 49ers! Use Debian GNU/Linux, Pos=
tgreSQL

pgsql-bugs by date:

Previous
From: rikard.pavelic@zg.htnet.hr
Date:
Subject: BUG #6701: IS NOT NULL doesn't work on complex composites
Next
From: Chan Pham
Date:
Subject: log_truncate_on_rotation=on is not truncating