Commit 6eb05251 authored by Bruce Momjian's avatar Bruce Momjian

Prevent under/over flow of float8 constants in parser. Small regression fix.

parent 5b5bbdb1
...@@ -8,7 +8,7 @@ ...@@ -8,7 +8,7 @@
* *
* *
* IDENTIFICATION * IDENTIFICATION
* $Header: /cvsroot/pgsql/src/backend/parser/scan.l,v 1.9 1997/02/14 04:15:59 momjian Exp $ * $Header: /cvsroot/pgsql/src/backend/parser/scan.l,v 1.10 1997/02/19 20:10:38 momjian Exp $
* *
*------------------------------------------------------------------------- *-------------------------------------------------------------------------
*/ */
...@@ -20,6 +20,7 @@ ...@@ -20,6 +20,7 @@
#include <stdlib.h> #include <stdlib.h>
#endif /* __linux__ */ #endif /* __linux__ */
#include <string.h> #include <string.h>
#include <errno.h>
#include "postgres.h" #include "postgres.h"
#include "miscadmin.h" #include "miscadmin.h"
...@@ -30,6 +31,7 @@ ...@@ -30,6 +31,7 @@
#include "parser/scansup.h" #include "parser/scansup.h"
#include "parser/sysfunc.h" #include "parser/sysfunc.h"
#include "parse.h" #include "parse.h"
#include "utils/builtins.h"
extern char *parseString; extern char *parseString;
extern char *parseCh; extern char *parseCh;
...@@ -109,7 +111,12 @@ other . ...@@ -109,7 +111,12 @@ other .
return (ICONST); return (ICONST);
} }
{real} { {real} {
yylval.dval = atof((char*)yytext); char* endptr;
errno = 0;
yylval.dval = strtod(((char *)yytext),&endptr);
if (*endptr != '\0' || errno == ERANGE)
elog(WARN,"\tBad float8 input format\n");
CheckFloat8Val(yylval.dval);
return (FCONST); return (FCONST);
} }
{quote} { {quote} {
......
...@@ -7,7 +7,7 @@ ...@@ -7,7 +7,7 @@
* *
* *
* IDENTIFICATION * IDENTIFICATION
* $Header: /cvsroot/pgsql/src/backend/utils/adt/float.c,v 1.11 1997/02/14 04:17:52 momjian Exp $ * $Header: /cvsroot/pgsql/src/backend/utils/adt/float.c,v 1.12 1997/02/19 20:10:49 momjian Exp $
* *
*------------------------------------------------------------------------- *-------------------------------------------------------------------------
*/ */
...@@ -127,7 +127,7 @@ static void CheckFloat4Val(double val) ...@@ -127,7 +127,7 @@ static void CheckFloat4Val(double val)
raise an elog warning if it is raise an elog warning if it is
*/ */
static void CheckFloat8Val(double val) void CheckFloat8Val(double val)
{ {
/* defining unsafe floats's will make float4 and float8 ops faster /* defining unsafe floats's will make float4 and float8 ops faster
at the cost of safety, of course! */ at the cost of safety, of course! */
......
...@@ -6,7 +6,7 @@ ...@@ -6,7 +6,7 @@
* *
* Copyright (c) 1994, Regents of the University of California * Copyright (c) 1994, Regents of the University of California
* *
* $Id: builtins.h,v 1.8 1996/11/16 04:59:10 momjian Exp $ * $Id: builtins.h,v 1.9 1997/02/19 20:11:05 momjian Exp $
* *
* NOTES * NOTES
* This should normally only be included by fmgr.h. * This should normally only be included by fmgr.h.
...@@ -260,6 +260,7 @@ extern char *filename_in(char *file); ...@@ -260,6 +260,7 @@ extern char *filename_in(char *file);
extern char *filename_out(char *s); extern char *filename_out(char *s);
/* float.c */ /* float.c */
extern void CheckFloat8Val(double val); /* used by lex */
extern float32 float4in(char *num); extern float32 float4in(char *num);
extern char *float4out(float32 num); extern char *float4out(float32 num);
extern float64 float8in(char *num); extern float64 float8in(char *num);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment