cerc-io/solidity
10
0
Fork 0
mirror of https://github.com/ethereum/solidity synced 2023-10-03 13:03:40 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Merge pull request #6454 from ethereum/docs-high-s-clar

Browse Source 
[DOCS] Explain high s value
This commit is contained in:
Chris Chinchilla 2019-04-23 13:04:55 +10:00 committed by GitHub
commit 13518820c7
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
docs/units-and-global-variables.rst
View File

@ -188,8 +188,12 @@ Mathematical and Cryptographic Functions
.. warning:: .. warning::
If you use ``ecrecover``, be aware that a valid signature can be turned into a different valid signature without requiring If you use ``ecrecover``, be aware that a valid signature can be turned into a different valid signature without
knowledge of the corresponding private key. This is usually not a problem unless you require signatures to be unique or requiring knowledge of the corresponding private key. In the Homestead hard fork, this issue was fixed
for _transaction_ signatures (see `EIP-2 <http://eips.ethereum.org/EIPS/eip-2#specification>`_), but
the ecrecover function remained unchanged.
This is usually not a problem unless you require signatures to be unique or
use them to identify items. OpenZeppelin have a `ECDSA helper library <https://docs.openzeppelin.org/docs/cryptography_ecdsa>`_ that you can use as a wrapper for ``ecrecover`` without this issue. use them to identify items. OpenZeppelin have a `ECDSA helper library <https://docs.openzeppelin.org/docs/cryptography_ecdsa>`_ that you can use as a wrapper for ``ecrecover`` without this issue.
.. note:: .. note::