Skip to content

Commit af2033b

Browse files
committed
Merge branch 'PHP-7.2' into PHP-7.3
* PHP-7.2: Fix #78510: Partially uninitialized buffer returned by sodium_crypto_generichash_init()
2 parents 60cfa57 + 8ead779 commit af2033b

File tree

3 files changed

+19
-0
lines changed

3 files changed

+19
-0
lines changed

NEWS

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,10 @@ PHP NEWS
3333
- PDO_MySQL:
3434
. Fixed bug #41997 (SP call yields additional empty result set). (cmb)
3535

36+
- sodium:
37+
. Fixed bug #78510 (Partially uninitialized buffer returned by
38+
sodium_crypto_generichash_init()). (Frank Denis, cmb)
39+
3640
29 Aug 2019, PHP 7.3.9
3741

3842
- Core:

ext/sodium/libsodium.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -935,6 +935,7 @@ PHP_FUNCTION(sodium_crypto_generichash_init)
935935
zend_throw_exception(sodium_exception_ce, "unsupported key length", 0);
936936
return;
937937
}
938+
memset(&state_tmp, 0, sizeof state_tmp);
938939
if (crypto_generichash_init((void *) &state_tmp, key, (size_t) key_len,
939940
(size_t) hash_len) != 0) {
940941
zend_throw_exception(sodium_exception_ce, "internal error", 0);

ext/sodium/tests/bug78510.phpt

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
--TEST--
2+
Bug #78510 (Partially uninitialized buffer returned by sodium_crypto_generichash_init())
3+
--SKIPIF--
4+
<?php
5+
if (!extension_loaded("sodium")) print "skip extension not loaded";
6+
?>
7+
--FILE--
8+
<?php
9+
$key = hex2bin('36be2998c85757e98c1abf3687c8db3a849a393701c05454023d9aba1096fd47');
10+
$y = sodium_crypto_generichash_init($key, 64);
11+
var_dump(bin2hex($y));
12+
?>
13+
--EXPECT--
14+
string(768) "48e9bdf267e6096a3ba7ca8485ae67bb2bf894fe72f36e3cf1361d5f3af54fa5d182e6ad7f520e511f6c3e2b8c68059b6bbd41fbabd9831f79217e1319cde05b000000000000000000000000000000000000000000000000000000000000000036be2998c85757e98c1abf3687c8db3a849a393701c05454023d9aba1096fd4700000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000008000000000000000000000000000000000000000000000000000000000000000"

0 commit comments

Comments
 (0)