github-wownero/wownero
1
0
Fork 0
mirror of https://github.com/wownero/wownero.git synced 2026-11-15 21:03:53 +01:00
Code Issues Packages Projects Releases Wiki Activity

Blockchain: fix wrong block_weight in handle_get_objects

Browse Source 
- When there are missing IDs, blocks.size() != arg.blocks.size(), so arg.blocks can't be indexed by `i` - the indices will be wrong, the wrong weight will be returned to some peer and this peer will ban our node.

Use `bl.second` instead of `arg.blocks[i]`. Also it saves one DB query per returned block.

(cherry picked from commit 3d856f4825283e66a7ebbe9ac32e3f864bc906e2)
This commit is contained in:
SChernykh
2026-10-04 18:50:20 -04:00
committed by such-gitea
parent 99d0c33dde
commit eb71cc9633
1 changed files with 1 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/cryptonote_core/blockchain.cpp
+1 -3
View File
@@ -2312,9 +2312,7 @@ bool Blockchain::handle_get_objects(NOTIFY_REQUEST_GET_OBJECTS::request& arg, NO
//pack block
e.block = std::move(bl.first);
e.block_weight = 0;
if (arg.prune && m_db->block_exists(arg.blocks[i]))
e.block_weight = m_db->get_block_weight(m_db->get_block_height(arg.blocks[i]));
e.block_weight = arg.prune ? m_db->get_block_weight(get_block_height(bl.second)) : 0;
}
return true;