2017-07-12 5 views
0

私は、次の機能があります:私は以下のような関数を呼び出すときに、私は長さが2400のハッシュを取り戻すためにC++ PBKDF2問題

void PBKDF2_HMAC_SHA_512_string(const char* pass, const char* salt, int32_t iterations, uint32_t HashLength, char* out) { 
    unsigned int i; 
    HashLength = HashLength/2; 
    unsigned char* digest = new unsigned char[HashLength]; 
    PKCS5_PBKDF2_HMAC(pass, strlen(pass), (const unsigned char*)salt, strlen(salt), iterations, EVP_sha512(), HashLength, digest); 
    for (i = 0; i < sizeof(digest); i++) { 
     sprintf(out + (i * 2), "%02x", 255 & digest[i]); 
    } 
} 

期待を、しかし、それは私に16を返します。

char PBKDF2Hash[1025]; //\0 terminating space? 
memset(PBKDF2Hash, 0, sizeof(PBKDF2Hash)); 
PBKDF2_HMAC_SHA_512_string("Password", "", 3500, 1024, PBKDF2Hash); 
//PBKDF2Hash is now always 16 long -> strlen(PBKDF2Hash), 
//while I expect it to be 2400 long? 
//How is this possible and why is this happening? 
//I can't figure it out 
+0

を引き起こす、PBKDF2_HMAC_SHA_512_stringで削除されていない、ということです。 – Mgetz

+0

@Mgetz、私の意見では、どのように動作するか知っています。私は1024長さの文字列を返す作業をしていましたが、私はMSVCに切り替えましたが、今は16文字の長さの文字列を返していますか?だから、これはどうやって起こることができ、私はPBKDF2について何を誤解しましたか? –

+0

"派生キーが出力されます。出力バッファーのサイズはkeylenを介して指定されます。"バッファーのサイズは、ハッシュに基づく結果キーのサイズではありません。違いは、 'new'と配列の構文を使うとき、ほとんどの場合Windowsはメモリをゼロにします。あるいは、以前にゼロにされたページを取得したばかりかもしれません。 – Mgetz

答えて

2

digestpointerなので、sizeof(digest)は配列の長さを与えません。異なるプラットフォームによっては、sizeof(digest)があなたに4または8を与えるかもしれませんが、これはあなたが望むものではありません。おそらくfor (i = 0; i < HashLength; i++)を使用してください。あなたのコードの

関係のない別の問題は、digestが、私はその最初の研究を示唆している、あなたはPBKDF2がどのように動作するかの誤解があると思うmemory leak

+0

Okeで見ることができるように、meestを使ってハッシュをクリアしますが、あなたのループ内で 'digest [i]'をどのように変更すればよいでしょう範囲を「HashLength」で変更することを参照してください。 'digest [i]'は 'HashLength'と同じ範囲には届かないので?関数の最後に 'delete digest'を使うべきですか? –

+1

@TVAvanHesteren、私は従いません。変更ダイジェスト[i]が意味することは、ループ内で変更されているとは思われません。 –

+0

あなたはforループの上限を大きくするように教えてくれますか?ループ内で関数はdigest [i]も使用しますが、ループ自体のダイジェストで上限をHashLengthに変更するとどうなりますか? –