Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Google] Sulley crash from sessions.py referencing self.fuzz_node.mutant.s_type #10

Closed
Fitblip opened this issue Apr 13, 2012 · 1 comment

Comments

@Fitblip
Copy link
Member

Fitblip commented Apr 13, 2012

What steps will reproduce the problem?

  1. Simple HTTP fuzzer against local XP target
  2. procmon detects access violation on a test case
  3. sessions.py references a null variable, crashes

What is the expected output? What do you see instead?
In the Sulley script output, I expect to see lines like this:

[03:23.34] xmitting: [1.2673]
[03:23.35] fuzzing 2674 of 18655
[03:23.35] xmitting: [1.2674]
[03:23.36] procmon detected access violation on test case #2674
[03:23.36] primitive lacks a name, type: string, default value: www.foo.bar
[03:23.36] [INVALID]:22732522 Unable to disassemble at 22732522 from thread 2240 caused access violation

And in this specific script, I see this exact output for faults discovered
prior to the Sulley crash. When I get the crash, this is what I see:

[03:24.19] fuzzing 3731 of 18655
[03:24.19] xmitting: [1.3731]
[03:24.20] procmon detected access violation on test case #3731
_Traceback (most recent call last):
File "http.py", line 74, in
sess.fuzz()
File "C:\dev\sulley\sulley\sessions.py", line 495, in fuzz
self.poll_pedrpc(target)
File "C:\dev\sulley\sulley\sessions.py", line 638, in poll_pedrpc
msg += "type: %s, default value: %s" % (self.fuzz_node.mutant.s_type,
self.fuzz_node.mutant.original_value)
AttributeError: repeat instance has no attribute 's_type'
_

What version of the product are you using? On what operating system?
Sulley up-to-date SVN as of 1/14/2009. This is Windows XP SP3.

Please provide any additional information below.
I'm not sure how self.fuzz_node.mutant loses s_type, but I added a little
try/except loop around it as a workaround for the time being.

Thanks,
-Josh

@Fitblip
Copy link
Member Author

Fitblip commented Sep 15, 2012

This is a fairly outdated version, and I've never experienced it myself. Closing.

@Fitblip Fitblip closed this as completed Sep 15, 2012
jtpereyda pushed a commit to jtpereyda/sulley that referenced this issue Nov 24, 2015
…-sel:warning-fix-sessions to develop

Inspection-only merge. Incidental fix from FUZZ-32.

* commit 'cb3f9bed99fae7dfb946e7c073cbb8b6b9f762dc':
  Removed `import signal` check; the import doesn't crash on Windows, only the use of signal.pause(). This eliminates a warning and some code bloat.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant