Clean up comments.

diff --git a/src/backend/libpq/md5.c b/src/backend/libpq/md5.c
index d4a67303197c724ceaaaee20c0b24378c55db12a..c47abc71ce1defa2a5a0372274eb6bd333ee4f77 100644 (file)
--- a/src/backend/libpq/md5.c
+++ b/src/backend/libpq/md5.c
@@ -10,13 +10,11 @@
  *
  *  Sverre H. Huseby 
  *
- * $Header: /cvsroot/pgsql/src/backend/libpq/md5.c,v 1.7 2001/09/27 23:16:23 momjian Exp $
+ * $Header: /cvsroot/pgsql/src/backend/libpq/md5.c,v 1.8 2001/09/29 19:49:50 tgl Exp $
  */
 
 #include "postgres.h"
 
-#include 
-
 #include "libpq/crypt.h"
 
 #ifdef FRONTEND
@@ -291,24 +289,31 @@ md5_hash(const void *buff, size_t len, char *hexsum)
 
 
 /*
- * puts  md5(username+passwd)  in buf provided buflen is at least 36 bytes
- * returns 1 on success, 0 on any kind of failure and sets errno accordingly
+ * Computes MD5 checksum of "passwd" (a null-terminated string) followed
+ * by "salt" (which need not be null-terminated).
+ *
+ * Output format is "md5" followed by a 32-hex-digit MD5 checksum.
+ * Hence, the output buffer "buf" must be at least 36 bytes long.
+ *
+ * Returns TRUE if okay, FALSE on error (out of memory).
  */
 bool EncryptMD5(const char *passwd, const char *salt, size_t salt_len,
                char *buf)
 {
-   char *crypt_buf = palloc(strlen(passwd) + salt_len);
+   size_t  passwd_len = strlen(passwd);
+   char *crypt_buf = palloc(passwd_len + salt_len);
    bool ret;
    
-   strcpy(buf, "md5");
    /*
     *  Place salt at the end because it may be known by users
     *  trying to crack the MD5 output.
     */ 
    strcpy(crypt_buf, passwd);
-   memcpy(crypt_buf+strlen(passwd), salt, salt_len);
+   memcpy(crypt_buf+passwd_len, salt, salt_len);
+
+   strcpy(buf, "md5");
+   ret = md5_hash(crypt_buf, passwd_len + salt_len, buf + 3);
 
-   ret = md5_hash(crypt_buf, strlen(passwd) + salt_len, buf + 3);
    pfree(crypt_buf);
 
    return ret;