[webkit-reviews] review granted: [Bug 227656] [OpenSSL] Implement HKDF and PBKDF2 support : [Attachment 433097] Patch

[bugzilla-daemon at webkit.org]

Fujii Hironori <Hironori.Fujii at sony.com> has granted Yoshiaki Jitsukawa
<yoshiaki.jitsukawa at sony.com>'s request for review:
Bug 227656: [OpenSSL] Implement HKDF and PBKDF2 support
https://bugs.webkit.org/show_bug.cgi?id=227656

Attachment 433097: Patch

https://bugs.webkit.org/attachment.cgi?id=433097&action=review




--- Comment #3 from Fujii Hironori <Hironori.Fujii at sony.com> ---
Comment on attachment 433097
  --> https://bugs.webkit.org/attachment.cgi?id=433097
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=433097&action=review

> Source/WebCore/crypto/openssl/CryptoAlgorithmPBKDF2OpenSSL.cpp:49
> +    if (PKCS5_PBKDF2_HMAC((const char*)key.key().data(), key.key().size(),
parameters.saltVector().data(), parameters.saltVector().size(),
parameters.iterations, algorithm, output.size(), output.data()) <= 0)

Even though WebKit code style guideline doesn't mention, some reviewer dislikes
C-style case. You should use C++-style cast. Unfortunately, it's
reinterpret_cast in this case.