diff options
| author | John Newbery <john@johnnewbery.com> | 2018-05-03 13:41:03 -0400 |
|---|---|---|
| committer | John Newbery <john@johnnewbery.com> | 2018-05-03 13:41:03 -0400 |
| commit | fad63ebe0f0db1e1d909064665be91ebd970bb0a (patch) | |
| tree | 9bef3b0d0e1d49fb68e26e70a4caaa6defa7bd9f /src/netaddress.h | |
| parent | 7eb7076f70078c06bef9752f22acf92fd86e616a (diff) | |
[logging] Don't incorrectly log that REJECT messages are unknown.
Reject messages are logged to debug.log if NET debug logging is enabled.
Because of the way the `ProcessMessages()` function is structured,
processing for REJECT messages will also drop through to the default
branch and incorrectly log `Unknown command "reject" from peer-?`. Fix
that by exiting from `ProcessMessages()` early.
without this PR:
```
2018-05-03T17:37:00.930600Z received: reject (21 bytes) peer=0
2018-05-03T17:37:00.930620Z Reject message code 16: spammy spam
2018-05-03T17:37:00.930656Z Unknown command "reject" from peer=0
```
with this PR:
```
2018-05-03T17:35:04.751246Z received: reject (21 bytes) peer=0
2018-05-03T17:35:04.751274Z Reject message code 16: spammy spam
```
Diffstat (limited to 'src/netaddress.h')
0 files changed, 0 insertions, 0 deletions