On Apr 4, 2011, at 1:46 AM, Joseph Adams <joeyadams3(dot)14159(at)gmail(dot)com> wrote:
> On Sun, Apr 3, 2011 at 11:23 PM, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
>> Thanks for the patch, but I think you forgot to worry about overflow:
>> 
>> rhaas=# select 9223372036854775807::money;
>>  money
>> --------
>>  -$1.00
>> (1 row)
> 
> cash_in doesn't test for overflow, either (tested on 8.4.0, 9.0.3, and HEAD):
> 
> joey=# select '9223372036854775807'::money;
> money
> --------
> -$1.00
> (1 row)
> 
> Is this a bug?

Seems like it. You have to feel sorry for the guy who deposits 9 quintillion dollars and then gets a note from the bank saying his account is overdrawn...

> Detail: unlike cash_in, numeric_cash does check for overflow
> (implicitly, through its use of numeric_int8).

Yeah.

...Robert

In response to

• Re: cast from integer to money at 2011-04-04 05:46:17 from Joseph Adams

Responses

• Re: cast from integer to money at 2011-04-04 11:09:31 from Andrew Dunstan
• Re: cast from integer to money at 2011-04-04 14:58:16 from Tom Lane

pgsql-hackers by date

Next:	From: Andrew Dunstan	Date: 2011-04-04 11:09:31
	Subject: Re: cast from integer to money
Previous:	From: Shigeru HANADA	Date: 2011-04-04 10:49:09
	Subject: Re: Re: [COMMITTERS] pgsql: Support comments on FOREIGN DATA WRAPPER and SERVER objects.