]> granicus.if.org Git - php/commit
Clean house in cryptographic hashing code
authorAlex Dowad <alexinbeijing@gmail.com>
Tue, 23 Jun 2020 14:32:41 +0000 (16:32 +0200)
committerAlex Dowad <alexinbeijing@gmail.com>
Wed, 24 Jun 2020 11:40:27 +0000 (13:40 +0200)
commit19d3e29bf5383b5b71b00727bd9bb5971bbbdce5
treecee4ee4c8791aeec0c295d5fc6eba529f83ae690
parente94afec7007dcd3557a40b020f6f184fa2338cdf
Clean house in cryptographic hashing code

- Remove dead code from php_crypt_r.c

  This code has been commented out since the file was added in 2008. It's safe to say
  that no-one is ever going to use it.

- Fix typo in comment in php_crypt_r.c

- Remove redundant Windows-only implementation of php_md5_crypt_r

  There is a portable implementation in the same file, which is selected if not
  building for Windows. But why should Windows have its own special implementation
  of this function at all? There doesn't seem to be any good reason.

  Better to use the portable implementation on all platforms.

- Don't define useless __CONST macro in php_crypt_r.h

  This preprocessor macro is not used anywhere.

- Add comment on functions for encoding data as Base64

- Remove dead code from crypt_blowfish.h

- Remove unneeded junk comments from crypt_freesec.c

- Remove dead code from crypt_blowfish.c

  This function has been commented out since 2011.
ext/standard/crypt.c
ext/standard/crypt_blowfish.c
ext/standard/crypt_blowfish.h
ext/standard/crypt_freesec.c
ext/standard/php_crypt_r.c
ext/standard/php_crypt_r.h