diff --git a/modules/chain/dap_chain_ch.c b/modules/chain/dap_chain_ch.c index 6c71b25332bd29758646d05ce2a4b1fb115d39db..98eef65ffe41e75028818a9269659657618a5404 100644 --- a/modules/chain/dap_chain_ch.c +++ b/modules/chain/dap_chain_ch.c @@ -619,17 +619,16 @@ static bool s_sync_in_chains_callback(void *a_arg) return false; } char *l_atom_hash_str = NULL; - l_atom_hash_str = DAP_NEW_STACK_SIZE(char, DAP_CHAIN_HASH_FAST_STR_SIZE); - dap_hash_fast_t l_atom_hash = {}; + dap_hash_fast_t l_atom_hash = { }; dap_hash_fast(l_atom, l_atom_size, &l_atom_hash); if (s_debug_more) - dap_get_data_hash_str_static(l_atom, l_atom_size, l_atom_hash_str); + l_atom_hash_str = dap_hash_fast_to_str_static(&l_atom_hash); dap_chain_atom_verify_res_t l_atom_add_res = l_chain->callback_atom_add(l_chain, l_atom, l_atom_size, &l_atom_hash, false); bool l_ack_send = false; switch (l_atom_add_res) { case ATOM_PASS: debug_if(s_debug_more, L_WARNING, "Atom with hash %s for %s:%s not accepted (code ATOM_PASS, already present)", - l_atom_hash_str, l_chain->net_name, l_chain->name); + l_atom_hash_str, l_chain->net_name, l_chain->name); l_ack_send = true; break; case ATOM_MOVE_TO_THRESHOLD: @@ -776,12 +775,9 @@ static bool s_stream_ch_packet_in(dap_stream_ch_t* a_ch, void* a_arg) l_args->addr = a_ch->stream->node; l_args->ack_req = true; memcpy(l_args->data, l_chain_pkt, l_ch_pkt->hdr.data_size); - if (s_debug_more) { - char *l_atom_hash_str; - dap_get_data_hash_str_static(l_chain_pkt->data, l_chain_pkt_data_size, l_atom_hash_str); - log_it(L_INFO, "In: CHAIN pkt: atom hash %s, size %zd, net id %" DAP_UINT64_FORMAT_U ", chain id %" DAP_UINT64_FORMAT_U, - l_atom_hash_str, l_chain_pkt_data_size, l_chain_pkt->hdr.net_id.uint64, l_chain_pkt->hdr.chain_id.uint64); - } + debug_if(s_debug_more, L_INFO, "In: CHAIN pkt: atom hash %s, size %zd, net id %" DAP_UINT64_FORMAT_U ", chain id %" DAP_UINT64_FORMAT_U, + dap_get_data_hash_str(l_chain_pkt->data, l_chain_pkt_data_size).s, l_chain_pkt_data_size, + l_chain_pkt->hdr.net_id.uint64, l_chain_pkt->hdr.chain_id.uint64); dap_proc_thread_callback_add(a_ch->stream_worker->worker->proc_queue_input, s_sync_in_chains_callback, l_args); } break; @@ -1498,11 +1494,8 @@ static bool s_stream_ch_packet_in(dap_stream_ch_t* a_ch, void* a_arg) } l_chain_pkt->hdr.data_size = l_chain_pkt_data_size; memcpy(l_args->data, l_chain_pkt, l_ch_pkt->hdr.data_size); - if (s_debug_legacy) { - char *l_atom_hash_str; - dap_get_data_hash_str_static(l_chain_pkt->data, l_chain_pkt_data_size, l_atom_hash_str); - log_it(L_INFO, "In: CHAIN_OLD pkt: atom hash %s (size %zd)", l_atom_hash_str, l_chain_pkt_data_size); - } + debug_if(s_debug_legacy, L_INFO, "In: CHAIN_OLD pkt: atom hash %s (size %zd)", + dap_get_data_hash_str(l_chain_pkt->data, l_chain_pkt_data_size).s, l_chain_pkt_data_size); dap_proc_thread_callback_add(a_ch->stream_worker->worker->proc_queue_input, s_sync_in_chains_callback, l_args); } break; diff --git a/modules/channel/chain-net-srv/dap_stream_ch_chain_net_srv.c b/modules/channel/chain-net-srv/dap_stream_ch_chain_net_srv.c index 5dcf39ce43476b5d7cfbb044f9d09d58f87b382d..66822232bd32e9e6f040a1d66de7d7b5900d516b 100644 --- a/modules/channel/chain-net-srv/dap_stream_ch_chain_net_srv.c +++ b/modules/channel/chain-net-srv/dap_stream_ch_chain_net_srv.c @@ -1124,7 +1124,6 @@ static bool s_grace_period_finish(dap_chain_net_srv_grace_usage_t *a_grace_item) } // make receipt or tx - char *l_receipt_hash_str; dap_chain_datum_tx_receipt_t *l_receipt = NULL; if (l_grace->usage->receipt_next){ l_receipt = l_grace->usage->receipt_next; @@ -1158,8 +1157,7 @@ static bool s_grace_period_finish(dap_chain_net_srv_grace_usage_t *a_grace_item) log_it(L_WARNING, "Tx already in chain, but receipt is not signed by client. Finish grace and wait receipt sign responce."); RET_WITH_DEL_A_GRACE(DAP_STREAM_CH_CHAIN_NET_SRV_PKT_TYPE_RESPONSE_ERROR_CODE_RECEIPT_NO_SIGN); } - dap_get_data_hash_str_static(l_receipt, l_receipt_size, l_receipt_hash_str); - dap_global_db_set(SRV_RECEIPTS_GDB_GROUP, l_receipt_hash_str, l_receipt, l_receipt_size, false, NULL, NULL); + dap_global_db_set(SRV_RECEIPTS_GDB_GROUP, dap_get_data_hash_str(l_receipt, l_receipt_size).s, l_receipt, l_receipt_size, false, NULL, NULL); // Form input transaction char *l_hash_str = dap_hash_fast_to_str_new(&l_grace->usage->tx_cond_hash); log_it(L_NOTICE, "Trying create input tx cond from tx %s with active receipt", l_hash_str); @@ -1447,9 +1445,7 @@ static bool s_stream_ch_packet_in(dap_stream_ch_t *a_ch, void *a_arg) } // Store receipt if any problems with transactions - char *l_receipt_hash_str; - dap_get_data_hash_str_static(l_receipt, l_receipt_size, l_receipt_hash_str); - dap_global_db_set(SRV_RECEIPTS_GDB_GROUP, l_receipt_hash_str, l_receipt, l_receipt_size, false, NULL, NULL); + dap_global_db_set(SRV_RECEIPTS_GDB_GROUP, dap_get_data_hash_str(l_receipt, l_receipt_size).s, l_receipt, l_receipt_size, false, NULL, NULL); size_t l_success_size; if (!l_usage->is_grace) { // Form input transaction diff --git a/modules/channel/chain-net/dap_stream_ch_chain_net.c b/modules/channel/chain-net/dap_stream_ch_chain_net.c index fe4c18c8626843aee823fb25758b151e16a0159a..ec9fffdf12fe254efea6d432fde89a440ddf0020 100644 --- a/modules/channel/chain-net/dap_stream_ch_chain_net.c +++ b/modules/channel/chain-net/dap_stream_ch_chain_net.c @@ -113,17 +113,12 @@ static bool s_stream_ch_packet_in(dap_stream_ch_t *a_ch, void* a_arg) dap_stream_ch_chain_net_t * l_ch_chain_net = DAP_STREAM_CH_CHAIN_NET(a_ch); if(l_ch_chain_net) { dap_stream_ch_pkt_t *l_ch_pkt = (dap_stream_ch_pkt_t *)a_arg; - if (l_ch_pkt->hdr.type == DAP_STREAM_CH_CHAIN_NET_PKT_TYPE_TEST) { - char *l_data_hash_str; - dap_get_data_hash_str_static(l_ch_pkt->data, l_ch_pkt->hdr.data_size, l_data_hash_str); - log_it(L_ATT, "Receive test data packet with hash %s", l_data_hash_str); - return false; - } - if (l_ch_pkt->hdr.data_size < sizeof(dap_stream_ch_chain_net_pkt_t)) { - log_it(L_WARNING, "Too small stream channel N packet size %u (header size %zu)", - l_ch_pkt->hdr.data_size, sizeof(dap_stream_ch_chain_net_pkt_t)); - return false; - } + if (l_ch_pkt->hdr.type == DAP_STREAM_CH_CHAIN_NET_PKT_TYPE_TEST) + return log_it(L_ATT, "Receive test data packet with hash %s", + dap_get_data_hash_str(l_ch_pkt->data, l_ch_pkt->hdr.data_size).s), false; + if (l_ch_pkt->hdr.data_size < sizeof(dap_stream_ch_chain_net_pkt_t)) + return log_it(L_WARNING, "Too small stream channel N packet size %u (header size %zu)", + l_ch_pkt->hdr.data_size, sizeof(dap_stream_ch_chain_net_pkt_t)), false; dap_stream_ch_chain_net_pkt_t *l_ch_chain_net_pkt = (dap_stream_ch_chain_net_pkt_t *)l_ch_pkt->data; if ((uint32_t)l_ch_chain_net_pkt->hdr.data_size + sizeof(dap_stream_ch_chain_net_pkt_t) > l_ch_pkt->hdr.data_size) { log_it(L_WARNING, "Too small stream channel N packet size %u (expected at least %zu)", diff --git a/modules/common/dap_chain_datum_decree.c b/modules/common/dap_chain_datum_decree.c index 5ef31fc6f91a9283bf35fcbbb5eb96abecc7b0d1..467d98c872e104cdd1e7229307f1a27e1b09600f 100644 --- a/modules/common/dap_chain_datum_decree.c +++ b/modules/common/dap_chain_datum_decree.c @@ -200,9 +200,7 @@ void dap_chain_datum_decree_dump_json(json_object *a_json_out, dap_chain_datum_d } dap_pkey_t *l_owner_pkey = /*DAP_NEW_STACK_SIZE(dap_pkey_t, l_tsd->size); memcpy(l_owner_pkey, l_tsd->data, l_tsd->size);*/ _dap_tsd_get_object(l_tsd, dap_pkey_t); - char *l_owner_pkey_str; - dap_get_data_hash_str_static(l_owner_pkey->pkey, l_owner_pkey->header.size, l_owner_pkey_str); - json_object_object_add(a_json_out, "Owner fingerprint", json_object_new_string(l_owner_pkey_str)); + json_object_object_add(a_json_out, "Owner fingerprint", json_object_new_string(dap_get_data_hash_str(l_owner_pkey->pkey, l_owner_pkey->header.size).s)); break; case DAP_CHAIN_DATUM_DECREE_TSD_TYPE_MIN_OWNER: if (l_tsd->size > sizeof(uint256_t)){ diff --git a/modules/consensus/dag-poa/dap_chain_cs_dag_poa.c b/modules/consensus/dag-poa/dap_chain_cs_dag_poa.c index b708d9d05db22af9ce2f1acdcaaae2d72cd6f7dc..c8523c0bd1a2e785214f3b5b69175514d6b28504 100644 --- a/modules/consensus/dag-poa/dap_chain_cs_dag_poa.c +++ b/modules/consensus/dag-poa/dap_chain_cs_dag_poa.c @@ -57,24 +57,20 @@ typedef struct dap_chain_cs_dag_poa_presign_callback{ void *arg; } dap_chain_cs_dag_poa_presign_callback_t; -struct round_timer_arg { +typedef struct dap_chain_cs_dag_poa_round_item { + dap_chain_hash_fast_t datum_hash; dap_chain_cs_dag_t *dag; - uint64_t round_id; UT_hash_handle hh; -}; +} dap_chain_cs_dag_poa_round_item_t; typedef struct dap_chain_cs_dag_poa_pvt { pthread_rwlock_t rounds_rwlock; - struct round_timer_arg *active_rounds; - dap_cert_t * events_sign_cert; - dap_cert_t ** auth_certs; - char * auth_certs_prefix; - uint16_t auth_certs_count; - uint16_t auth_certs_count_verify; // Number of signatures, needed for event verification - bool auto_confirmation; - bool auto_round_complete; - uint32_t confirmations_timeout; // wait signs over min value (auth_certs_count_verify) - uint32_t wait_sync_before_complete; + dap_chain_cs_dag_poa_round_item_t *event_items; + dap_cert_t *events_sign_cert, **auth_certs; + char *auth_certs_prefix; + uint16_t auth_certs_count, auth_certs_count_verify; // Number of signatures, needed for event verification + bool auto_confirmation, auto_round_complete; + uint32_t confirmations_timeout, wait_sync_before_complete; dap_chain_cs_dag_poa_presign_callback_t *callback_pre_sign; dap_interval_timer_t mempool_timer; } dap_chain_cs_dag_poa_pvt_t; @@ -83,23 +79,21 @@ typedef struct dap_chain_cs_dag_poa_pvt { static void s_callback_delete(dap_chain_cs_dag_t * a_dag); static int s_callback_new(dap_chain_t * a_chain, dap_config_t * a_chain_cfg); -static void s_poa_round_clean(void *a_arg); static int s_callback_created(dap_chain_t * a_chain, dap_config_t *a_chain_cfg); static int s_callback_event_verify(dap_chain_cs_dag_t *a_dag, dap_chain_cs_dag_event_t *a_dag_event, dap_hash_fast_t *a_event_hash); static dap_chain_cs_dag_event_t * s_callback_event_create(dap_chain_cs_dag_t * a_dag, dap_chain_datum_t * a_datum, dap_chain_hash_fast_t * a_hashes, size_t a_hashes_count, size_t* a_event_size); -static bool s_callback_round_event_to_chain(struct round_timer_arg *a_arg); +static bool s_callback_round_event_to_chain(dap_chain_cs_dag_poa_round_item_t *a_arg); static int s_callback_event_round_sync(dap_chain_cs_dag_t * a_dag, const char a_op_code, const char *a_group, const char *a_key, const void *a_value, const size_t a_value_size); static bool s_round_event_ready_minimum_check(dap_chain_cs_dag_t *a_dag, dap_chain_cs_dag_event_t *a_event, size_t a_event_size, char *a_event_hash_hex_str); -static void s_round_event_cs_done(dap_chain_cs_dag_t * a_dag, uint64_t a_round_id); +static void s_round_event_cs_done(dap_chain_cs_dag_poa_round_item_t *a_event_item); // CLI commands static int s_cli_dag_poa(int argc, char ** argv, void **a_str_reply); static bool s_seed_mode = false; -static dap_interval_timer_t s_poa_round_timer = NULL; static bool s_debug_more = false; /** @@ -292,8 +286,13 @@ static int s_cli_dag_poa(int argc, char ** argv, void **a_str_reply) l_poa_pvt->events_sign_cert->name, l_event_new_hash_base58_str); } ret = 0; - if (l_event_is_ready && l_poa_pvt->auto_round_complete) // cs done (minimum signs & verify passed) - s_round_event_cs_done(l_dag, l_event->header.round_id); + if (l_event_is_ready && l_poa_pvt->auto_round_complete) { // cs done (minimum signs & verify passed) + dap_chain_cs_dag_poa_round_item_t l_event_item = { + .datum_hash = l_round_item->round_info.datum_hash, + .dag = l_dag + }; + s_round_event_cs_done(&l_event_item); + } } else { if(!dap_strcmp(l_hash_out_type, "hex")) { dap_cli_server_cmd_set_reply_text(a_str_reply, @@ -396,15 +395,13 @@ static int s_callback_new(dap_chain_t * a_chain, dap_config_t * a_chain_cfg) log_it(L_NOTICE,"Initialized DAG-PoA consensus with %u/%u minimum consensus",l_poa_pvt->auth_certs_count,l_poa_pvt->auth_certs_count_verify); l_dag->chain->callback_created = s_callback_created; - if (!l_dag->is_add_directly && l_poa_pvt->auto_round_complete) { - dap_chain_net_t *l_net = dap_chain_net_by_id(a_chain->net_id); - dap_chain_node_role_t l_role = dap_chain_net_get_role(l_net); - if (l_role.enums == NODE_ROLE_ROOT_MASTER || l_role.enums == NODE_ROLE_ROOT) { - if (!s_poa_round_timer) { - s_poa_round_timer = dap_interval_timer_create(600 * 1000, s_poa_round_clean, a_chain); - log_it(L_MSG, "DAG-PoA: Round timer is started"); - } - DAP_CHAIN_PVT(a_chain)->cs_started = true; + if ( !l_dag->is_add_directly && l_poa_pvt->auto_round_complete ) { + switch ( dap_chain_net_get_role(dap_chain_net_by_id(a_chain->net_id)).enums ) { + case NODE_ROLE_ROOT_MASTER: + case NODE_ROLE_ROOT: + DAP_CHAIN_PVT(a_chain)->cs_started = true; + default: + break; } } @@ -420,69 +417,6 @@ static int s_callback_new(dap_chain_t * a_chain, dap_config_t * a_chain_cfg) return 0; } -/** - * @brief s_poa_round_check_callback_load_round - * @param a_global_db_context - * @param a_rc - * @param a_group - * @param a_key - * @param a_values_total - * @param a_values_shift - * @param a_values_count - * @param a_values - * @param a_arg - */ -static bool s_poa_round_check_callback_round_clean(dap_global_db_instance_t *a_dbi, - int a_rc, const char *a_group, - const size_t a_values_total, const size_t a_values_count, - dap_global_db_obj_t *a_values, void *a_arg) -{ - dap_chain_cs_dag_t *l_dag = (dap_chain_cs_dag_t *)a_arg; - dap_chain_cs_dag_poa_t *l_poa = DAP_CHAIN_CS_DAG_POA(l_dag); - dap_chain_cs_dag_poa_pvt_t *l_poa_pvt = PVT(l_poa); - - if (a_values_count) { - for (size_t i = 0; i < a_values_count; i++) { - if (!strcmp(DAG_ROUND_CURRENT_KEY, a_values[i].key)) - continue; - if (a_values[i].value_len <= sizeof(dap_chain_cs_dag_event_round_item_t) + sizeof(dap_chain_cs_dag_event_t)) { - log_it(L_WARNING, "Too small round item in DAG PoA rounds GDB group"); - dap_global_db_del_sync(a_group, a_values[i].key); - continue; - } - dap_chain_cs_dag_event_round_item_t *l_event_round_item = (dap_chain_cs_dag_event_round_item_t *)a_values[i].value; - uint64_t l_time_diff = dap_nanotime_now() - l_event_round_item->round_info.ts_update; - uint64_t l_timeuot = dap_nanotime_from_sec(l_poa_pvt->confirmations_timeout + l_poa_pvt->wait_sync_before_complete + 10); - uint64_t l_round_id = ((dap_chain_cs_dag_event_t *)l_event_round_item->event_n_signs)->header.round_id; - if (((int64_t)l_time_diff > 0 && l_time_diff > l_timeuot) || l_round_id <= l_dag->round_completed) { - pthread_rwlock_rdlock(&l_poa_pvt->rounds_rwlock); - struct round_timer_arg *l_arg = NULL; - HASH_FIND(hh, l_poa_pvt->active_rounds, &l_round_id, sizeof(uint64_t), l_arg); - pthread_rwlock_unlock(&l_poa_pvt->rounds_rwlock); - if (l_arg) { - log_it(L_INFO, "Event %s is from currently active round [id %"DAP_UINT64_FORMAT_U"]", a_values[i].key, l_round_id); - continue; - } - dap_global_db_del_sync(a_group, a_values[i].key); - log_it(L_DEBUG, "DAG-PoA: Remove event %s from round %"DAP_UINT64_FORMAT_U" %s.", - a_values[i].key, l_round_id, l_time_diff > l_timeuot ? "by timer" : "owing to round completion"); - } - } - } - return true; -} - -/** - * @brief Timer callback for round check - * @param a_chain Chain object - * @return Always true - */ -static void s_poa_round_clean(void *a_arg) -{ - dap_chain_cs_dag_t *l_dag = DAP_CHAIN_CS_DAG((dap_chain_t *)a_arg); - dap_global_db_get_all(l_dag->gdb_group_events_round_new, 0, s_poa_round_check_callback_round_clean, l_dag); -} - static bool s_round_event_ready_minimum_check(dap_chain_cs_dag_t *a_dag, dap_chain_cs_dag_event_t *a_event, size_t a_event_size, char * a_event_hash_hex_str) { @@ -601,58 +535,78 @@ static bool s_callback_round_event_to_chain_callback_get_round_item(dap_global_d { if (a_rc != DAP_GLOBAL_DB_RC_SUCCESS) return false; - struct round_timer_arg *l_arg = (struct round_timer_arg *)a_arg; + dap_chain_cs_dag_poa_round_item_t *l_arg = (dap_chain_cs_dag_poa_round_item_t*)a_arg; dap_chain_cs_dag_t *l_dag = l_arg->dag; - uint64_t l_round_id = l_arg->round_id; dap_chain_cs_dag_poa_pvt_t *l_poa_pvt = PVT(DAP_CHAIN_CS_DAG_POA(l_dag)); pthread_rwlock_wrlock(&l_poa_pvt->rounds_rwlock); - HASH_DEL(l_poa_pvt->active_rounds, l_arg); + HASH_DEL(l_poa_pvt->event_items, l_arg); pthread_rwlock_unlock(&l_poa_pvt->rounds_rwlock); - DAP_DELETE(a_arg); uint16_t l_max_signs_count = 0; dap_list_t *l_dups_list = NULL; - for (size_t i = 0; i < a_values_count; i++) { + size_t i, e, k; + const char *l_complete_keys[a_values_count], *l_expired_keys[a_values_count]; + for (i = 0, e = 0, k = 0; i < a_values_count; i++) { if (!strcmp(DAG_ROUND_CURRENT_KEY, a_values[i].key)) continue; - dap_chain_cs_dag_event_round_item_t *l_round_item = (dap_chain_cs_dag_event_round_item_t *)a_values[i].value; + if (a_values[i].value_len <= sizeof(dap_chain_cs_dag_event_round_item_t) + sizeof(dap_chain_cs_dag_event_t)) { + log_it(L_WARNING, "Incorrect round item size, dump it"); + dap_global_db_del_sync(a_group, a_values[i].key); + continue; + } + dap_chain_cs_dag_event_round_item_t *l_round_item = (dap_chain_cs_dag_event_round_item_t*)a_values[i].value; dap_chain_cs_dag_event_t *l_event = (dap_chain_cs_dag_event_t *)l_round_item->event_n_signs; - if (l_event->header.round_id == l_round_id && - l_round_item->round_info.reject_count < l_poa_pvt->auth_certs_count_verify) { + if ( dap_hash_fast_compare( &l_arg->datum_hash, &l_round_item->round_info.datum_hash ) + && l_round_item->round_info.reject_count < l_poa_pvt->auth_certs_count_verify) + { l_dups_list = dap_list_append(l_dups_list, l_round_item); if (l_event->header.signs_count > l_max_signs_count) l_max_signs_count = l_event->header.signs_count; + l_complete_keys[k++] = a_values[i].key; + } + else if ( dap_nanotime_from_sec(l_poa_pvt->wait_sync_before_complete + l_poa_pvt->confirmations_timeout + 10) + < dap_nanotime_now() - l_round_item->round_info.ts_update ) + { + l_expired_keys[e++] = a_values[i].key; } } dap_chain_cs_dag_event_round_item_t *l_chosen_item = s_round_event_choose_dup(l_dups_list, l_max_signs_count); + dap_list_free(l_dups_list); + char l_datum_hash_str[DAP_HASH_FAST_STR_SIZE]; + dap_hash_fast_to_str(&l_arg->datum_hash, l_datum_hash_str, sizeof(l_datum_hash_str)); if (l_chosen_item) { size_t l_event_size = l_chosen_item->event_size; dap_chain_cs_dag_event_t *l_new_atom = (dap_chain_cs_dag_event_t *)l_chosen_item->event_n_signs; - char *l_event_hash_hex_str; - dap_get_data_hash_str_static(l_new_atom, l_event_size, l_event_hash_hex_str); + dap_hash_fast_t l_atom_hash; + dap_hash_fast(l_new_atom, l_event_size, &l_atom_hash); + char l_event_hash_hex_str[DAP_HASH_FAST_STR_SIZE]; dap_hash_fast_to_str(&l_atom_hash, l_event_hash_hex_str, DAP_HASH_FAST_STR_SIZE); dap_chain_datum_t *l_datum = dap_chain_cs_dag_event_get_datum(l_new_atom, l_event_size); - l_dag->round_completed = dap_max(l_new_atom->header.round_id, l_dag->round_current); int l_verify_datum = dap_chain_net_verify_datum_for_add(l_dag->chain, l_datum, &l_chosen_item->round_info.datum_hash); if (!l_verify_datum) { - dap_hash_fast_t l_atom_hash = {}; - dap_hash_fast(l_new_atom, l_event_size, &l_atom_hash); dap_chain_atom_verify_res_t l_res = l_dag->chain->callback_atom_add(l_dag->chain, l_new_atom, l_event_size, &l_atom_hash, true); - if (l_res == ATOM_ACCEPT) - s_poa_round_clean(l_dag->chain); - log_it(L_INFO, "Event %s from round %"DAP_UINT64_FORMAT_U" %s", - l_event_hash_hex_str, l_round_id, dap_chain_atom_verify_res_str[l_res]); + if (l_res == ATOM_ACCEPT) { + for (; k; --k) { + log_it(L_INFO, "Remove event %s with datum %s, round complete", l_complete_keys[k - 1], l_datum_hash_str); + dap_global_db_del_sync(a_group, l_complete_keys[k - 1]); + } + for (; e; --e) { + log_it(L_INFO, "Event %s with datum %s has expired, dump it", l_expired_keys[e - 1], l_datum_hash_str); + dap_global_db_del_sync(a_group, l_expired_keys[e - 1]); + } + } + log_it(L_INFO, "Event %s with datum %s is %s", + l_event_hash_hex_str, l_datum_hash_str, dap_chain_atom_verify_res_str[l_res]); } else { - char l_datum_hash_str[DAP_CHAIN_HASH_FAST_STR_SIZE]; - dap_chain_hash_fast_to_str(&l_chosen_item->round_info.datum_hash, l_datum_hash_str, DAP_CHAIN_HASH_FAST_STR_SIZE); - log_it(L_INFO, "Event %s from round %"DAP_UINT64_FORMAT_U" not added into chain, because the inner datum %s doesn't pass verification (%s)", - l_event_hash_hex_str, l_round_id, l_datum_hash_str, dap_chain_net_verify_datum_err_code_to_str(l_datum, l_verify_datum)); + log_it(L_ERROR, "Event %s is not chained: datum %s doesn't pass verification, error \"%s\"", + l_event_hash_hex_str, l_datum_hash_str, dap_chain_net_verify_datum_err_code_to_str(l_datum, l_verify_datum)); + for (; k; --k) { + log_it(L_INFO, "Remove event %s with unverified datum %s", l_complete_keys[k - 1], l_datum_hash_str); + dap_global_db_del_sync(a_group, l_complete_keys[k - 1]); + } } - } else { /* !l_chosen_item */ - l_dag->round_completed = l_dag->round_current; - log_it(L_WARNING, "No candidates for round id %"DAP_UINT64_FORMAT_U, l_round_id); - - } - dap_list_free(l_dups_list); - return true; + } else /* !l_chosen_item */ + log_it(L_WARNING, "No valid candidates to wrap datum %s in current round #%"DAP_UINT64_FORMAT_U"", + l_datum_hash_str, l_dag->round_id); + return DAP_DELETE(l_arg), true; } /** @@ -660,39 +614,25 @@ static bool s_callback_round_event_to_chain_callback_get_round_item(dap_global_d * @param a_callback_arg * @return */ -static bool s_callback_round_event_to_chain(struct round_timer_arg *a_callback_arg) +static bool s_callback_round_event_to_chain(dap_chain_cs_dag_poa_round_item_t *a_callback_arg) { - dap_chain_cs_dag_t *l_dag = a_callback_arg->dag; - dap_global_db_get_all(l_dag->gdb_group_events_round_new, 0, s_callback_round_event_to_chain_callback_get_round_item, a_callback_arg); - return false; + return dap_global_db_get_all(a_callback_arg->dag->gdb_group_events_round_new, 0, s_callback_round_event_to_chain_callback_get_round_item, a_callback_arg), + false; } -static void s_round_event_cs_done(dap_chain_cs_dag_t * a_dag, uint64_t a_round_id) +static void s_round_event_cs_done(dap_chain_cs_dag_poa_round_item_t *a_event_item) { - dap_chain_cs_dag_poa_t *l_poa = DAP_CHAIN_CS_DAG_POA(a_dag); - dap_chain_cs_dag_poa_pvt_t *l_poa_pvt = PVT(l_poa); - struct round_timer_arg *l_callback_arg = NULL; + dap_chain_cs_dag_poa_pvt_t *l_poa_pvt = PVT( DAP_CHAIN_CS_DAG_POA(a_event_item->dag) ); + dap_chain_cs_dag_poa_round_item_t *l_event_item = NULL; pthread_rwlock_wrlock(&l_poa_pvt->rounds_rwlock); - HASH_FIND(hh, l_poa_pvt->active_rounds, &a_round_id, sizeof(uint64_t), l_callback_arg); - if (!l_callback_arg) { - l_callback_arg = DAP_NEW_Z(struct round_timer_arg); - if (!l_callback_arg) { - pthread_rwlock_unlock(&l_poa_pvt->rounds_rwlock); - log_it(L_CRITICAL, "%s", c_error_memory_alloc); - return; - } - l_callback_arg->dag = a_dag; - l_callback_arg->round_id = a_round_id; - HASH_ADD(hh, l_poa_pvt->active_rounds, round_id, sizeof(uint64_t), l_callback_arg); - // Placement into chain by confirmation timer - if (!dap_timerfd_start(PVT(l_poa)->confirmations_timeout * 1000, (dap_timerfd_callback_t)s_callback_round_event_to_chain, l_callback_arg)) { - HASH_DEL(l_poa_pvt->active_rounds, l_callback_arg); - pthread_rwlock_unlock(&l_poa_pvt->rounds_rwlock); - DAP_DELETE(l_callback_arg); - log_it(L_ERROR,"Can't run timer for round ID %"DAP_UINT64_FORMAT_U, a_round_id); - return; - } - log_it(L_NOTICE,"Run timer for %d sec for round ID %"DAP_UINT64_FORMAT_U, PVT(l_poa)->confirmations_timeout, a_round_id); + HASH_FIND(hh, l_poa_pvt->event_items, &a_event_item->datum_hash, sizeof(dap_hash_fast_t), l_event_item); + if (!l_event_item) { + l_event_item = DAP_DUP(a_event_item); + if ( !dap_timerfd_start(l_poa_pvt->confirmations_timeout * 1000, (dap_timerfd_callback_t)s_callback_round_event_to_chain, l_event_item) ) + return DAP_DELETE(l_event_item), pthread_rwlock_unlock(&l_poa_pvt->rounds_rwlock), log_it(L_CRITICAL, "Timer creation failed"); + HASH_ADD(hh, l_poa_pvt->event_items, datum_hash, sizeof(dap_hash_fast_t), l_event_item); + log_it(L_INFO, "Confirmation timer for datum %s started [%d s]", + dap_chain_hash_fast_to_str_static(&l_event_item->datum_hash), l_poa_pvt->confirmations_timeout); } pthread_rwlock_unlock(&l_poa_pvt->rounds_rwlock); } @@ -709,22 +649,25 @@ static bool s_callback_sync_all_on_start(dap_global_db_instance_t *a_dbi, int a_ static void s_round_changes_notify(dap_store_obj_t *a_obj, void *a_arg) { - dap_chain_cs_dag_t *l_dag = (dap_chain_cs_dag_t *)a_arg; - assert(l_dag); + assert(a_arg); + dap_chain_cs_dag_t *l_dag = (dap_chain_cs_dag_t*)a_arg; dap_chain_net_t *l_net = dap_chain_net_by_id(l_dag->chain->net_id); - log_it(L_DEBUG, "%s.%s: op_code='%c' group=\"%s\" key=\"%s\" value_size=%zu", - l_net->pub.name, l_dag->chain->name, dap_store_obj_get_type(a_obj), a_obj->group, a_obj->key, a_obj->value_len); - if (dap_store_obj_get_type(a_obj) == DAP_GLOBAL_DB_OPTYPE_ADD) { - if (dap_strcmp(a_obj->key, DAG_ROUND_CURRENT_KEY)) // check key for round increment, if no than process event - s_callback_event_round_sync(l_dag, dap_store_obj_get_type(a_obj), a_obj->group, a_obj->key, a_obj->value, a_obj->value_len); - else - log_it(L_INFO, "Global round ID: %lu", *(uint64_t*)a_obj->value); + dap_global_db_optype_t l_type = dap_store_obj_get_type(a_obj); + log_it(L_DEBUG, "%s.%s: op_code '%c', group \"%s\", key \"%s\", value_size %zu", + l_net->pub.name, l_dag->chain->name, l_type, a_obj->group, a_obj->key, a_obj->value_len); + if ( !dap_strcmp(a_obj->key, DAG_ROUND_CURRENT_KEY) ) + return; + switch ( l_type ) { + case DAP_GLOBAL_DB_OPTYPE_ADD: + s_callback_event_round_sync(l_dag, l_type, a_obj->group, a_obj->key, a_obj->value, a_obj->value_len); + default: + break; } } static void s_timer_process_callback(void *a_arg) { - dap_chain_node_mempool_process_all((dap_chain_t *)a_arg, false); + dap_chain_node_mempool_process_all( (dap_chain_t*)a_arg, false ); } /** @@ -756,19 +699,15 @@ static int s_callback_created(dap_chain_t * a_chain, dap_config_t *a_chain_net_c dap_global_db_cluster_add_notify_callback(l_dag_cluster, s_round_changes_notify, l_dag); dap_chain_net_add_auth_nodes_to_cluster(l_net, l_dag_cluster); dap_link_manager_add_net_associate(l_net->pub.id.uint64, l_dag_cluster->links_cluster); - - byte_t *l_current_round = dap_global_db_get_sync(l_dag->gdb_group_events_round_new, DAG_ROUND_CURRENT_KEY, NULL, NULL, NULL); - l_dag->round_current = l_current_round ? *(uint64_t*)l_current_round : 0; - DAP_DELETE(l_current_round); - log_it(L_INFO, "Current round id %"DAP_UINT64_FORMAT_U, l_dag->round_current); - + l_dag->round_id = 0; PVT(l_poa)->mempool_timer = dap_interval_timer_create(15000, s_timer_process_callback, a_chain); - dap_chain_node_role_t l_role = dap_chain_net_get_role(l_net); - if (l_role.enums == NODE_ROLE_ROOT_MASTER || l_role.enums == NODE_ROLE_ROOT) { - l_dag->round_completed = l_dag->round_current++; - log_it(L_MSG, "Round complete ID %"DAP_UINT64_FORMAT_U", current ID %"DAP_UINT64_FORMAT_U, l_dag->round_completed, l_dag->round_current); - dap_global_db_get_all(l_dag->gdb_group_events_round_new, 0, s_callback_sync_all_on_start, l_dag); + switch ( dap_chain_net_get_role(l_net).enums ) { + case NODE_ROLE_ROOT_MASTER: + case NODE_ROLE_ROOT: + dap_global_db_get_all(l_dag->gdb_group_events_round_new, 0, s_callback_sync_all_on_start, l_dag); + default: + break; } return 0; } @@ -780,7 +719,6 @@ static int s_callback_created(dap_chain_t * a_chain, dap_config_t *a_chain_net_c */ static void s_callback_delete(dap_chain_cs_dag_t * a_dag) { - dap_interval_timer_delete(s_poa_round_timer); dap_chain_cs_dag_poa_t * l_poa = DAP_CHAIN_CS_DAG_POA ( a_dag ); if ( l_poa->_pvt ) { @@ -819,26 +757,19 @@ static dap_chain_cs_dag_event_t * s_callback_event_create(dap_chain_cs_dag_t * a dap_chain_hash_fast_t * a_hashes, size_t a_hashes_count, size_t* a_event_size) { dap_return_val_if_fail(a_dag && a_dag->chain && DAP_CHAIN_CS_DAG_POA(a_dag), NULL); - dap_chain_cs_dag_poa_t * l_poa = DAP_CHAIN_CS_DAG_POA(a_dag); - if ( PVT(l_poa)->events_sign_cert == NULL){ + dap_chain_cs_dag_poa_t *l_poa = DAP_CHAIN_CS_DAG_POA(a_dag); + if ( !PVT(l_poa)->events_sign_cert ) log_it(L_ERROR, "Can't sign event with events_sign_cert in [dag-poa] section"); - return NULL; - } - if ( s_seed_mode || (a_hashes && a_hashes_count) ){ + else if ( s_seed_mode || (a_hashes && a_hashes_count) ) { if ( !PVT(l_poa)->callback_pre_sign || !PVT(l_poa)->callback_pre_sign->callback) { - dap_chain_cs_dag_event_t * l_event = dap_chain_cs_dag_event_new(a_dag->chain->id, a_dag->chain->cells->id, a_dag->round_current, + return dap_chain_cs_dag_event_new(a_dag->chain->id, a_dag->chain->cells->id, a_dag->round_id, a_datum, PVT(l_poa)->events_sign_cert->enc_key, a_hashes, a_hashes_count, a_event_size); - return l_event; } else { - dap_chain_cs_dag_event_t *l_event = dap_chain_cs_dag_event_new(a_dag->chain->id, a_dag->chain->cells->id, a_dag->round_current, - a_datum, NULL, - a_hashes, a_hashes_count, a_event_size); - int ret = PVT(l_poa)->callback_pre_sign->callback(a_dag->chain, l_event, *a_event_size, PVT(l_poa)->callback_pre_sign->arg); - if (ret) { - DAP_DELETE(l_event); - return NULL; - } + dap_chain_cs_dag_event_t *l_event = dap_chain_cs_dag_event_new(a_dag->chain->id, a_dag->chain->cells->id, a_dag->round_id, + a_datum, NULL, a_hashes, a_hashes_count, a_event_size); + if ( PVT(l_poa)->callback_pre_sign->callback(a_dag->chain, l_event, *a_event_size, PVT(l_poa)->callback_pre_sign->arg) ) + return DAP_DELETE(l_event), NULL; *a_event_size = dap_chain_cs_dag_event_sign_add(&l_event, *a_event_size, PVT(l_poa)->events_sign_cert->enc_key); return l_event; } @@ -849,11 +780,8 @@ static dap_chain_cs_dag_event_t * s_callback_event_create(dap_chain_cs_dag_t * a static int s_callback_event_round_sync(dap_chain_cs_dag_t * a_dag, const char a_op_code, const char *a_group, const char *a_key, const void *a_value, const size_t a_value_size) { - if (a_op_code != DAP_GLOBAL_DB_OPTYPE_ADD || !a_key || !a_value || !a_value_size) - return 0; - - if (!strcmp(DAG_ROUND_CURRENT_KEY, a_key)) - return 0; + dap_return_val_if_pass(a_op_code != DAP_GLOBAL_DB_OPTYPE_ADD || !a_key || !a_value + || !a_value_size || !strcmp(DAG_ROUND_CURRENT_KEY, a_key), 0); dap_chain_cs_dag_poa_t * l_poa = DAP_CHAIN_CS_DAG_POA(a_dag); dap_chain_cs_dag_poa_pvt_t *l_poa_pvt = PVT(l_poa); @@ -867,58 +795,36 @@ static int s_callback_event_round_sync(dap_chain_cs_dag_t * a_dag, const char a_ dap_chain_cs_dag_event_round_item_t *l_round_item = (dap_chain_cs_dag_event_round_item_t *)a_value; size_t l_event_size = l_round_item->event_size; dap_chain_cs_dag_event_t *l_event = (dap_chain_cs_dag_event_t*)DAP_DUP_SIZE(l_round_item->event_n_signs, l_event_size); - if (!l_event) { - log_it(L_CRITICAL, "Memory allocation failed"); - return -1; - } - if (l_event->header.round_id < a_dag->round_completed) { - struct round_timer_arg *l_round_active = NULL; - uint64_t l_round_id = l_event->header.round_id; - pthread_rwlock_wrlock(&l_poa_pvt->rounds_rwlock); - HASH_FIND(hh, l_poa_pvt->active_rounds, &l_round_id, sizeof(uint64_t), l_round_active); - pthread_rwlock_unlock(&l_poa_pvt->rounds_rwlock); - if (!l_round_active) { - log_it(L_DEBUG, "DAG event came from too old round [last complete id %"DAP_UINT64_FORMAT_U" > %"DAP_UINT64_FORMAT_U"], skip it", - a_dag->round_completed, l_event->header.round_id); - DAP_DELETE(l_event); - return -2; - } - } - if (dap_chain_cs_dag_event_sign_exists(l_event, l_event_size, l_poa_pvt->events_sign_cert->enc_key) - || dap_chain_cs_dag_event_round_sign_exists(l_round_item, l_poa_pvt->events_sign_cert->enc_key)) { - // if my sign exists - if (l_poa_pvt->auto_round_complete && - s_round_event_ready_minimum_check(a_dag, l_event, l_event_size, (char *)a_key)) - // cs done (minimum signs & verify passed) - s_round_event_cs_done(a_dag, l_event->header.round_id); - DAP_DELETE(l_event); - return 0; + if (!l_event) + return log_it(L_CRITICAL, "Memory allocation failed"), -1; + + if ( dap_chain_cs_dag_event_sign_exists(l_event, l_event_size, l_poa_pvt->events_sign_cert->enc_key) + || dap_chain_cs_dag_event_round_sign_exists(l_round_item, l_poa_pvt->events_sign_cert->enc_key) + && l_poa_pvt->auto_round_complete + && s_round_event_ready_minimum_check(a_dag, l_event, l_event_size, (char*)a_key) ) + { + dap_chain_cs_dag_poa_round_item_t l_event_item = { .datum_hash = l_round_item->round_info.datum_hash, .dag = a_dag }; + return DAP_DELETE(l_event), s_round_event_cs_done(&l_event_item), 0; } size_t l_event_size_new = 0; int ret = 0; - if (l_event->header.round_id > a_dag->round_current) { - log_it(L_DEBUG, "Refresh current round ID [%lu -> %lu]", a_dag->round_current, l_event->header.round_id); - a_dag->round_current = l_event->header.round_id; - } - if (!l_poa_pvt->callback_pre_sign || !l_poa_pvt->callback_pre_sign->callback || - !(ret = l_poa_pvt->callback_pre_sign->callback(a_dag->chain, l_event, l_event_size, - l_poa_pvt->callback_pre_sign->arg))) { + if ( !l_poa_pvt->callback_pre_sign || !l_poa_pvt->callback_pre_sign->callback + || !(ret = l_poa_pvt->callback_pre_sign->callback(a_dag->chain, l_event, l_event_size, + l_poa_pvt->callback_pre_sign->arg)) ) + { l_event_size_new = dap_chain_cs_dag_event_sign_add(&l_event, l_event_size, l_poa_pvt->events_sign_cert->enc_key); - //char *l_event_new_hash_hex_str; - //dap_get_data_hash_str_static(l_event, l_event_size_new, l_event_new_hash_hex_str); - dap_chain_cs_dag_event_gdb_set(a_dag, /*l_event_new_hash_hex_str*/ (char*)a_key, l_event, l_event_size_new, l_round_item); + dap_chain_cs_dag_event_gdb_set(a_dag, (char*)a_key, l_event, l_event_size_new, l_round_item); } else { // set sign for reject l_round_item = DAP_DUP_SIZE(a_value, a_value_size); if (dap_chain_cs_dag_event_round_sign_add(&l_round_item, a_value_size, l_poa_pvt->events_sign_cert->enc_key)) { - log_it(L_NOTICE,"Can't sign event %s, because sign rejected by pre_sign callback, ret code=%d", a_key, ret); + log_it(L_NOTICE,"Can't sign event %s, because sign rejected by pre_sign callback, ret code %d", a_key, ret); l_round_item->round_info.reject_count++; - dap_chain_cs_dag_event_gdb_set(a_dag, (char *)a_key, l_event, l_event_size, l_round_item); + dap_chain_cs_dag_event_gdb_set(a_dag, (char*)a_key, l_event, l_event_size, l_round_item); } DAP_DELETE(l_round_item); } - DAP_DELETE(l_event); - return 0; + return DAP_DELETE(l_event), ret; } /** diff --git a/modules/mempool/dap_chain_mempool.c b/modules/mempool/dap_chain_mempool.c index 8fc3ad433d34ad4c9d290455a8c7292d15091d75..7ca0d0de3b5ed30b80e0ad7cca192fe492a0b965 100644 --- a/modules/mempool/dap_chain_mempool.c +++ b/modules/mempool/dap_chain_mempool.c @@ -1497,9 +1497,7 @@ void dap_chain_mempool_filter(dap_chain_t *a_chain, int *a_removed){ continue; } //Filter hash - char *l_hash_content_str; - dap_get_data_hash_str_static(l_datum->data, l_datum->header.data_size, l_hash_content_str); - if (dap_strcmp(l_hash_content_str, l_objs[i].key) != 0) { + if (dap_strcmp(dap_get_data_hash_str(l_datum->data, l_datum->header.data_size).s, l_objs[i].key) != 0) { l_removed++; log_it(L_NOTICE, "Removed datum from mempool with \"%s\" key group %s. The hash of the contents of the " "datum does not match the key.", l_objs[i].key, l_gdb_group); diff --git a/modules/net/dap_chain_node_cli_cmd.c b/modules/net/dap_chain_node_cli_cmd.c index e8b1aa3c31b465ea5b2720b75d735ca27e84e088..ecd031bae9adcb64b94fa534ac6f296437413b84 100644 --- a/modules/net/dap_chain_node_cli_cmd.c +++ b/modules/net/dap_chain_node_cli_cmd.c @@ -566,8 +566,6 @@ int com_global_db(int a_argc, char ** a_argv, void **a_str_reply) switch (l_subcmd) { case SUMCMD_GET: // Get value { - char *l_hash_str; - dap_get_data_hash_str_static(l_value, l_value_len, l_hash_str); char *l_value_str = DAP_NEW_Z_SIZE(char, l_value_len * 2 + 2); if(!l_value_str) { log_it(L_CRITICAL, "%s", c_error_memory_alloc); @@ -580,7 +578,7 @@ int com_global_db(int a_argc, char ** a_argv, void **a_str_reply) size_t ret = dap_bin2hex(l_value_str, l_value, l_value_len); json_object_object_add(json_obj_rec, "command status", json_object_new_string("Record found")); json_object_object_add(json_obj_rec, "lenght(byte)", json_object_new_uint64(l_value_len)); - json_object_object_add(json_obj_rec, "hash", json_object_new_string(l_hash_str)); + json_object_object_add(json_obj_rec, "hash", json_object_new_string(dap_get_data_hash_str(l_value, l_value_len).s)); json_object_object_add(json_obj_rec, "pinned", l_is_pinned ? json_object_new_string("Yes") : json_object_new_string("No") ); json_object_object_add(json_obj_rec, "value", json_object_new_string(l_value_str)); DAP_DELETE(l_value_str); @@ -6819,9 +6817,8 @@ int com_tx_create_json(int a_argc, char ** a_argv, void **reply) // Add transaction to mempool char *l_gdb_group_mempool_base_tx = dap_chain_net_get_gdb_group_mempool_new(l_chain);// get group name for mempool - char *l_tx_hash_str; - dap_get_data_hash_str_static(l_datum_tx->data, l_datum_tx->header.data_size, l_tx_hash_str); - bool l_placed = !dap_global_db_set(l_gdb_group_mempool_base_tx,l_tx_hash_str, l_datum_tx, l_datum_tx_size, false, NULL, NULL); + char *l_tx_hash_str = dap_get_data_hash_str(l_datum_tx->data, l_datum_tx->header.data_size).s; + bool l_placed = !dap_global_db_set(l_gdb_group_mempool_base_tx, l_tx_hash_str, l_datum_tx, l_datum_tx_size, false, NULL, NULL); DAP_DEL_Z(l_datum_tx); DAP_DELETE(l_gdb_group_mempool_base_tx); @@ -8247,46 +8244,26 @@ static int s_signer_cmd(int a_arg_index, int a_argc, char **a_argv, void **a_str dap_cli_server_cmd_find_option_val(a_argv, a_arg_index, a_argc, l_opts_signer[i].name, (const char **) &l_opts_sign[i]); } - if (!l_opts_sign[OPT_CERT]) { - dap_cli_server_cmd_set_reply_text(a_str_reply, "%s need to be selected", l_opts_signer[OPT_CERT].name); - return -1; - } + if (!l_opts_sign[OPT_CERT]) + return dap_cli_server_cmd_set_reply_text(a_str_reply, "%s need to be selected", l_opts_signer[OPT_CERT].name), -1; dap_chain_net_t *l_network = dap_chain_net_by_name(l_opts_sign[OPT_NET]); - if (!l_network) { - dap_cli_server_cmd_set_reply_text(a_str_reply, "%s network not found", l_opts_sign[OPT_NET]); - return -1; - } + if ( !l_network ) + return dap_cli_server_cmd_set_reply_text(a_str_reply, "%s network not found", l_opts_sign[OPT_NET]), -1; dap_chain_t *l_chain = dap_chain_net_get_chain_by_name(l_network, l_opts_sign[OPT_CHAIN]); - if (!l_chain) { - dap_cli_server_cmd_set_reply_text(a_str_reply, "%s chain not found", l_opts_sign[OPT_CHAIN]); - return -1; - } + if (!l_chain) + return dap_cli_server_cmd_set_reply_text(a_str_reply, "%s chain not found", l_opts_sign[OPT_CHAIN]), -1; - int l_ret = 0; dap_sign_t *l_sign = NULL; - dap_chain_datum_t *l_datum = NULL; - - l_ret = s_get_key_from_file(l_opts_sign[OPT_FILE], l_opts_sign[OPT_MIME], l_opts_sign[OPT_CERT], &l_sign); - if (!l_ret) { - dap_cli_server_cmd_set_reply_text(a_str_reply, "%s cert not found", l_opts_sign[OPT_CERT]); - return -1; - } - - l_datum = dap_chain_datum_create(DAP_CHAIN_DATUM_SIGNER, l_sign->pkey_n_sign, l_sign->header.sign_size); - if (!l_datum) { - dap_cli_server_cmd_set_reply_text(a_str_reply, "not created datum"); - return -1; - } - - l_ret = l_chain->callback_add_datums(l_chain, &l_datum, 1); - - char *l_key_str; - dap_get_data_hash_str_static(l_datum->data, l_datum->header.data_size, l_key_str); - dap_cli_server_cmd_set_reply_text(a_str_reply, "hash: %s", l_key_str); - DAP_DELETE(l_datum); - return l_ret; + if ( !s_get_key_from_file(l_opts_sign[OPT_FILE], l_opts_sign[OPT_MIME], l_opts_sign[OPT_CERT], &l_sign) ) + return dap_cli_server_cmd_set_reply_text(a_str_reply, "%s cert not found", l_opts_sign[OPT_CERT]), -1; + + dap_chain_datum_t * l_datum = dap_chain_datum_create(DAP_CHAIN_DATUM_SIGNER, l_sign->pkey_n_sign, l_sign->header.sign_size); + if (!l_datum) + return dap_cli_server_cmd_set_reply_text(a_str_reply, "not created datum"), -1; + dap_cli_server_cmd_set_reply_text(a_str_reply, "hash: %s", dap_get_data_hash_str(l_datum->data, l_datum->header.data_size)); + return DAP_DELETE(l_datum), l_chain->callback_add_datums(l_chain, &l_datum, 1); } diff --git a/modules/service/stake/dap_chain_net_srv_stake_lock.c b/modules/service/stake/dap_chain_net_srv_stake_lock.c index 627defb078f38f725e0d69e3f087ef2b7e9cc688..7a5d72bcd9cf9762641eb2dd54eb296efec5856c 100644 --- a/modules/service/stake/dap_chain_net_srv_stake_lock.c +++ b/modules/service/stake/dap_chain_net_srv_stake_lock.c @@ -1043,10 +1043,8 @@ static int s_stake_lock_callback_verificator(dap_ledger_t *a_ledger, dap_chain_t if (!l_burning_tx) { char l_burning_tx_hash_str[DAP_CHAIN_HASH_FAST_STR_SIZE] = { '\0' }; dap_hash_fast_to_str(&l_burning_tx_hash, l_burning_tx_hash_str, DAP_CHAIN_HASH_FAST_STR_SIZE); - char *l_take_tx_hash_str; - dap_get_data_hash_str_static(a_tx_in, dap_chain_datum_tx_get_size(a_tx_in), l_take_tx_hash_str); debug_if(s_debug_more, L_ERROR, "[Legacy] Can't find burning tx with hash %s, obtained from the receipt of take tx %s", - l_burning_tx_hash_str, l_take_tx_hash_str); + l_burning_tx_hash_str, dap_get_data_hash_str(a_tx_in, dap_chain_datum_tx_get_size(a_tx_in)).s); return -10; } } else diff --git a/modules/type/dag/dap_chain_cs_dag.c b/modules/type/dag/dap_chain_cs_dag.c index 261d64f99f8733f3853d5795658d2d4af39b3ed4..b8d6b9cddb7ded3eb9d4013a45bc17f05572fb6c 100644 --- a/modules/type/dag/dap_chain_cs_dag.c +++ b/modules/type/dag/dap_chain_cs_dag.c @@ -683,11 +683,7 @@ static bool s_chain_callback_datums_pool_proc(dap_chain_t *a_chain, dap_chain_da * or we have successfully chosen the hash(es) to link with. * No additional conditions required. */ - byte_t *l_current_round_bytes = dap_global_db_get_sync(l_dag->gdb_group_events_round_new, DAG_ROUND_CURRENT_KEY, NULL, NULL, NULL); - uint64_t l_current_round = l_current_round_bytes ? *(uint64_t*)l_current_round_bytes : 0; - DAP_DEL_Z(l_current_round_bytes); - l_dag->round_completed = l_current_round++; - l_dag->round_current = l_current_round; + ++l_dag->round_id; uint64_t l_event_size = 0; dap_chain_cs_dag_event_t * l_event = l_dag->callback_cs_event_create ? l_dag->callback_cs_event_create(l_dag, a_datum, l_hashes, l_hashes_linked, &l_event_size) @@ -708,18 +704,15 @@ static bool s_chain_callback_datums_pool_proc(dap_chain_t *a_chain, dap_chain_da } else return true; } - - dap_global_db_set_sync(l_dag->gdb_group_events_round_new, DAG_ROUND_CURRENT_KEY, - &l_current_round, sizeof(uint64_t), false); dap_chain_cs_dag_event_round_item_t l_round_item = { .round_info.datum_hash = l_datum_hash }; - char *l_event_hash_hex_str = DAP_NEW_STACK_SIZE(char, DAP_CHAIN_HASH_FAST_STR_SIZE); + char l_event_hash_hex_str[DAP_CHAIN_HASH_FAST_STR_SIZE]; dap_chain_hash_fast_to_str(&l_event_hash, l_event_hash_hex_str, DAP_CHAIN_HASH_FAST_STR_SIZE); l_res = dap_chain_cs_dag_event_gdb_set(l_dag, l_event_hash_hex_str, l_event, l_event_size, &l_round_item); DAP_DELETE(l_event); log_it(l_res ? L_INFO : L_ERROR, - l_res ? "Event %s placed in the new forming round [id %"DAP_UINT64_FORMAT_U"]" - : "Can't add new event [%s] to the new events round [id %"DAP_UINT64_FORMAT_U"]", - l_event_hash_hex_str, l_current_round); + l_res ? "Event %s placed into new forming round [id %"DAP_UINT64_FORMAT_U"]" + : "Can't add new event %s to new events round [id %"DAP_UINT64_FORMAT_U"]", + l_event_hash_hex_str, l_dag->round_id); return l_res; } @@ -1285,13 +1278,10 @@ static dap_chain_datum_t *s_chain_callback_datum_iter_get_next(dap_chain_datum_i return a_datum_iter->cur; } -static bool s_json_dag_pack_round(json_object * a_json_out, dap_global_db_obj_t * a_objs, int i){ - json_object * json_obj_event_i = json_object_new_object(); - if (!strcmp(DAG_ROUND_CURRENT_KEY, a_objs[i].key)) { - json_object_object_add(json_obj_event_i, a_objs[i].key, json_object_new_uint64(*(uint64_t *)a_objs[i].value)); - json_object_array_add(a_json_out, json_obj_event_i); +static bool s_json_dag_pack_round(json_object * a_json_out, dap_global_db_obj_t * a_objs, int i) { + if ( !strcmp(DAG_ROUND_CURRENT_KEY, a_objs[i].key) ) return true; - } + json_object * json_obj_event_i = json_object_new_object(); dap_chain_cs_dag_event_t * l_event = (dap_chain_cs_dag_event_t *) ((dap_chain_cs_dag_event_round_item_t *)a_objs[i].value)->event_n_signs; char buf[DAP_TIME_STR_SIZE]; diff --git a/modules/type/dag/dap_chain_cs_dag_event.c b/modules/type/dag/dap_chain_cs_dag_event.c index 8e4fc060129010fbf0010edf8a07953441c6aec9..f89726a8330a77a48eb7ff8b83812c22ea743670 100644 --- a/modules/type/dag/dap_chain_cs_dag_event.c +++ b/modules/type/dag/dap_chain_cs_dag_event.c @@ -48,46 +48,38 @@ dap_chain_cs_dag_event_t *dap_chain_cs_dag_event_new(dap_chain_id_t a_chain_id, dap_chain_hash_fast_t *a_hashes, size_t a_hashes_count, size_t *a_event_size) { assert(a_event_size); - size_t l_hashes_size = sizeof(*a_hashes)*a_hashes_count; - size_t l_datum_size = dap_chain_datum_size(a_datum); - dap_chain_cs_dag_event_t * l_event_new = NULL; - size_t l_event_size = sizeof(l_event_new->header) - + l_hashes_size - + l_datum_size; - l_event_new = DAP_NEW_Z_SIZE(dap_chain_cs_dag_event_t, l_event_size); - l_event_new->header.ts_created = dap_time_now(); - l_event_new->header.cell_id.uint64 = a_cell_id.uint64; - l_event_new->header.chain_id.uint64 = a_chain_id.uint64; - l_event_new->header.hash_count = a_hashes_count; - l_event_new->header.round_id = a_round_id; - - if ( l_hashes_size ){ - memcpy(l_event_new->hashes_n_datum_n_signs, a_hashes, l_hashes_size ); - } + size_t l_hashes_size = sizeof(*a_hashes) * a_hashes_count, + l_datum_size = dap_chain_datum_size(a_datum), + l_event_size = sizeof(dap_chain_class_dag_event_hdr_t) + l_hashes_size + l_datum_size; + dap_chain_cs_dag_event_t *l_event_new = DAP_NEW_Z_SIZE(dap_chain_cs_dag_event_t, l_event_size); + *l_event_new = (dap_chain_cs_dag_event_t) { + { // .round_id = a_round_id, + .ts_created = dap_time_now(), + .chain_id = a_chain_id, + .cell_id = a_cell_id, + .hash_count = a_hashes_count + } + }; - memcpy(l_event_new->hashes_n_datum_n_signs+l_hashes_size, a_datum,l_datum_size ); + if ( l_hashes_size ) + memcpy( l_event_new->hashes_n_datum_n_signs, a_hashes, l_hashes_size ); + memcpy( l_event_new->hashes_n_datum_n_signs + l_hashes_size, a_datum,l_datum_size ); if ( a_key ){ - dap_sign_t * l_sign = dap_sign_create(a_key, l_event_new, l_event_size, 0); - if ( l_sign ){ - size_t l_sign_size = dap_sign_get_size(l_sign); - l_event_size += l_sign_size; - l_event_new = (dap_chain_cs_dag_event_t *)DAP_REALLOC(l_event_new, l_event_size); - if (!l_event_new) { - log_it(L_CRITICAL, "Not enough memeory"); - DAP_DELETE(l_sign); - return NULL; - } - memcpy(l_event_new->hashes_n_datum_n_signs + l_hashes_size + l_datum_size, l_sign, l_sign_size); - l_event_new->header.signs_count++; - log_it(L_INFO,"Created event size %zd, signed with sign size %zd", l_event_size, l_sign_size); - DAP_DELETE(l_sign); - }else { - log_it(L_ERROR,"Can't sign dag event!"); - DAP_DELETE(l_event_new); - return NULL; - } - }else { + dap_sign_t *l_sign = dap_sign_create(a_key, l_event_new, l_event_size, 0); + if ( !l_sign ) + return DAP_DELETE(l_event_new), log_it(L_ERROR,"Can't sign dag event!"), NULL; + size_t l_sign_size = dap_sign_get_size(l_sign); + l_event_size += l_sign_size; + dap_chain_cs_dag_event_t *l_event_newer = (dap_chain_cs_dag_event_t*)DAP_REALLOC(l_event_new, l_event_size); + if (!l_event_newer) + return DAP_DEL_MULTY(l_event_new, l_sign), log_it(L_CRITICAL, "Not enough memeory"), NULL; + l_event_new = l_event_newer; + memcpy(l_event_new->hashes_n_datum_n_signs + l_hashes_size + l_datum_size, l_sign, l_sign_size); + l_event_new->header.signs_count++; + log_it(L_INFO,"Created event size %zd, signed with sign size %zd", l_event_size, l_sign_size); + DAP_DELETE(l_sign); + } else { log_it(L_NOTICE, "Created unsigned dag event"); } if (a_event_size) @@ -110,9 +102,7 @@ uint64_t dap_chain_cs_dag_event_calc_size_excl_signs(dap_chain_cs_dag_event_t *a return 0; dap_chain_datum_t *l_datum = (dap_chain_datum_t *)(a_event->hashes_n_datum_n_signs + l_hashes_size); uint64_t l_ret = dap_chain_datum_size(l_datum) + l_hashes_size + sizeof(a_event->header); - if (a_limit_size && a_limit_size < l_ret) - return 0; - return l_ret; + return a_limit_size && a_limit_size < l_ret ? 0 : l_ret; } /** diff --git a/modules/type/dag/include/dap_chain_cs_dag.h b/modules/type/dag/include/dap_chain_cs_dag.h index 1020634067194f1684b3fb16e7197d3690456880..42fc9e30ddd3f523e484e0626216b18493da7934 100644 --- a/modules/type/dag/include/dap_chain_cs_dag.h +++ b/modules/type/dag/include/dap_chain_cs_dag.h @@ -61,7 +61,7 @@ typedef struct dap_chain_cs_dag dap_chain_hash_fast_t static_genesis_event_hash; dap_chain_cs_dag_hal_item_t *hal; - atomic_uint_fast64_t round_current, round_completed; + atomic_uint_fast64_t round_id; uint16_t datum_add_hashes_count; char * gdb_group_events_round_new;