Add most of Neil Conway's cleanups.
authorBruce Momjian
Tue, 27 Aug 2002 15:15:23 +0000 (15:15 +0000)
committerBruce Momjian
Tue, 27 Aug 2002 15:15:23 +0000 (15:15 +0000)
src/backend/libpq/auth.c
src/include/libpq/pqcomm.h

index 83f7cb10c22c9af166627621c47127c1e79e7e5e..a36535cb97491348e52c64c715b8a24e0aac272f 100644 (file)
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *   $Header: /cvsroot/pgsql/src/backend/libpq/auth.c,v 1.83 2002/08/18 03:03:25 momjian Exp $
+ *   $Header: /cvsroot/pgsql/src/backend/libpq/auth.c,v 1.84 2002/08/27 15:15:22 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -809,7 +809,7 @@ recv_and_check_password_packet(Port *port)
        return STATUS_EOF;
    }
 
-   /* Do not echo failed password to logs, for security. */
+   /* Do not echo password to logs, for security. */
    elog(DEBUG5, "received password packet");
 
    result = md5_crypt_verify(port, port->user, buf.data);
index f0ea4e359436ba15cd83e2f5a879a6b8f0327c28..728cce81cd1702477e3ecef05a22c45830990ce7 100644 (file)
@@ -9,7 +9,7 @@
  * Portions Copyright (c) 1996-2002, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $Id: pqcomm.h,v 1.66 2002/08/18 03:03:26 momjian Exp $
+ * $Id: pqcomm.h,v 1.67 2002/08/27 15:15:23 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -106,10 +106,18 @@ typedef uint32 PacketLen;
 
 /*
  * Startup message parameters sizes.  These must not be changed without changing
- * the protcol version.  These are all strings that are '\0' terminated only if
+ * the protocol version.  These are all strings that are '\0' terminated only if
  * there is room.
  */
 
+/*
+ * FIXME: remove the fixed size limitations on database & user name, use
+ * variable length fields instead. The actual values will still be
+ * limited by NAMEDATALEN, but this will at least allow changing
+ * NAMEDATALEN to increase database & user name limits without changing
+ * the protocol. -neilc, 2002/08/27
+ */
 /* These should all be of near-unlimited length, perhap 10k */
 #define SM_DATABASE        64
 /* SM_USER should be the same size as the others.  bjm 2002-06-02 */
@@ -120,7 +128,7 @@ typedef uint32 PacketLen;
 #define SM_UNUSED      64
 #define SM_TTY         64
 
-typedef uint32 ProtocolVersion; /* Fe/Be protocol version nr. */
+typedef uint32 ProtocolVersion; /* Fe/Be protocol version number */
 
 typedef struct StartupPacket
 {