Skip to content
This repository was archived by the owner on Feb 26, 2021. It is now read-only.

Recover from rttvar errors#28

Merged
J12934 merged 4 commits intomasterfrom
recover-from-rttvar-errors
Jan 20, 2020
Merged

Recover from rttvar errors#28
J12934 merged 4 commits intomasterfrom
recover-from-rttvar-errors

Conversation

@J12934
Copy link
Member

@J12934 J12934 commented Oct 1, 2019

Nmap scans sometimes randomly crash with its log output looking something like this:

SCANNING location: "1.2.3.4/24", parameters: "--top-ports 50"
RTTVAR has grown to over 2.3 seconds, decreasing to 2.0

Failed to perform Job "32184673-aa9f-4e17-af0e-bbca77122157" Error: Failed to execute nmap portscan.
    at ScannerScaffolding.worker [as _worker] (/src/src/nmap.js:138:23)
    at process._tickCallback (internal/process/next_tick.js:68:7)
Job Failure submitted succesfully.

This error completely crashes the scan, even thought the log indicates that nmap is able to recover from it.

"Unfortunately" this error happens relatively rarely. So debugging / fixing this might be more of a long term effort.

@J12934 J12934 added the bug Something isn't working label Oct 1, 2019
@J12934 J12934 self-assigned this Oct 1, 2019
@rfelber
Copy link
Member

rfelber commented Jan 19, 2020

Hi @J12934,

it seems this RTTVar errors are more or less a normal behavior of nmap if the network or target responses to slow. I get this error also some time at my local maschine with nmap (without SCB) ...

RTT = Round-Trip-Time

https://www.hackerboard.de/threads/meldungen-beim-nmap-scan.1316/

@J12934
Copy link
Member Author

J12934 commented Jan 20, 2020

Ah never understood it meant round trip time 👍
Yeah this PR tries to correct this, as the current / previous implementation seems to crash as soon as this message pops up.

I think this branch is stable enough to be merged. We've been using it for our scans since and didn't encounter any problems with it so far.

@J12934 J12934 marked this pull request as ready for review January 20, 2020 08:32
@J12934 J12934 merged commit 551c5a5 into master Jan 20, 2020
@J12934 J12934 deleted the recover-from-rttvar-errors branch January 20, 2020 08:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants