HomePhabricator

[herald] Patch bug in splitOverflowTgMsg

Description

[herald] Patch bug in splitOverflowTgMsg

Summary:
T2972

Patch a bug in splitOverflowTgMsg

The function was resetting the count of the 'next' message to zero. However, the 'next' message will include the line currently being iterated on in the loop. So, length should not be reset to zero but to the current length of that line.

In most cases the function will still work. Error cases were discovered when adding token send parsing changed msg lengths.

Test Plan: npm test

Reviewers: #bitcoin_abc, Fabien

Reviewed By: #bitcoin_abc, Fabien

Subscribers: Fabien

Differential Revision: https://reviews.bitcoinabc.org/D13865

Details

Provenance
bytesofmanAuthored on May 5 2023, 14:34
bytesofmanPushed on May 5 2023, 18:11
Reviewer
Restricted Project
Differential Revision
D13865: [herald] Patch bug in splitOverflowTgMsg
Parents
rABC1f51c892cf03: [cmake] Improve EVP_MD_CTX_new detection
Branches
Unknown
Tags
Unknown