From 37ece3668538831927b4e65b1d7d1ef98647a130 Mon Sep 17 00:00:00 2001 From: Nicolas Gotchac Date: Tue, 21 Mar 2017 13:56:32 +0100 Subject: [PATCH 1/9] Fix Password Dialog forms style issue (#4968) --- js/src/modals/PasswordManager/passwordManager.css | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/js/src/modals/PasswordManager/passwordManager.css b/js/src/modals/PasswordManager/passwordManager.css index f0dfdd93f..f16f4f369 100644 --- a/js/src/modals/PasswordManager/passwordManager.css +++ b/js/src/modals/PasswordManager/passwordManager.css @@ -77,7 +77,8 @@ } .form { + box-sizing: border-box; margin-top: 0; - padding: 0.75rem 1.5rem 1.5rem 1.5rem; + padding: 0.75rem 1.5rem 1.5rem; background-color: rgba(255, 255, 255, 0.05); } From 213f61e82602f78601e40bed63934f1328da93bb Mon Sep 17 00:00:00 2001 From: GitLab Build Bot Date: Tue, 21 Mar 2017 13:17:48 +0000 Subject: [PATCH 2/9] [ci skip] js-precompiled 20170321-131503 --- Cargo.lock | 2 +- js/package.json | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/Cargo.lock b/Cargo.lock index 1543c9e6b..8f6636b45 100644 --- a/Cargo.lock +++ b/Cargo.lock @@ -1699,7 +1699,7 @@ dependencies = [ [[package]] name = "parity-ui-precompiled" version = "1.4.0" -source = "git+https://github.com/ethcore/js-precompiled.git#7039b5b8e44196718333dc3fcdfcadae2a97c7fd" +source = "git+https://github.com/ethcore/js-precompiled.git#18fe217afb0c47fc54f9972fe87b8592b330a2e7" dependencies = [ "parity-dapps-glue 1.4.0 (registry+https://github.com/rust-lang/crates.io-index)", ] diff --git a/js/package.json b/js/package.json index 9d50ebe07..c8de94346 100644 --- a/js/package.json +++ b/js/package.json @@ -1,6 +1,6 @@ { "name": "parity.js", - "version": "1.7.19", + "version": "1.7.20", "main": "release/index.js", "jsnext:main": "src/index.js", "author": "Parity Team ", From 9b9cd451d1254e4b5573338155a189cc978ab5c4 Mon Sep 17 00:00:00 2001 From: Nicolas Gotchac Date: Tue, 21 Mar 2017 15:36:47 +0100 Subject: [PATCH 3/9] Fix references to api outside of `parity.js` (#4981) --- js/src/dapps/registry/Application/application.js | 4 ++-- js/src/dapps/tokenreg/Accounts/actions.js | 2 +- js/src/dapps/tokenreg/Status/actions.js | 3 +-- js/src/dapps/tokenreg/Status/status.js | 3 +-- js/src/dapps/tokenreg/Tokens/actions.js | 3 ++- 5 files changed, 7 insertions(+), 8 deletions(-) diff --git a/js/src/dapps/registry/Application/application.js b/js/src/dapps/registry/Application/application.js index e4f754dd6..33b3662b1 100644 --- a/js/src/dapps/registry/Application/application.js +++ b/js/src/dapps/registry/Application/application.js @@ -23,6 +23,7 @@ import CircularProgress from 'material-ui/CircularProgress'; import { Card, CardText } from 'material-ui/Card'; import { nullableProptype } from '~/util/proptypes'; +import { api } from '../parity'; import styles from './application.css'; import Accounts from '../Accounts'; @@ -39,7 +40,7 @@ export default class Application extends Component { }; getChildContext () { - return { muiTheme, api: window.parity.api }; + return { muiTheme, api }; } static propTypes = { @@ -49,7 +50,6 @@ export default class Application extends Component { }; render () { - const { api } = window.parity; const { contract, fee } = this.props; let warning = null; diff --git a/js/src/dapps/tokenreg/Accounts/actions.js b/js/src/dapps/tokenreg/Accounts/actions.js index e561334c9..538f1a831 100644 --- a/js/src/dapps/tokenreg/Accounts/actions.js +++ b/js/src/dapps/tokenreg/Accounts/actions.js @@ -14,7 +14,7 @@ // You should have received a copy of the GNU General Public License // along with Parity. If not, see . -const { api } = window.parity; +import { api } from '../parity'; export const SET_ACCOUNTS = 'SET_ACCOUNTS'; export const setAccounts = (accounts) => ({ diff --git a/js/src/dapps/tokenreg/Status/actions.js b/js/src/dapps/tokenreg/Status/actions.js index 9d789ca34..deb473d9b 100644 --- a/js/src/dapps/tokenreg/Status/actions.js +++ b/js/src/dapps/tokenreg/Status/actions.js @@ -17,8 +17,7 @@ import Contracts from '~/contracts'; import { loadToken, setTokenPending, deleteToken, setTokenData } from '../Tokens/actions'; - -const { api } = window.parity; +import { api } from '../parity'; export const SET_LOADING = 'SET_LOADING'; export const setLoading = (isLoading) => ({ diff --git a/js/src/dapps/tokenreg/Status/status.js b/js/src/dapps/tokenreg/Status/status.js index db007318f..6f77f386b 100644 --- a/js/src/dapps/tokenreg/Status/status.js +++ b/js/src/dapps/tokenreg/Status/status.js @@ -16,12 +16,11 @@ import React, { Component, PropTypes } from 'react'; +import { api } from '../parity'; import Chip from '../Chip'; import styles from './status.css'; -const { api } = window.parity; - export default class Status extends Component { static propTypes = { address: PropTypes.string.isRequired, diff --git a/js/src/dapps/tokenreg/Tokens/actions.js b/js/src/dapps/tokenreg/Tokens/actions.js index 6b7983b0d..57581e651 100644 --- a/js/src/dapps/tokenreg/Tokens/actions.js +++ b/js/src/dapps/tokenreg/Tokens/actions.js @@ -16,8 +16,9 @@ import { URL_TYPE } from '../Inputs/validation'; import { getTokenTotalSupply, urlToHash } from '../utils'; +import { api } from '../parity'; -const { bytesToHex } = window.parity.api.util; +const { bytesToHex } = api.util; export const SET_TOKENS_LOADING = 'SET_TOKENS_LOADING'; export const setTokensLoading = (isLoading) => ({ From 1a6f23ad2bfaf135628a525c02f1b82601be8407 Mon Sep 17 00:00:00 2001 From: GitLab Build Bot Date: Tue, 21 Mar 2017 14:58:06 +0000 Subject: [PATCH 4/9] [ci skip] js-precompiled 20170321-145513 --- Cargo.lock | 2 +- js/package.json | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/Cargo.lock b/Cargo.lock index 8f6636b45..014704746 100644 --- a/Cargo.lock +++ b/Cargo.lock @@ -1699,7 +1699,7 @@ dependencies = [ [[package]] name = "parity-ui-precompiled" version = "1.4.0" -source = "git+https://github.com/ethcore/js-precompiled.git#18fe217afb0c47fc54f9972fe87b8592b330a2e7" +source = "git+https://github.com/ethcore/js-precompiled.git#5bb0ae216eec333a00d6fac3e5f6b253ca5be9c0" dependencies = [ "parity-dapps-glue 1.4.0 (registry+https://github.com/rust-lang/crates.io-index)", ] diff --git a/js/package.json b/js/package.json index c8de94346..ac6513014 100644 --- a/js/package.json +++ b/js/package.json @@ -1,6 +1,6 @@ { "name": "parity.js", - "version": "1.7.20", + "version": "1.7.21", "main": "release/index.js", "jsnext:main": "src/index.js", "author": "Parity Team ", From cb881108c3e085fb6d1bfc9b6cec23cd672d429b Mon Sep 17 00:00:00 2001 From: Jaco Greeff Date: Tue, 21 Mar 2017 16:58:52 +0100 Subject: [PATCH 5/9] eth_sign where account === undefined (#4964) * Update for case where account === undefined * Update tests to not mask account === undefined * default account = {} where undefined (thanks @tomusdrw) --- .../transactionPendingFormConfirm.js | 13 ++++++++----- .../transactionPendingFormConfirm.spec.js | 6 +++++- 2 files changed, 13 insertions(+), 6 deletions(-) diff --git a/js/src/views/Signer/components/TransactionPendingForm/TransactionPendingFormConfirm/transactionPendingFormConfirm.js b/js/src/views/Signer/components/TransactionPendingForm/TransactionPendingFormConfirm/transactionPendingFormConfirm.js index 13bf27876..8a20b3332 100644 --- a/js/src/views/Signer/components/TransactionPendingForm/TransactionPendingFormConfirm/transactionPendingFormConfirm.js +++ b/js/src/views/Signer/components/TransactionPendingForm/TransactionPendingFormConfirm/transactionPendingFormConfirm.js @@ -27,7 +27,7 @@ import styles from './transactionPendingFormConfirm.css'; export default class TransactionPendingFormConfirm extends Component { static propTypes = { - account: PropTypes.object.isRequired, + account: PropTypes.object, address: PropTypes.string.isRequired, disabled: PropTypes.bool, isSending: PropTypes.bool.isRequired, @@ -36,6 +36,7 @@ export default class TransactionPendingFormConfirm extends Component { }; static defaultProps = { + account: {}, focus: false }; @@ -80,7 +81,7 @@ export default class TransactionPendingFormConfirm extends Component { getPasswordHint () { const { account } = this.props; - const accountHint = account && account.meta && account.meta.passwordHint; + const accountHint = account.meta && account.meta.passwordHint; if (accountHint) { return accountHint; @@ -149,14 +150,16 @@ export default class TransactionPendingFormConfirm extends Component { const { account } = this.props; const { password } = this.state; - if (account && account.hardware) { + if (account.hardware) { return null; } + const isAccount = account.uuid; + return ( { it('renders the password', () => { expect(instance.renderPassword()).not.to.be.null; }); + + it('renders the hint', () => { + expect(instance.renderHint()).to.be.null; + }); }); }); From c7e699223987f0af1c20d40cdaa64d6841a94d77 Mon Sep 17 00:00:00 2001 From: Nicolas Gotchac Date: Tue, 21 Mar 2017 17:01:33 +0100 Subject: [PATCH 6/9] Try to fix WS race condition connection (#4976) * Try to fix wrong token logic * Linting --- js/src/secureApi.js | 43 ++++++++++++++++++++++++++----------------- 1 file changed, 26 insertions(+), 17 deletions(-) diff --git a/js/src/secureApi.js b/js/src/secureApi.js index 60983fce1..9f4a6c078 100644 --- a/js/src/secureApi.js +++ b/js/src/secureApi.js @@ -241,7 +241,7 @@ export default class SecureApi extends Api { this.transport.updateToken(token, false); log.debug('connecting with token', token); - return this.transport.connect() + const connectPromise = this.transport.connect() .then(() => { log.debug('connected with', token); @@ -258,26 +258,35 @@ export default class SecureApi extends Api { log.debug('did not connect ; error', error); } - // Check if the Node is up - return this.isNodeUp() - .then((isNodeUp) => { - // If it's not up, try again in a few... - if (!isNodeUp) { - const timeout = this.transport.retryTimeout; + return false; + }); - log.debug('node is not up ; will try again in', timeout, 'ms'); + return Promise + .all([ + connectPromise, + this.isNodeUp() + ]) + .then(([ connected, isNodeUp ]) => { + if (connected) { + return true; + } - return new Promise((resolve, reject) => { - window.setTimeout(() => { - this._connectWithToken(token).then(resolve).catch(reject); - }, timeout); - }); - } + // If it's not up, try again in a few... + if (!isNodeUp) { + const timeout = this.transport.retryTimeout; - // The token is invalid - log.debug('tried with a wrong token', token); - return false; + log.debug('node is not up ; will try again in', timeout, 'ms'); + + return new Promise((resolve, reject) => { + window.setTimeout(() => { + this._connectWithToken(token).then(resolve).catch(reject); + }, timeout); }); + } + + // The token is invalid + log.debug('tried with a wrong token', token); + return false; }); } From 030d01102ceeb9d37248a0aa28d9546a172aea9c Mon Sep 17 00:00:00 2001 From: Nicolas Gotchac Date: Tue, 21 Mar 2017 17:02:41 +0100 Subject: [PATCH 7/9] Fix MethodDecoding for Arrays (#4977) * Fix TypedInputs * Remove unused code in inputQueries * Use TypedInputs in Contract Events * Linting * Don't re-render events every second... --- js/src/ui/CopyToClipboard/copyToClipboard.css | 2 +- js/src/ui/Form/TypedInput/typedInput.css | 1 - js/src/ui/Form/TypedInput/typedInput.js | 49 ++++++++++++++----- js/src/ui/MethodDecoding/methodDecoding.js | 12 +---- js/src/util/abi.js | 2 +- js/src/views/Contract/Events/Event/event.js | 47 +++++------------- js/src/views/Contract/Events/events.js | 6 +++ js/src/views/Contract/Queries/inputQuery.js | 25 +--------- js/src/views/Contract/Queries/queries.js | 24 +-------- js/src/views/WriteContract/writeContract.js | 4 ++ 10 files changed, 64 insertions(+), 108 deletions(-) diff --git a/js/src/ui/CopyToClipboard/copyToClipboard.css b/js/src/ui/CopyToClipboard/copyToClipboard.css index 3b6db7843..4e2bb8cc4 100644 --- a/js/src/ui/CopyToClipboard/copyToClipboard.css +++ b/js/src/ui/CopyToClipboard/copyToClipboard.css @@ -28,6 +28,6 @@ text-overflow: ellipsis; } -.container { +.container > * { display: flex; } diff --git a/js/src/ui/Form/TypedInput/typedInput.css b/js/src/ui/Form/TypedInput/typedInput.css index 88b269da3..1281c203c 100644 --- a/js/src/ui/Form/TypedInput/typedInput.css +++ b/js/src/ui/Form/TypedInput/typedInput.css @@ -25,7 +25,6 @@ color: rgba(255, 255, 255, 0.498039); -webkit-user-select: none; font-size: 12px; - top: 11px; position: relative; } } diff --git a/js/src/ui/Form/TypedInput/typedInput.js b/js/src/ui/Form/TypedInput/typedInput.js index 16b3b0ca1..a6a84ad21 100644 --- a/js/src/ui/Form/TypedInput/typedInput.js +++ b/js/src/ui/Form/TypedInput/typedInput.js @@ -24,6 +24,7 @@ import AddIcon from 'material-ui/svg-icons/content/add'; import RemoveIcon from 'material-ui/svg-icons/content/remove'; import { fromWei, toWei } from '~/api/util/wei'; +import { bytesToHex } from '~/api/util/format'; import Input from '~/ui/Form/Input'; import InputAddressSelect from '~/ui/Form/InputAddressSelect'; import Select from '~/ui/Form/Select'; @@ -68,7 +69,8 @@ export default class TypedInput extends Component { }; componentWillMount () { - const { isEth, value } = this.props; + const { isEth } = this.props; + const value = this.getValue(); if (typeof isEth === 'boolean' && value) { // Remove formatting commas @@ -95,14 +97,15 @@ export default class TypedInput extends Component { const { type } = param; if (type === ABI_TYPES.ARRAY) { - const { accounts, className, label, value = param.default } = this.props; + const { accounts, className, label } = this.props; const { subtype, length } = param; + const value = this.getValue() || param.default; const fixedLength = !!length; const inputs = range(length || value.length).map((_, index) => { const onChange = (inputValue) => { - const newValues = [].concat(this.props.value); + const newValues = [].concat(value); newValues[index] = inputValue; this.props.onChange(newValues); @@ -191,7 +194,14 @@ export default class TypedInput extends Component { } if (type === ABI_TYPES.BYTES) { - return this.renderDefault(); + let value = this.getValue(); + + // Convert to hex if it's an array + if (Array.isArray(value)) { + value = bytesToHex(value); + } + + return this.renderDefault(value); } // If the `isEth` prop is present (true or false) @@ -260,7 +270,7 @@ export default class TypedInput extends Component { : bnValue.toFixed(); // we need a string representation, could be >15 digits } - renderInteger (value = this.props.value, onChange = this.onChange) { + renderInteger (value = this.getValue(), onChange = this.onChange) { const { allowCopy, className, label, error, hint, min, max, readOnly } = this.props; const param = this.getParam(); @@ -268,7 +278,7 @@ export default class TypedInput extends Component { return ( { + getParam () { const { param } = this.props; if (typeof param === 'string') { @@ -450,4 +462,15 @@ export default class TypedInput extends Component { return param; } + + /** + * If the value comes from `decodeMethodInput`, + * it can be an object of the shape: + * { value: Object, type: String } + */ + getValue (value = this.props.value) { + return value && value.value + ? value.value + : value; + } } diff --git a/js/src/ui/MethodDecoding/methodDecoding.js b/js/src/ui/MethodDecoding/methodDecoding.js index 0bf8cce74..0aa603bca 100644 --- a/js/src/ui/MethodDecoding/methodDecoding.js +++ b/js/src/ui/MethodDecoding/methodDecoding.js @@ -560,7 +560,7 @@ class MethodDecoding extends Component { key={ index } param={ input.type } readOnly - value={ this.renderValue(input.value) } + value={ input.value } /> ); }); @@ -568,16 +568,6 @@ class MethodDecoding extends Component { return inputs; } - renderValue (value) { - const { api } = this.context; - - if (api.util.isArray(value)) { - return api.util.bytesToHex(value); - } - - return value.toString(); - } - renderTokenValue (value) { const { token } = this.props; diff --git a/js/src/util/abi.js b/js/src/util/abi.js index 34f575bb0..e2670f387 100644 --- a/js/src/util/abi.js +++ b/js/src/util/abi.js @@ -101,7 +101,7 @@ export function parseAbiType (type) { }; } - if (type === 'bytes') { + if (type === 'bytes' || type === 'fixedBytes') { return { type: BYTES_TYPE, default: '0x' diff --git a/js/src/views/Contract/Events/Event/event.js b/js/src/views/Contract/Events/Event/event.js index 64e4cdd49..3c1572b51 100644 --- a/js/src/views/Contract/Events/Event/event.js +++ b/js/src/views/Contract/Events/Event/event.js @@ -19,7 +19,7 @@ import moment from 'moment'; import React, { Component, PropTypes } from 'react'; import { FormattedMessage } from 'react-intl'; -import { IdentityIcon, IdentityName, Input, InputAddress } from '~/ui'; +import { IdentityIcon, IdentityName, TypedInput } from '~/ui'; import ShortenedHash from '~/ui/ShortenedHash'; import { txLink } from '~/3rdparty/etherscan/links'; @@ -112,41 +112,16 @@ export default class Event extends Component { } renderParam (name, param) { - const { api } = this.context; - - switch (param.type) { - case 'address': - return ( - - ); - - default: - let value; - - if (api.util.isInstanceOf(param.value, BigNumber)) { - value = param.value.toFormat(0); - } else if (api.util.isArray(param.value)) { - value = api.util.bytesToHex(param.value); - } else { - value = param.value.toString(); - } - - return ( - - ); - } + return ( + + ); } formatBlockTimestamp (block) { diff --git a/js/src/views/Contract/Events/events.js b/js/src/views/Contract/Events/events.js index a709c137a..a15565757 100644 --- a/js/src/views/Contract/Events/events.js +++ b/js/src/views/Contract/Events/events.js @@ -46,6 +46,12 @@ export default class Events extends Component { events: [] }; + shouldComponentUpdate (nextProps) { + return (nextProps.events !== this.props.events) || + (nextProps.netVersion !== this.props.netVersion) || + (nextProps.isLoading !== this.props.isLoading); + } + render () { const { events, isLoading, netVersion } = this.props; diff --git a/js/src/views/Contract/Queries/inputQuery.js b/js/src/views/Contract/Queries/inputQuery.js index 8d4bddf02..7a75727f6 100644 --- a/js/src/views/Contract/Queries/inputQuery.js +++ b/js/src/views/Contract/Queries/inputQuery.js @@ -138,10 +138,8 @@ class InputQuery extends Component { .map((out, index) => ({ name: out.name, type: out.type, - value: results[index], - display: this.renderValue(results[index]) + value: results[index] })) - .sort((outA, outB) => outA.display.length - outB.display.length) .map((out, index) => { const input = ( ); @@ -195,25 +193,6 @@ class InputQuery extends Component { ); } - renderValue (token) { - const { api } = this.context; - const { type, value } = token; - - if (value === null || value === undefined) { - return 'no data'; - } - - if (type === 'array' || type === 'fixedArray') { - return value.map((tok) => this.renderValue(tok)); - } - - if (Array.isArray(value)) { - return api.util.bytesToHex(value); - } - - return value; - } - onClick = () => { const { inputs, values } = this.state; const { contract, name, outputs, signature } = this.props; diff --git a/js/src/views/Contract/Queries/queries.js b/js/src/views/Contract/Queries/queries.js index de939a93d..eef428eed 100644 --- a/js/src/views/Contract/Queries/queries.js +++ b/js/src/views/Contract/Queries/queries.js @@ -139,15 +139,14 @@ export default class Queries extends Component { ); } - renderValue (tokenValue, output, key) { - if (typeof tokenValue === 'undefined') { + renderValue (value, output, key) { + if (typeof value === 'undefined') { return null; } const { accountsInfo } = this.props; const { name, type } = output; const label = `${name ? `${name}: ` : ''}${type}`; - const value = this.getTokenValue(tokenValue); return ( this.getTokenValue(tok)); - } - - if (Array.isArray(value)) { - return api.util.bytesToHex(value); - } - - return value; - } - _sortEntries (a, b) { return a.name.localeCompare(b.name); } diff --git a/js/src/views/WriteContract/writeContract.js b/js/src/views/WriteContract/writeContract.js index c4b08e3dd..86262820d 100644 --- a/js/src/views/WriteContract/writeContract.js +++ b/js/src/views/WriteContract/writeContract.js @@ -578,6 +578,10 @@ class WriteContract extends Component { } renderContract (contract) { + if (!contract) { + return null; + } + const { bytecode } = contract; const abi = contract.interface; From 3687a7c717b8e79e54567c432503b634c4fb4746 Mon Sep 17 00:00:00 2001 From: GitLab Build Bot Date: Tue, 21 Mar 2017 16:30:00 +0000 Subject: [PATCH 8/9] [ci skip] js-precompiled 20170321-162716 --- Cargo.lock | 2 +- js/package.json | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/Cargo.lock b/Cargo.lock index 014704746..56a1865dc 100644 --- a/Cargo.lock +++ b/Cargo.lock @@ -1699,7 +1699,7 @@ dependencies = [ [[package]] name = "parity-ui-precompiled" version = "1.4.0" -source = "git+https://github.com/ethcore/js-precompiled.git#5bb0ae216eec333a00d6fac3e5f6b253ca5be9c0" +source = "git+https://github.com/ethcore/js-precompiled.git#47da49294ad958933e85a9c4f0f2bb4df5dc47de" dependencies = [ "parity-dapps-glue 1.4.0 (registry+https://github.com/rust-lang/crates.io-index)", ] diff --git a/js/package.json b/js/package.json index ac6513014..1e770ac34 100644 --- a/js/package.json +++ b/js/package.json @@ -1,6 +1,6 @@ { "name": "parity.js", - "version": "1.7.21", + "version": "1.7.22", "main": "release/index.js", "jsnext:main": "src/index.js", "author": "Parity Team ", From 797a3e1cd9c134ff097fc67e115f20e7fdf25245 Mon Sep 17 00:00:00 2001 From: Robert Habermeier Date: Tue, 21 Mar 2017 17:36:38 +0100 Subject: [PATCH 9/9] EIP198 and built-in activation (#4926) * EIP198 and built-in activation * address review --- Cargo.lock | 1 + ethcore/Cargo.toml | 1 + ethcore/res/ethereum/foundation.json | 1 + ethcore/src/builtin.rs | 272 +++++++++++++++++++++++++-- ethcore/src/engines/mod.rs | 17 +- ethcore/src/executive.rs | 13 +- ethcore/src/lib.rs | 1 + json/src/spec/builtin.rs | 32 +++- 8 files changed, 303 insertions(+), 35 deletions(-) diff --git a/Cargo.lock b/Cargo.lock index 56a1865dc..02ac8995d 100644 --- a/Cargo.lock +++ b/Cargo.lock @@ -406,6 +406,7 @@ dependencies = [ "linked-hash-map 0.3.0 (registry+https://github.com/rust-lang/crates.io-index)", "log 0.3.6 (registry+https://github.com/rust-lang/crates.io-index)", "lru-cache 0.1.0 (registry+https://github.com/rust-lang/crates.io-index)", + "num 0.1.32 (registry+https://github.com/rust-lang/crates.io-index)", "num_cpus 1.2.0 (registry+https://github.com/rust-lang/crates.io-index)", "rand 0.3.14 (registry+https://github.com/rust-lang/crates.io-index)", "rlp 0.1.0", diff --git a/ethcore/Cargo.toml b/ethcore/Cargo.toml index cdf0135e7..d1c9d624b 100644 --- a/ethcore/Cargo.toml +++ b/ethcore/Cargo.toml @@ -44,6 +44,7 @@ ethcore-stratum = { path = "../stratum" } ethcore-bloom-journal = { path = "../util/bloom" } hardware-wallet = { path = "../hw" } stats = { path = "../util/stats" } +num = "0.1" [dependencies.hyper] git = "https://github.com/ethcore/hyper" diff --git a/ethcore/res/ethereum/foundation.json b/ethcore/res/ethereum/foundation.json index 7338b2f2b..77b0d5533 100644 --- a/ethcore/res/ethereum/foundation.json +++ b/ethcore/res/ethereum/foundation.json @@ -189,6 +189,7 @@ "0000000000000000000000000000000000000002": { "builtin": { "name": "sha256", "pricing": { "linear": { "base": 60, "word": 12 } } } }, "0000000000000000000000000000000000000003": { "builtin": { "name": "ripemd160", "pricing": { "linear": { "base": 600, "word": 120 } } } }, "0000000000000000000000000000000000000004": { "builtin": { "name": "identity", "pricing": { "linear": { "base": 15, "word": 3 } } } }, + "0000000000000000000000000000000000000005": { "builtin": { "name": "modexp", "activate_at": "0x7fffffffffffff", "pricing": { "modexp": { "divisor": 20 } } } }, "3282791d6fd713f1e94f4bfd565eaa78b3a0599d": { "balance": "1337000000000000000000" }, diff --git a/ethcore/src/builtin.rs b/ethcore/src/builtin.rs index ca3b13181..9927435dd 100644 --- a/ethcore/src/builtin.rs +++ b/ethcore/src/builtin.rs @@ -14,11 +14,16 @@ // You should have received a copy of the GNU General Public License // along with Parity. If not, see . +use std::cmp::{max, min}; +use std::io::{self, Read}; + +use byteorder::{ByteOrder, BigEndian}; use crypto::sha2::Sha256 as Sha256Digest; use crypto::ripemd160::Ripemd160 as Ripemd160Digest; use crypto::digest::Digest; -use std::cmp::min; -use util::{U256, H256, Hashable, BytesRef}; +use num::{BigUint, Zero, One}; + +use util::{U256, H256, Uint, Hashable, BytesRef}; use ethkey::{Signature, recover as ec_recover}; use ethjson; @@ -30,8 +35,8 @@ pub trait Impl: Send + Sync { /// A gas pricing scheme for built-in contracts. pub trait Pricer: Send + Sync { - /// The gas cost of running this built-in for the given size of input data. - fn cost(&self, in_size: usize) -> U256; + /// The gas cost of running this built-in for the given input data. + fn cost(&self, input: &[u8]) -> U256; } /// A linear pricing model. This computes a price using a base cost and a cost per-word. @@ -40,40 +45,94 @@ struct Linear { word: usize, } +/// A special pricing model for modular exponentiation. +struct Modexp { + divisor: usize, +} + impl Pricer for Linear { - fn cost(&self, in_size: usize) -> U256 { - U256::from(self.base) + U256::from(self.word) * U256::from((in_size + 31) / 32) + fn cost(&self, input: &[u8]) -> U256 { + U256::from(self.base) + U256::from(self.word) * U256::from((input.len() + 31) / 32) } } -/// Pricing scheme and execution definition for a built-in contract. +impl Pricer for Modexp { + fn cost(&self, input: &[u8]) -> U256 { + let mut reader = input.chain(io::repeat(0)); + let mut buf = [0; 32]; + + // read lengths as U256 here for accurate gas calculation. + let mut read_len = || { + reader.read_exact(&mut buf[..]).expect("reading from zero-extended memory cannot fail; qed"); + U256::from(H256::from_slice(&buf[..])) + }; + let base_len = read_len(); + let exp_len = read_len(); + let mod_len = read_len(); + + // floor(max(length_of_MODULUS, length_of_BASE) ** 2 * max(length_of_EXPONENT, 1) / GQUADDIVISOR) + // TODO: is saturating the best behavior here? + let m = max(mod_len, base_len); + match m.overflowing_mul(m) { + (_, true) => U256::max_value(), + (val, _) => { + match val.overflowing_mul(max(exp_len, U256::one())) { + (_, true) => U256::max_value(), + (val, _) => val / (self.divisor as u64).into() + } + } + } + } +} + +/// Pricing scheme, execution definition, and activation block for a built-in contract. +/// +/// Call `cost` to compute cost for the given input, `execute` to execute the contract +/// on the given input, and `is_active` to determine whether the contract is active. +/// +/// Unless `is_active` is true, pub struct Builtin { pricer: Box, native: Box, + activate_at: u64, } impl Builtin { /// Simple forwarder for cost. - pub fn cost(&self, s: usize) -> U256 { self.pricer.cost(s) } + pub fn cost(&self, input: &[u8]) -> U256 { self.pricer.cost(input) } /// Simple forwarder for execute. pub fn execute(&self, input: &[u8], output: &mut BytesRef) { self.native.execute(input, output) } + + /// Whether the builtin is activated at the given block number. + pub fn is_active(&self, at: u64) -> bool { at >= self.activate_at } } impl From for Builtin { fn from(b: ethjson::spec::Builtin) -> Self { - let pricer = match b.pricing { + let pricer: Box = match b.pricing { ethjson::spec::Pricing::Linear(linear) => { Box::new(Linear { base: linear.base, word: linear.word, }) } + ethjson::spec::Pricing::Modexp(exp) => { + Box::new(Modexp { + divisor: if exp.divisor == 0 { + warn!("Zero modexp divisor specified. Falling back to default."); + 10 + } else { + exp.divisor + } + }) + } }; Builtin { pricer: pricer, native: ethereum_builtin(&b.name), + activate_at: b.activate_at.map(Into::into).unwrap_or(0), } } } @@ -85,6 +144,7 @@ fn ethereum_builtin(name: &str) -> Box { "ecrecover" => Box::new(EcRecover) as Box, "sha256" => Box::new(Sha256) as Box, "ripemd160" => Box::new(Ripemd160) as Box, + "modexp" => Box::new(ModexpImpl) as Box, _ => panic!("invalid builtin name: {}", name), } } @@ -95,6 +155,7 @@ fn ethereum_builtin(name: &str) -> Box { // - ec recovery // - sha256 // - ripemd160 +// - modexp (EIP198) #[derive(Debug)] struct Identity; @@ -108,6 +169,9 @@ struct Sha256; #[derive(Debug)] struct Ripemd160; +#[derive(Debug)] +struct ModexpImpl; + impl Impl for Identity { fn execute(&self, input: &[u8], output: &mut BytesRef) { output.write(0, input); @@ -166,9 +230,76 @@ impl Impl for Ripemd160 { } } +impl Impl for ModexpImpl { + fn execute(&self, input: &[u8], output: &mut BytesRef) { + let mut reader = input.chain(io::repeat(0)); + let mut buf = [0; 32]; + + // read lengths as usize. + // ignoring the first 24 bytes might technically lead us to fall out of consensus, + // but so would running out of addressable memory! + let mut read_len = |reader: &mut io::Chain<&[u8], io::Repeat>| { + reader.read_exact(&mut buf[..]).expect("reading from zero-extended memory cannot fail; qed"); + BigEndian::read_u64(&buf[24..]) as usize + }; + + let base_len = read_len(&mut reader); + let exp_len = read_len(&mut reader); + let mod_len = read_len(&mut reader); + + // read the numbers themselves. + let mut buf = vec![0; max(mod_len, max(base_len, exp_len))]; + let mut read_num = |len| { + reader.read_exact(&mut buf[..len]).expect("reading from zero-extended memory cannot fail; qed"); + BigUint::from_bytes_be(&buf[..len]) + }; + + let base = read_num(base_len); + let exp = read_num(exp_len); + let modulus = read_num(mod_len); + + // calculate modexp: exponentiation by squaring. + fn modexp(mut base: BigUint, mut exp: BigUint, modulus: BigUint) -> BigUint { + match (base == BigUint::zero(), exp == BigUint::zero()) { + (_, true) => return BigUint::one(), // n^0 % m + (true, false) => return BigUint::zero(), // 0^n % m, n>0 + (false, false) if modulus <= BigUint::one() => return BigUint::zero(), // a^b % 1 = 0. + _ => {} + } + + let mut result = BigUint::one(); + base = base % &modulus; + + // fast path for base divisible by modulus. + if base == BigUint::zero() { return result } + while exp != BigUint::zero() { + // exp has to be on the right here to avoid move. + if BigUint::one() & &exp == BigUint::one() { + result = (result * &base) % &modulus; + } + + exp = exp >> 1; + base = (base.clone() * base) % &modulus; + } + + result + } + + // write output to given memory, left padded and same length as the modulus. + let bytes = modexp(base, exp, modulus).to_bytes_be(); + + // always true except in the case of zero-length modulus, which leads to + // output of length and value 1. + if bytes.len() <= mod_len { + let res_start = mod_len - bytes.len(); + output.write(res_start, &bytes); + } + } +} + #[cfg(test)] mod tests { - use super::{Builtin, Linear, ethereum_builtin, Pricer}; + use super::{Builtin, Linear, ethereum_builtin, Pricer, Modexp}; use ethjson; use util::{U256, BytesRef}; @@ -295,24 +426,126 @@ mod tests { assert_eq!(&o[..], &(FromHex::from_hex("ffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffff").unwrap())[..]);*/ } + #[test] + fn modexp() { + use rustc_serialize::hex::FromHex; + + let f = Builtin { + pricer: Box::new(Modexp { divisor: 20 }), + native: ethereum_builtin("modexp"), + activate_at: 0, + }; + // fermat's little theorem example. + { + let input = FromHex::from_hex("\ + 0000000000000000000000000000000000000000000000000000000000000001\ + 0000000000000000000000000000000000000000000000000000000000000020\ + 0000000000000000000000000000000000000000000000000000000000000020\ + 03\ + fffffffffffffffffffffffffffffffffffffffffffffffffffffffefffffc2e\ + fffffffffffffffffffffffffffffffffffffffffffffffffffffffefffffc2f" + ).unwrap(); + + let mut output = vec![0u8; 32]; + let expected = FromHex::from_hex("0000000000000000000000000000000000000000000000000000000000000001").unwrap(); + let expected_cost = 1638; + + f.execute(&input[..], &mut BytesRef::Fixed(&mut output[..])); + assert_eq!(output, expected); + assert_eq!(f.cost(&input[..]), expected_cost.into()); + } + + // second example from EIP: zero base. + { + let input = FromHex::from_hex("\ + 0000000000000000000000000000000000000000000000000000000000000000\ + 0000000000000000000000000000000000000000000000000000000000000020\ + 0000000000000000000000000000000000000000000000000000000000000020\ + fffffffffffffffffffffffffffffffffffffffffffffffffffffffefffffc2e\ + fffffffffffffffffffffffffffffffffffffffffffffffffffffffefffffc2f" + ).unwrap(); + + let mut output = vec![0u8; 32]; + let expected = FromHex::from_hex("0000000000000000000000000000000000000000000000000000000000000000").unwrap(); + let expected_cost = 1638; + + f.execute(&input[..], &mut BytesRef::Fixed(&mut output[..])); + assert_eq!(output, expected); + assert_eq!(f.cost(&input[..]), expected_cost.into()); + } + + // another example from EIP: zero-padding + { + let input = FromHex::from_hex("\ + 0000000000000000000000000000000000000000000000000000000000000001\ + 0000000000000000000000000000000000000000000000000000000000000002\ + 0000000000000000000000000000000000000000000000000000000000000020\ + 03\ + ffff\ + 80" + ).unwrap(); + + let mut output = vec![0u8; 32]; + let expected = FromHex::from_hex("3b01b01ac41f2d6e917c6d6a221ce793802469026d9ab7578fa2e79e4da6aaab").unwrap(); + let expected_cost = 102; + + f.execute(&input[..], &mut BytesRef::Fixed(&mut output[..])); + assert_eq!(output, expected); + assert_eq!(f.cost(&input[..]), expected_cost.into()); + } + + // zero-length modulus. + { + let input = FromHex::from_hex("\ + 0000000000000000000000000000000000000000000000000000000000000001\ + 0000000000000000000000000000000000000000000000000000000000000002\ + 0000000000000000000000000000000000000000000000000000000000000000\ + 03\ + ffff" + ).unwrap(); + + let mut output = vec![]; + let expected_cost = 0; + + f.execute(&input[..], &mut BytesRef::Flexible(&mut output)); + assert_eq!(output.len(), 0); // shouldn't have written any output. + assert_eq!(f.cost(&input[..]), expected_cost.into()); + } + } + #[test] #[should_panic] fn from_unknown_linear() { let _ = ethereum_builtin("foo"); } + #[test] + fn is_active() { + let pricer = Box::new(Linear { base: 10, word: 20} ); + let b = Builtin { + pricer: pricer as Box, + native: ethereum_builtin("identity"), + activate_at: 100_000, + }; + + assert!(!b.is_active(99_999)); + assert!(b.is_active(100_000)); + assert!(b.is_active(100_001)); + } + #[test] fn from_named_linear() { let pricer = Box::new(Linear { base: 10, word: 20 }); let b = Builtin { pricer: pricer as Box, native: ethereum_builtin("identity"), + activate_at: 1, }; - assert_eq!(b.cost(0), U256::from(10)); - assert_eq!(b.cost(1), U256::from(30)); - assert_eq!(b.cost(32), U256::from(30)); - assert_eq!(b.cost(33), U256::from(50)); + assert_eq!(b.cost(&[0; 0]), U256::from(10)); + assert_eq!(b.cost(&[0; 1]), U256::from(30)); + assert_eq!(b.cost(&[0; 32]), U256::from(30)); + assert_eq!(b.cost(&[0; 33]), U256::from(50)); let i = [0u8, 1, 2, 3]; let mut o = [255u8; 4]; @@ -327,13 +560,14 @@ mod tests { pricing: ethjson::spec::Pricing::Linear(ethjson::spec::Linear { base: 10, word: 20, - }) + }), + activate_at: None, }); - assert_eq!(b.cost(0), U256::from(10)); - assert_eq!(b.cost(1), U256::from(30)); - assert_eq!(b.cost(32), U256::from(30)); - assert_eq!(b.cost(33), U256::from(50)); + assert_eq!(b.cost(&[0; 0]), U256::from(10)); + assert_eq!(b.cost(&[0; 1]), U256::from(30)); + assert_eq!(b.cost(&[0; 32]), U256::from(30)); + assert_eq!(b.cost(&[0; 33]), U256::from(50)); let i = [0u8, 1, 2, 3]; let mut o = [255u8; 4]; diff --git a/ethcore/src/engines/mod.rs b/ethcore/src/engines/mod.rs index 2cc1ff21f..f292a06d8 100644 --- a/ethcore/src/engines/mod.rs +++ b/ethcore/src/engines/mod.rs @@ -189,19 +189,14 @@ pub trait Engine : Sync + Send { /// updating consensus state and potentially issuing a new one. fn handle_message(&self, _message: &[u8]) -> Result<(), Error> { Err(EngineError::UnexpectedMessage.into()) } + /// Attempt to get a handle to a built-in contract. + /// Only returns references to activated built-ins. // TODO: builtin contract routing - to do this properly, it will require removing the built-in configuration-reading logic // from Spec into here and removing the Spec::builtins field. - /// Determine whether a particular address is a builtin contract. - fn is_builtin(&self, a: &Address) -> bool { self.builtins().contains_key(a) } - /// Determine the code execution cost of the builtin contract with address `a`. - /// Panics if `is_builtin(a)` is not true. - fn cost_of_builtin(&self, a: &Address, input: &[u8]) -> U256 { - self.builtins().get(a).expect("queried cost of nonexistent builtin").cost(input.len()) - } - /// Execution the builtin contract `a` on `input` and return `output`. - /// Panics if `is_builtin(a)` is not true. - fn execute_builtin(&self, a: &Address, input: &[u8], output: &mut BytesRef) { - self.builtins().get(a).expect("attempted to execute nonexistent builtin").execute(input, output); + fn builtin(&self, a: &Address, block_number: ::header::BlockNumber) -> Option<&Builtin> { + self.builtins() + .get(a) + .and_then(|b| if b.is_active(block_number) { Some(b) } else { None }) } /// Find out if the block is a proposal block and should not be inserted into the DB. diff --git a/ethcore/src/executive.rs b/ethcore/src/executive.rs index db1fe6a4e..cdae8a85f 100644 --- a/ethcore/src/executive.rs +++ b/ethcore/src/executive.rs @@ -261,17 +261,22 @@ impl<'a, B: 'a + StateBackend> Executive<'a, B> { } trace!("Executive::call(params={:?}) self.env_info={:?}", params, self.info); - if self.engine.is_builtin(¶ms.code_address) { - // if destination is builtin, try to execute it + // if destination is builtin, try to execute it + if let Some(builtin) = self.engine.builtin(¶ms.code_address, self.info.number) { + // Engines aren't supposed to return builtins until activation, but + // prefer to fail rather than silently break consensus. + if !builtin.is_active(self.info.number) { + panic!("Consensus failure: engine implementation prematurely enabled built-in at {}", params.code_address); + } let default = []; let data = if let Some(ref d) = params.data { d as &[u8] } else { &default as &[u8] }; let trace_info = tracer.prepare_trace_call(¶ms); - let cost = self.engine.cost_of_builtin(¶ms.code_address, data); + let cost = builtin.cost(data); if cost <= params.gas { - self.engine.execute_builtin(¶ms.code_address, data, &mut output); + builtin.execute(data, &mut output); self.state.discard_checkpoint(); // trace only top level calls to builtins to avoid DDoS attacks diff --git a/ethcore/src/lib.rs b/ethcore/src/lib.rs index a78e2120f..75c8a80e1 100644 --- a/ethcore/src/lib.rs +++ b/ethcore/src/lib.rs @@ -106,6 +106,7 @@ extern crate ethcore_stratum; extern crate ethabi; extern crate hardware_wallet; extern crate stats; +extern crate num; #[macro_use] extern crate log; diff --git a/json/src/spec/builtin.rs b/json/src/spec/builtin.rs index 81a066585..892bf532e 100644 --- a/json/src/spec/builtin.rs +++ b/json/src/spec/builtin.rs @@ -16,6 +16,8 @@ //! Spec builtin deserialization. +use uint::Uint; + /// Linear pricing. #[derive(Debug, PartialEq, Deserialize, Clone)] pub struct Linear { @@ -25,12 +27,22 @@ pub struct Linear { pub word: usize, } +/// Pricing for modular exponentiation. +#[derive(Debug, PartialEq, Deserialize, Clone)] +pub struct Modexp { + /// Price divisor. + pub divisor: usize, +} + /// Pricing variants. #[derive(Debug, PartialEq, Deserialize, Clone)] pub enum Pricing { /// Linear pricing. #[serde(rename="linear")] Linear(Linear), + /// Pricing for modular exponentiation. + #[serde(rename="modexp")] + Modexp(Modexp), } /// Spec builtin. @@ -40,12 +52,15 @@ pub struct Builtin { pub name: String, /// Builtin pricing. pub pricing: Pricing, + /// Activation block. + pub activate_at: Option, } #[cfg(test)] mod tests { use serde_json; - use spec::builtin::{Builtin, Pricing, Linear}; + use spec::builtin::{Builtin, Pricing, Linear, Modexp}; + use uint::Uint; #[test] fn builtin_deserialization() { @@ -56,5 +71,20 @@ mod tests { let deserialized: Builtin = serde_json::from_str(s).unwrap(); assert_eq!(deserialized.name, "ecrecover"); assert_eq!(deserialized.pricing, Pricing::Linear(Linear { base: 3000, word: 0 })); + assert!(deserialized.activate_at.is_none()); + } + + #[test] + fn activate_at() { + let s = r#"{ + "name": "late_start", + "activate_at": 100000, + "pricing": { "modexp": { "divisor": 5 } } + }"#; + + let deserialized: Builtin = serde_json::from_str(s).unwrap(); + assert_eq!(deserialized.name, "late_start"); + assert_eq!(deserialized.pricing, Pricing::Modexp(Modexp { divisor: 5 })); + assert_eq!(deserialized.activate_at, Some(Uint(100000.into()))); } }