summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorMichael Meskes <meskes@postgresql.org>2013-09-08 12:59:43 +0200
committerMichael Meskes <meskes@postgresql.org>2013-09-08 13:13:32 +0200
commit994f68d15b8c622eeda77675121861d1c9b7d34d (patch)
tree66a1f6bfbd250e02a3b75623e0f4e3160e1f5d11 /src
parent3f092dbedf97ed2c6a03e3f4b2b3686f21cf2019 (diff)
Return error if allocation of new element was not possible.
Found by Coverity.
Diffstat (limited to 'src')
-rw-r--r--src/interfaces/ecpg/pgtypeslib/numeric.c11
1 files changed, 9 insertions, 2 deletions
diff --git a/src/interfaces/ecpg/pgtypeslib/numeric.c b/src/interfaces/ecpg/pgtypeslib/numeric.c
index 6989b664514..e8b162a4d13 100644
--- a/src/interfaces/ecpg/pgtypeslib/numeric.c
+++ b/src/interfaces/ecpg/pgtypeslib/numeric.c
@@ -430,14 +430,18 @@ PGTYPESnumeric_to_asc(numeric *num, int dscale)
numeric *numcopy = PGTYPESnumeric_new();
char *s;
- if (dscale < 0)
- dscale = num->dscale;
+ if (numcopy == NULL)
+ return NULL;
if (PGTYPESnumeric_copy(num, numcopy) < 0)
{
PGTYPESnumeric_free(numcopy);
return NULL;
}
+
+ if (dscale < 0)
+ dscale = num->dscale;
+
/* get_str_from_var may change its argument */
s = get_str_from_var(numcopy, dscale);
PGTYPESnumeric_free(numcopy);
@@ -1520,6 +1524,9 @@ numericvar_to_double(numeric *var, double *dp)
char *endptr;
numeric *varcopy = PGTYPESnumeric_new();
+ if (varcopy == NULL)
+ return -1;
+
if (PGTYPESnumeric_copy(var, varcopy) < 0)
{
PGTYPESnumeric_free(varcopy);