• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

KarpelesLab / libwallet / 25301339880
24%

Build:
DEFAULT BRANCH: master
Ran 04 May 2026 04:47AM UTC
Jobs 1
Files 113
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

04 May 2026 04:14AM UTC coverage: 20.891% (+0.06%) from 20.83%
25301339880

push

github

MagicalTux
wlttx/bitcoin: fix double txid byte-reversal that broke every send

Root cause of the long-standing bitcoin "bad-txns-inputs-
missingorspent" failures: parseTxoRef was reversing the txid
bytes before storing them in BtcTxInput.TXID, but
outscript.BtcTxInput.Bytes() ALSO reverses TXID at marshal time
(btctx.go:548 — the comment there even calls out that TXID is
stored in displayable / big-endian form internally and the
marshaler converts to wire format). Compounded, the wire bytes
emitted in our broadcast tx were the modchain-reported txid
WITH ITS BYTES REVERSED — i.e. the broadcast referenced a
bogus txid the chain has never seen, so the node rejected it
with "missing or spent".

Verified end-to-end with the user's actual failure: modchain
reported owned UTXO d36a0d…b9b:0 (alive on-chain, gettxout
confirms 0.1 LTC at that displayable id). Our broadcast wrote
wire bytes that, re-reversed, displayed as 9bbf61…6ad3:0
(no such UTXO). Litecoin node returned -25.

Same bug pattern in SignRawBitcoinTx (mpurse / Counterparty
flows): when looking up wire-decoded inputs against modchain's
owned set, we reversed TXID before formatting the lookup key.
TXID is already in displayable form post-UnmarshalBinary
(btctx.go:626 reverses on read), so the extra reverse made
every input report as "not owned by this account".

Both fixes: drop the reverse, let outscript handle the wire
conversion. Regression test pins the byte-order convention.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

1 of 10 new or added lines in 1 file covered. (10.0%)

3192 of 15279 relevant lines covered (20.89%)

124.45 hits per line

Uncovered Changes

Lines Coverage ∆ File
9
15.54
2.09% wlttx/bitcoin.go
Jobs
ID Job ID Ran Files Coverage
1 25301339880.1 04 May 2026 04:47AM UTC 113
20.89
GitHub Action Run
Source Files on build 25301339880
  • Tree
  • List 113
  • Changed 1
  • Source Changed 1
  • Coverage Changed 1
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • Github Actions Build #25301339880
  • fc328ddd on github
  • Prev Build on master (#25297984418)
  • Next Build on master (#25302114988)
  • Delete
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc