Back out password packet length check.
authorBruce Momjian
Fri, 30 Aug 2002 16:00:41 +0000 (16:00 +0000)
committerBruce Momjian
Fri, 30 Aug 2002 16:00:41 +0000 (16:00 +0000)
Improve wording of pre-7.3 syntax mention.

doc/src/sgml/ref/copy.sgml
doc/src/sgml/ref/select.sgml
src/backend/libpq/auth.c

index c5c8fcbb94749ec536c72c01757f66fe23b20c2b..677ec5792a977a8221d6339588e28f861211930a 100644 (file)
@@ -1,5 +1,5 @@
 
 
@@ -700,7 +700,7 @@ ZW      ZIMBABWE
     There is no COPY statement in SQL92.
    
    
-    The following syntax was used by pre-7.3 servers and is still supported:
+    The following syntax was used by pre-7.3 applications and is still supported:
     
     COPY [ BINARY ] table [ WITH OIDS ]
         FROM { 'filename' | stdin }
index 1feaa6f7b480211707a1e07e129f9c7b2ef46db8..7192de53e0dd33e3ed90cbc8dc24d5f74782fb6d 100644 (file)
@@ -1,5 +1,5 @@
 
 
@@ -869,7 +869,7 @@ SELECT name FROM distributors ORDER BY code;
     FOR UPDATE cannot be used in contexts where returned rows can't be clearly
     identified with individual table rows; for example it can't be used with
     aggregation.  FOR UPDATE may also appear before LIMIT for portability with
-    pre-7.3 servers.
+    pre-7.3 applications.
    
   
 
index fb43f8e4af8e2c3c70efd150e712f285c4d1fbed..216343f36c97bab18d380a61f85f4123b6859fdd 100644 (file)
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *   $Header: /cvsroot/pgsql/src/backend/libpq/auth.c,v 1.87 2002/08/29 21:50:36 momjian Exp $
+ *   $Header: /cvsroot/pgsql/src/backend/libpq/auth.c,v 1.88 2002/08/30 16:00:41 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -709,20 +709,6 @@ recv_and_check_password_packet(Port *port)
    if (pq_eof() == EOF || pq_getint(&len, 4) == EOF)
        return STATUS_EOF;      /* client didn't want to send password */
 
-   /*
-    * Since the remote client has not yet been authenticated, we need
-    * to be careful when using the data they send us. The 8K limit is
-    * arbitrary, and somewhat bogus: the intent is to ensure we don't
-    * allocate an enormous chunk of memory.
-    */
-
-   if (len < 1 || len > 8192)
-   {
-       elog(LOG, "Invalid password packet length: %d; "
-            "must satisfy 1 <= length <= 8192", len);
-       return STATUS_EOF;
-   }
-
    initStringInfo(&buf);
    if (pq_getstr(&buf) == EOF) /* receive password */
    {