The required_auths field on custom_operation was being ignored during authority checking. This commit causes it to be checked correctly, and adds a unit test verifying as much.
64 lines
2.6 KiB
C++
64 lines
2.6 KiB
C++
/*
|
|
* Copyright (c) 2015 Cryptonomex, Inc., and contributors.
|
|
*
|
|
* The MIT License
|
|
*
|
|
* Permission is hereby granted, free of charge, to any person obtaining a copy
|
|
* of this software and associated documentation files (the "Software"), to deal
|
|
* in the Software without restriction, including without limitation the rights
|
|
* to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
|
|
* copies of the Software, and to permit persons to whom the Software is
|
|
* furnished to do so, subject to the following conditions:
|
|
*
|
|
* The above copyright notice and this permission notice shall be included in
|
|
* all copies or substantial portions of the Software.
|
|
*
|
|
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
|
|
* IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
|
|
* FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
|
|
* AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
|
|
* LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
|
|
* OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
|
|
* THE SOFTWARE.
|
|
*/
|
|
|
|
#pragma once
|
|
#include <graphene/chain/protocol/base.hpp>
|
|
|
|
namespace graphene { namespace chain {
|
|
|
|
/**
|
|
* @brief provides a generic way to add higher level protocols on top of witness consensus
|
|
* @ingroup operations
|
|
*
|
|
* There is no validation for this operation other than that required auths are valid and a fee
|
|
* is paid that is appropriate for the data contained.
|
|
*/
|
|
struct custom_operation : public base_operation
|
|
{
|
|
struct fee_parameters_type {
|
|
uint64_t fee = GRAPHENE_BLOCKCHAIN_PRECISION;
|
|
uint32_t price_per_kbyte = 10;
|
|
};
|
|
|
|
asset fee;
|
|
account_id_type payer;
|
|
flat_set<account_id_type> required_auths;
|
|
uint16_t id = 0;
|
|
vector<char> data;
|
|
|
|
account_id_type fee_payer()const { return payer; }
|
|
void validate()const;
|
|
share_type calculate_fee(const fee_parameters_type& k)const;
|
|
void get_required_active_authorities( flat_set<account_id_type>& auths )const {
|
|
auths.insert( required_auths.begin(), required_auths.end() );
|
|
}
|
|
};
|
|
|
|
} } // namespace graphene::chain
|
|
|
|
FC_REFLECT( graphene::chain::custom_operation::fee_parameters_type, (fee)(price_per_kbyte) )
|
|
FC_REFLECT( graphene::chain::custom_operation, (fee)(payer)(required_auths)(id)(data) )
|
|
|
|
GRAPHENE_EXTERNAL_SERIALIZATION( extern, graphene::chain::custom_operation::fee_parameters_type )
|
|
GRAPHENE_EXTERNAL_SERIALIZATION( extern, graphene::chain::custom_operation )
|