Errors in ProcessMessages should not end up in getinfo errors #6809

issue laanwj opened this issue on October 12, 2015

laanwj commented at 8:26 PM on October 12, 2015: member
It's generally harmless if people send corrupted packets, but having the error appear in "errors" makes it look much more serious:
```
"errors" : "EXCEPTION: NSt8ios_base7failureE       \nnon-canonical ReadCompactSize()       \nbitcoin in ProcessMessages()       \n"
```
This happens due to the use of PrintExceptionContinue in the catch clauses, which set strMiscWarning.
laanwj added the label P2P on Oct 12, 2015
laanwj referenced this in commit 9b63758974 on Nov 27, 2015
laanwj closed this on Nov 27, 2015
daira referenced this in commit f08e31382b on Feb 21, 2020
zkbot referenced this in commit 69e0e7ee62 on Feb 21, 2020
denverbdr referenced this in commit 2c8e5c086f on Mar 6, 2020
MarcoFalke locked this on Sep 8, 2021

Contributors

Labels