W. J. van der Laan 46b4937bc1
Merge bitcoin/bitcoin#23142: Return false on corrupt tx rather than asserting
0ab4c3b27265401c59e40adc494041927dc9dbe3 Return false on corrupt tx rather than asserting (Samuel Dobson)

Pull request description:

  Takes up #19793

  Rather than asserting, we log an error and return CORRUPT so that the user is informed. This type of error isn't critical so it isn't worth `assert`ing.

ACKs for top commit:
  achow101:
    ACK 0ab4c3b27265401c59e40adc494041927dc9dbe3
  laanwj:
    Code review ACK 0ab4c3b27265401c59e40adc494041927dc9dbe3
  ryanofsky:
    Code review ACK 0ab4c3b27265401c59e40adc494041927dc9dbe3. There may be room for more improvements later like better error messages or easier recovery options, but changing from an assert to an error seems like a clear improvement, and this seems to avoid all the pitfalls of the last PR that tried this.

Tree-SHA512: 4a1a412e7c473d176c4e09123b85f390a6b0ea195e78d28ebd50b13814b7852f8225a172511a2efb6affb555b11bd4e667c19eb8c78b060c5444b62f0fae5f7a
2021-10-01 10:32:10 +02:00
..
2021-09-10 11:18:58 +08:00
2021-09-29 14:10:29 +02:00
2021-08-20 16:59:41 +02:00
2021-09-20 13:34:48 +01:00
2021-07-30 11:21:51 +02:00
2021-07-30 11:21:51 +02:00
2021-03-17 17:59:22 -07:00
2021-09-29 09:48:36 +08:00
2021-09-29 09:48:36 +08:00
2021-09-16 21:16:39 +09:00
2021-09-11 10:47:02 +03:00
2021-04-06 14:50:17 +08:00
2021-08-23 21:38:34 -04:00
2021-09-29 18:46:30 +02:00
2021-08-20 16:59:41 +02:00