From c156adac53b448780c7c9ef366c12237a7cea8e1 Mon Sep 17 00:00:00 2001 From: Braydon Fuller Date: Mon, 13 Jul 2015 12:22:55 -0400 Subject: [PATCH 1/3] doc: add documentation for shared library libbitcoinconsensus --- doc/README.md | 1 + doc/shared-libraries.md | 41 +++++++++++++++++++++++++++++++++++++++++ 2 files changed, 42 insertions(+) create mode 100644 doc/shared-libraries.md diff --git a/doc/README.md b/doc/README.md index ecddf50d4..7b0c39d38 100644 --- a/doc/README.md +++ b/doc/README.md @@ -58,6 +58,7 @@ The Bitcoin repo's [root README](https://github.com/bitcoin/bitcoin/blob/master/ - [Translation Strings Policy](translation_strings_policy.md) - [Unit Tests](unit-tests.md) - [Unauthenticated REST Interface](REST-interface.md) +- [Shared Libraries](shared-libraries.md) - [BIPS](bips.md) - [Dnsseed Policy](dnsseed-policy.md) diff --git a/doc/shared-libraries.md b/doc/shared-libraries.md new file mode 100644 index 000000000..c8f4939e7 --- /dev/null +++ b/doc/shared-libraries.md @@ -0,0 +1,41 @@ +Shared Libraries +================ + +## bitcoinconsensus + +The purpose of this library is to make the verification functionality that is critical to Bitcoin's consensus available to other applications, e.g. to language bindings. + +### API + +The interface is defined in the C header `bitcoinconsensus.h` located in `src/script/bitcoinconsensus.h`. + +#### Version + +`bitcoinconsensus_version` returns an `unsigned int` with the the API version *(currently at an experimental `0`)*. + +#### Script Validation + +`bitcoinconsensus_verify_script` returns an `int` with the status of the verification. It will be `1` if the input script correctly spends the previous output `scriptPubKey`. + +##### Parameters +- `const unsigned char *scriptPubKey` - The previous output script that encumbers spending. +- `unsigned int scriptPubKeyLen` - The number of bytes for the `scriptPubKey`. +- `const unsigned char *txTo` - The transaction with the input that is spending the previous output. +- `unsigned int txToLen` - The number of bytes for the `txTo`. +- `unsigned int nIn` - The index of the input in `txTo` that spends the `scriptPubKey`. +- `unsigned int flags` - The script validation flags *(see below)*. +- `bitcoinconsensus_error* err` - Will have the error/success code for the operation *(see below)*. + +##### Script Flags +- `bitcoinconsensus_SCRIPT_FLAGS_VERIFY_NONE` +- `bitcoinconsensus_SCRIPT_FLAGS_VERIFY_P2SH` - Evaluate P2SH ([BIP16](https://github.com/bitcoin/bips/blob/master/bip-0016.mediawiki)) subscripts +- `bitcoinconsensus_SCRIPT_FLAGS_VERIFY_DERSIG` - Enforce strict DER ([BIP66](https://github.com/bitcoin/bips/blob/master/bip-0066.mediawiki)) compliance + +##### Errors +- `bitcoinconsensus_ERR_OK` +- `bitcoinconsensus_ERR_TX_INDEX` - An invalid index for `txTo` +- `bitcoinconsensus_ERR_TX_SIZE_MISMATCH` - `txToLen` did not match with the size of `txTo` +- `bitcoinconsensus_ERR_DESERIALIZE` - An error deserializing `txTo` + +### Example Implementations +- [node-libbitcoinconsensus](https://github.com/bitpay/node-libbitcoinconsensus) (Node.js Bindings) From b1bac4ea495a16dc0661fcf1af90ff70191f8e74 Mon Sep 17 00:00:00 2001 From: Braydon Fuller Date: Tue, 14 Jul 2015 21:44:03 -0400 Subject: [PATCH 2/3] Add description for bitcoinconsensus_ERR_OK --- doc/shared-libraries.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/shared-libraries.md b/doc/shared-libraries.md index c8f4939e7..4b8279e8a 100644 --- a/doc/shared-libraries.md +++ b/doc/shared-libraries.md @@ -32,7 +32,7 @@ The interface is defined in the C header `bitcoinconsensus.h` located in `src/s - `bitcoinconsensus_SCRIPT_FLAGS_VERIFY_DERSIG` - Enforce strict DER ([BIP66](https://github.com/bitcoin/bips/blob/master/bip-0066.mediawiki)) compliance ##### Errors -- `bitcoinconsensus_ERR_OK` +- `bitcoinconsensus_ERR_OK` - No errors with input parameters *(see the return value of `bitcoinconsensus_verify_script` for the verification status)* - `bitcoinconsensus_ERR_TX_INDEX` - An invalid index for `txTo` - `bitcoinconsensus_ERR_TX_SIZE_MISMATCH` - `txToLen` did not match with the size of `txTo` - `bitcoinconsensus_ERR_DESERIALIZE` - An error deserializing `txTo` From 3361edd01046aab137928d1e6e2468dbfc4b946e Mon Sep 17 00:00:00 2001 From: Braydon Fuller Date: Wed, 15 Jul 2015 08:40:18 -0400 Subject: [PATCH 3/3] doc: Add NBitcoin to bitcoinconsensus examples --- doc/shared-libraries.md | 1 + 1 file changed, 1 insertion(+) diff --git a/doc/shared-libraries.md b/doc/shared-libraries.md index 4b8279e8a..1fc32112c 100644 --- a/doc/shared-libraries.md +++ b/doc/shared-libraries.md @@ -38,4 +38,5 @@ The interface is defined in the C header `bitcoinconsensus.h` located in `src/s - `bitcoinconsensus_ERR_DESERIALIZE` - An error deserializing `txTo` ### Example Implementations +- [NBitcoin](https://github.com/NicolasDorier/NBitcoin/blob/master/NBitcoin/Script.cs#L814) (.NET Bindings) - [node-libbitcoinconsensus](https://github.com/bitpay/node-libbitcoinconsensus) (Node.js Bindings)