Scanner abruptly exits with "error: can't start new thread" after certain extent about angularjs-csti-scanner HOT 13 CLOSED

@jovyn Thank you for reporting this issue. I think this is due to a design issue in the crawler.

Currently, the crawler starts new threads in the thread that finished. I'm going to change it so that new threads will be started from the main thread.

from angularjs-csti-scanner.

@jovyn Do you know after how much requests this happens?

from angularjs-csti-scanner.

Referring to my Burp logs (I chained the scanner via Burp) I can see about 8492 requests.
This may not be the exact number everytime this happens. Maeanwhile let me run it against some other site and share the feedback with you.

from angularjs-csti-scanner.

@jovyn I think I fixed this issue on the develop branch, however, I can't test it since I could not reproduce the issue after ~8500 requests. Could you test if it works for you?

from angularjs-csti-scanner.

Hey @tijme I installed the new version from the develop branch and ran the scanner, but I am getting the below errors for larger scanning requests (about 7000 requests) :

_[ERROR]
error : Memory allocation failed : growing buffer
error : Memory allocation failed : growing buffer_

followed by \:

_Traceback (most recent call last):
  File ".\extended.develop.py", line 177, in <module>
    main()
  File ".\extended.develop.py", line 84, in main
    driver.start()
  File "C:\Python27\lib\site-packages\acstis\Driver.py", line 136, in start
    crawler.start_with(startpoint)
  File "C:\Python27\lib\site-packages\nyawc\Crawler.py", line 95, in start_with
    self.__crawler_start()
  File "C:\Python27\lib\site-packages\nyawc\Crawler.py", line 166, in __crawler_start
    self.__spawn_new_requests()
  File "C:\Python27\lib\site-packages\nyawc\Crawler.py", line 111, in __spawn_new_requests
    if self.__spawn_new_request():
  File "C:\Python27\lib\site-packages\nyawc\Crawler.py", line 131, in __spawn_new_request
    self.__request_start(first_in_line)
  File "C:\Python27\lib\site-packages\nyawc\Crawler.py", line 230, in __request_start
    thread.start()
  File "C:\Python27\lib\threading.py", line 736, in start
    _start_new_thread(self.__bootstrap, ())
thread.error: can't start new thread_

from angularjs-csti-scanner.

I did not explicitly specify the number of threads (-mt option) and I the default number of threads the scanner takes is 8.
I think the scanner is trying to create too many threads and is going beyond the default value of 8.

Reference : https://johnsofteng.wordpress.com/2010/03/05/python-thread-error-cant-start-new-thread/

from angularjs-csti-scanner.

Tried specifying the number of threads using the -mt option. I still get the same error.

from angularjs-csti-scanner.

@jovyn Did you update the dependencies? If you run pip freeze the nyawc dependency must be on version 1.7.9.

from angularjs-csti-scanner.

Yes I did . The nyawc dependency is on 1.7.9

PS C:\Angular-CSTI-Scanner\Develop_Branch> pip freeze

acstis==3.0.2
alabaster==0.7.10
Babel==2.5.1
beautifulsoup4==4.6.0
certifi==2017.7.27.1
chardet==3.0.4
colorama==0.3.9
colorlog==2.10.0
docutils==0.14
idna==2.5
imagesize==0.7.1
Jinja2==2.9.6
lxml==4.0.0
MarkupSafe==1.0
nyawc==1.7.9
pockets==0.5.1
Pygments==2.2.0
pytz==2017.2
requests==2.18.1
requests-toolbelt==0.8.0
selenium==3.4.3
six==1.11.0
snowballstemmer==1.2.1
Sphinx==1.5.5
sphinx-better-theme==0.13
sphinxcontrib-napoleon==0.6.1
urllib3==1.21.1

from angularjs-csti-scanner.

Hi @jovyn, I tried to reproduce the issue again but unfortunately I did not succeed yet.

The thread count of the process is always 9 at my machine (8 + the main thread). Besides that the memory usage always stays at ~ 80MB (also if I'm scanning thousands of URLs).

Would it be possible for you to provide the information below?

1. What OS (including version) do you use?
2. What version of Python are you using?
3. What version of ACSTIS are you using?
4. How did you install ACSTIS?
5. Are you still using a proxy?
6. Could you sent me the exact command (including arguments) you are executing (e.g. python extended.py -d http://example.com)?
7. Would it be possible for you to share the URL you are scanning?
8. How much memory is the process using?
9. How many thread is the process using?

from angularjs-csti-scanner.

Hey @tijme ,

Below are my responses >
What OS (including version) do you use? -- Windows 10 (64-bit)
What version of Python are you using? -- Python 2.7.13
What version of ACSTIS are you using? -- Version 3.0.2
How did you install ACSTIS? -- I downloaded the .zip from (https://github.com/tijme/angularjs-csti-scanner/tree/develop) and then did a pip install --upgrade --force-reinstall .\angularjs-csti-scanner-develop.zip
Are you still using a proxy? -- Yes. however i tried acstis w/o the proxy settings as well.
Could you sent me the exact command (including arguments) you are executing (e.g. python extended.py -d http://example.com)? -- python .\extended.py -c -d "https://www.example.com/" -tc "Burp_Cert.pem" -mt 12 also tried acstis -c -d "https://www.example.com/" -tc "Burp_Cert.pem" -mt 12 . I have also tried without the -mt option as well as tried lesser threads (8 or 9)
Would it be possible for you to share the URL you are scanning? -- Sorry @tijme I wont be able to share the target URL.
How much memory is the process using? -- Not sure, will get back to you on this
How many thread is the process using? -- Not sure, will get back to you on this

from angularjs-csti-scanner.

@jovyn I was able to reproduce the issue and I found out it occurs on Windows only.

The scanner used a lot of memory per request since it cached the lxml tree of every response. When it reached the ~8000 requests it used an average of 2GB of memory. 2GB is the limit for a 32-bit application running on the 64-bit Windows 10 OS (source), which is why it crashed on your machine.

By removing the lxml tree caching you can now scan up to 60000 requests with 2GB of memory. I will continue to improve this in the future. If you want to scan more requests already you could try to install the 64-bit version of Python.

I just released this fix in version 3.0.5 of ACSTIS.

from angularjs-csti-scanner.

from angularjs-csti-scanner.