postgres.git / summary / log / commit / refs

commit    96fc1a4f778eaad63c4bb41622e37a07fd433749
Author:   Tom Lane <tgl@sss.pgh.pa.us>
Date:     Tue Aug 16 00:48:29 2005 +0000
First released in: 8.1.0, 7.3.11, 7.4.9, 8.0.4

    Reject operator names >= NAMEDATALEN characters.  These will not work
    anyway, and in assert-enabled builds you are likely to get an assertion
    failure.  Backpatch as far as 7.3; 7.2 seems not to have the problem.


 src/backend/parser/scan.l | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/src/backend/parser/scan.l b/src/backend/parser/scan.l
index 7e944467bd0..433e42bf6a0 100644
--- a/src/backend/parser/scan.l
+++ b/src/backend/parser/scan.l
@@ -10,7 +10,7 @@
  * Portions Copyright (c) 1994, Regents of the University of California
  *
  * IDENTIFICATION
- *	  $PostgreSQL: pgsql/src/backend/parser/scan.l,v 1.119 2004/12/31 22:00:27 pgsql Exp $
+ *	  $PostgreSQL: pgsql/src/backend/parser/scan.l,v 1.119.4.1 2005/08/16 00:48:29 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -534,6 +534,15 @@ other			.
 							return yytext[0];
 					}
 
+					/*
+					 * Complain if operator is too long.  Unlike the case
+					 * for identifiers, we make this an error not a notice-
+					 * and-truncate, because the odds are we are looking at
+					 * a syntactic mistake anyway.
+					 */
+					if (nchars >= NAMEDATALEN)
+						yyerror("operator too long");
+
 					/* Convert "!=" operator to "<>" for compatibility */
 					if (strcmp(yytext, "!=") == 0)
 						yylval.str = pstrdup("<>");


[parent: ab648632eb4d]