Skip to content
This repository has been archived by the owner on Apr 28, 2023. It is now read-only.

First version of angr courses - WIP #74

Merged
merged 6 commits into from
Sep 20, 2016
Merged

First version of angr courses - WIP #74

merged 6 commits into from
Sep 20, 2016

Conversation

P1kachu
Copy link
Contributor

@P1kachu P1kachu commented Jun 6, 2016

(The only interesting files for this PR are the ones in this commit)

Here is a first idea on how the angr courses could be written.
I was planning on creating a file where the explanation would NOT be in the comments of the python code, but I am not sure it would have worked with the test.py script.
No need to merge it right now though. Some more could be added before I think, unless you like this and want to provide a template for others.

Tell me what you think, so that I can continue or fix :)

Signed-off-by: Stanislas P1kachu Lejay <p1kachu@lse.epita.fr>
Signed-off-by: Stanislas P1kachu Lejay <p1kachu@lse.epita.fr>
Signed-off-by: Stanislas P1kachu Lejay <p1kachu@lse.epita.fr>
Signed-off-by: Stanislas P1kachu Lejay <p1kachu@lse.epita.fr>
@P1kachu P1kachu changed the title First version of angr courses First version of angr courses - WIP Jun 6, 2016
@P1kachu P1kachu mentioned this pull request Jun 9, 2016
Signed-off-by: Stanislas P1kachu Lejay <p1kachu@lse.epita.fr>
@bananaappletw
Copy link
Contributor

It this pr still going on?

I try to learn angr recently.

I am glad to see any tutorial that helps people to understand more about angr.

Thanks.

@P1kachu
Copy link
Contributor Author

P1kachu commented Sep 6, 2016

I am waiting for some upstream approval before going any further :)

@ltfish
Copy link
Member

ltfish commented Sep 6, 2016

@P1kachu sorry, we didn't realize that you are waiting for feedbacks from us.

We believe having angr courses will be awesome. However, since angr is still evolving in a fast pace, online angr courses will have to be adapted constantly to the latest angr design and implementation. It'll be great if there are more people (I mean, more than just yourself) is involved in this process.

For more feedbacks, I'm kindly poking @zardus :-)

@ssatanss
Copy link

Good job keep them comming :D

@zardus
Copy link
Member

zardus commented Sep 12, 2016

Yo! Sorry, this got buried in the CGC rush.

I think this is a great start. When one of us has a moment, we'll get it hooked into our CI (should just be a matter of adding the .md to the doc-tester) and then merge it in.

Going forward, I think it'd be very useful to have a lesson on the static analysis in angr, and how/when one would apply it. I'd be interested in learning that as well ;-)

@rhelmot rhelmot merged commit b5311ce into angr:master Sep 20, 2016
@rhelmot
Copy link
Member

rhelmot commented Sep 20, 2016

In the future, please put fixes unrelated to the subject of a PR in a separate PR 😜

@rhelmot
Copy link
Member

rhelmot commented Sep 20, 2016

I added this to CI with bdd0af7!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants