Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commitac0aa07

Browse files
committed
Guard against bad "dscale" values in numeric_recv().
We were not checking to see if the supplied dscale was valid for the givendigit array when receiving binary-format numeric values. While dscale canvalidly be more than the number of nonzero fractional digits, it shouldn'tbe less; that case causes fractional digits to be hidden on display eventhough they're there and participate in arithmetic.Bug #12053 from Tommaso Sala indicates that there's at least one brokenclient library out there that sometimes supplies an incorrect dscale value,leading to strange behavior. This suggests that simply throwing an errormight not be the best response; it would lead to failures in applicationsthat might seem to be working fine today. What seems the least risky fixis to truncate away any digits that would be hidden by dscale. Thispreserves the existing behavior in terms of what will be printed for thetransmitted value, while preventing subsequent arithmetic from producingresults inconsistent with that.In passing, throw a specific error for the case of dscale being outsidethe range that will fit into a numeric's header. Before you got "valueoverflows numeric format", which is a bit misleading.Back-patch to all supported branches.
1 parentf82132f commitac0aa07

File tree

1 file changed

+15
-0
lines changed

1 file changed

+15
-0
lines changed

‎src/backend/utils/adt/numeric.c

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -704,6 +704,8 @@ numeric_recv(PG_FUNCTION_ARGS)
704704
alloc_var(&value,len);
705705

706706
value.weight= (int16)pq_getmsgint(buf,sizeof(int16));
707+
/* we allow any int16 for weight --- OK? */
708+
707709
value.sign= (uint16)pq_getmsgint(buf,sizeof(uint16));
708710
if (!(value.sign==NUMERIC_POS||
709711
value.sign==NUMERIC_NEG||
@@ -713,6 +715,11 @@ numeric_recv(PG_FUNCTION_ARGS)
713715
errmsg("invalid sign in external \"numeric\" value")));
714716

715717
value.dscale= (uint16)pq_getmsgint(buf,sizeof(uint16));
718+
if ((value.dscale&NUMERIC_DSCALE_MASK)!=value.dscale)
719+
ereport(ERROR,
720+
(errcode(ERRCODE_INVALID_BINARY_REPRESENTATION),
721+
errmsg("invalid scale in external \"numeric\" value")));
722+
716723
for (i=0;i<len;i++)
717724
{
718725
NumericDigitd=pq_getmsgint(buf,sizeof(NumericDigit));
@@ -724,6 +731,14 @@ numeric_recv(PG_FUNCTION_ARGS)
724731
value.digits[i]=d;
725732
}
726733

734+
/*
735+
* If the given dscale would hide any digits, truncate those digits away.
736+
* We could alternatively throw an error, but that would take a bunch of
737+
* extra code (about as much as trunc_var involves), and it might cause
738+
* client compatibility issues.
739+
*/
740+
trunc_var(&value,value.dscale);
741+
727742
apply_typmod(&value,typmod);
728743

729744
res=make_result(&value);

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp