angr_ctf icon indicating copy to clipboard operation
angr_ctf copied to clipboard

Fix 07

Open Myts2 opened this issue 6 years ago • 4 comments

The original script doesn't work ,and I think it caused by angr's upgrade. I just fix some code to make it work on again. For example, the usage of 'angr.storage.SimFile' has changed,no longer to need file_options, and it can't load SimSymbolicMemory now. Other change is just adjustable.

Myts2 avatar Jul 19 '18 04:07 Myts2

I think it might help if you describe the changes you made.

solomonbstoner avatar Jul 22 '18 02:07 solomonbstoner

The original script doesn't work ,and I think it caused by angr's upgrade. I just fix some code to make it work on again. For example, the usage of 'angr.storage.SimFile' has changed,no longer to need file_options, and it can't load SimSymbolicMemory now. Other change is just adjustable.

Myts2 avatar Jul 22 '18 04:07 Myts2

What I meant was to add describe whatever you changed (and why you changed it) into the pull request. It makes it alot easier for the repo owner to evaluate your code and see if he wants to accept or reject your pull request. When you leave no description at all, no one knows what you're doing.

solomonbstoner avatar Jul 22 '18 10:07 solomonbstoner

Well....Alright. I'm sorry about this...

Myts2 avatar Jul 22 '18 13:07 Myts2