comment potentially-confusing use of struct crypt_data type
authorRich Felker <dalias@aerifal.cx>
Sat, 20 Apr 2013 18:07:01 +0000 (14:07 -0400)
committerRich Felker <dalias@aerifal.cx>
Sat, 20 Apr 2013 18:07:01 +0000 (14:07 -0400)
src/crypt/crypt.c
src/crypt/crypt_r.c

index f1e310f6f1ca406c12410c57c3f22b54e3bea0e8..46500737b6598ae58f740ba2808e35507e2b73ba 100644 (file)
@@ -5,7 +5,12 @@ char *__crypt_r(const char *, const char *, struct crypt_data *);
 
 char *crypt(const char *key, const char *salt)
 {
-       /* Note: update this size when we add more hash types */
+       /* This buffer is sufficiently large for all
+        * currently-supported hash types. It needs to be updated if
+        * longer hashes are added. The cast to struct crypt_data * is
+        * purely to meet the public API requirements of the crypt_r
+        * function; the implementation of crypt_r uses the object
+        * purely as a char buffer. */
        static char buf[128];
        return __crypt_r(key, salt, (struct crypt_data *)buf);
 }
index 3257e8b936ce7f5cbd08e1d5fc0745c181116570..5982c4c9abc7db638b9c949c905ad37a03824f02 100644 (file)
@@ -11,6 +11,10 @@ char *__crypt_sha512(const char *, const char *, char *);
 
 char *__crypt_r(const char *key, const char *salt, struct crypt_data *data)
 {
+       /* Per the crypt_r API, the caller has provided a pointer to
+        * struct crypt_data; however, this implementation does not
+        * use the structure to store any internal state, and treats
+        * it purely as a char buffer for storing the result. */
        char *output = (char *)data;
        if (salt[0] == '$' && salt[1] && salt[2]) {
                if (salt[1] == '1' && salt[2] == '$')