GlcoinCore/elements
1
0
Fork 0
mirror of https://github.com/ElementsProject/elements.git synced 2026-08-18 13:17:55 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

CA: Standardize prefixes so that value and asset commitments use the same constant values for null/empty and explicit values.

Browse source
This commit is contained in:
Mark Friedenbach 2017-03-08 17:25:41 -08:00 • committed by Gregory Sanders
parent ba3926500f
commit 8e288cbb66
2 changed files with 44 additions and 14 deletions
Download patch file Download diff file Expand all files Collapse all files

6
src/primitives/transaction.cpp
View file

@ -11,8 +11,7 @@
void CTxOutAsset::SetNull()
{
vchAssetTag.resize(1);
vchAssetTag[0] = 0xff;
vchAssetTag.clear();
vchSurjectionproof.clear();
}
@ -32,8 +31,7 @@ CTxOutValue::CTxOutValue(CAmount nAmountIn)
void CTxOutValue::SetNull()
{
vchCommitment.resize(1);
vchCommitment[0] = 0xff;
vchCommitment.clear();
}
bool CTxOutValue::IsValid() const

52
src/primitives/transaction.h
View file

@ -40,14 +40,37 @@ public:
template <typename Stream, typename Operation>
inline void SerializationOp(Stream& s, Operation ser_action) {
vchAssetTag.resize(nAssetTagSize);
READWRITE(REF(CFlatData(&vchAssetTag[0], &vchAssetTag[nAssetTagSize])));
unsigned char version = vchAssetTag.empty()? 0: vchAssetTag[0];
READWRITE(version);
if (ser_action.ForRead()) {
switch (version) {
/* Null */
case 0:
vchAssetTag.clear();
return;
/* Explicit asset */
case 1:
/* Trust-me! asset generation */
case 0xff:
/* Confidential asset */
case 10:
case 11:
vchAssetTag.resize(nAssetTagSize);
break;
default:
throw std::ios_base::failure("Unrecognized serialization prefix");
return;
}
vchAssetTag[0] = version;
}
if (vchAssetTag.size() > 1)
READWRITE(REF(CFlatData(&vchAssetTag[1], &vchAssetTag[vchAssetTag.size()])));
// The surjection proof is serialized as part of the witness data
}
bool IsNull() const
{
return vchAssetTag.empty() || vchAssetTag[0]==0xff;
return vchAssetTag.empty();
}
void SetNull();
@ -99,11 +122,15 @@ public:
template <typename Stream, typename Operation>
inline void SerializationOp(Stream& s, Operation ser_action) {
// We only serialize the value commitment here.
// The ECDH key and range proof are serialized through CTxOutWitnessSerializer.
READWRITE(vchCommitment.front());
unsigned char version = vchCommitment.empty()? 0: vchCommitment[0];
READWRITE(version);
if (ser_action.ForRead()) {
switch (vchCommitment.front()) {
switch (version) {
/* Null */
case 0:
vchCommitment.clear();
return;
/* Explicit value */
case 1:
vchCommitment.resize(nExplicitSize);
break;
@ -111,20 +138,25 @@ public:
case 2:
case 3:
break;
/* Committed value */
case 8:
case 9:
vchCommitment.resize(nCommittedSize);
break;
default:
vchCommitment.resize(1);
throw std::ios_base::failure("Unrecognized serialization prefix");
return;
}
vchCommitment[0] = version;
}
READWRITE(REF(CFlatData(&vchCommitment[1], &vchCommitment[vchCommitment.size()])));
if (vchCommitment.size() > 1)
READWRITE(REF(CFlatData(&vchCommitment[1], &vchCommitment[vchCommitment.size()])));
// We only serialize the value commitment here.
// The ECDH key and range proof are serialized through CTxOutWitnessSerializer.
}
void SetNull();
bool IsNull() const { return vchCommitment[0] == 0xff; }
bool IsNull() const { return vchCommitment.empty(); }
bool IsValid() const;