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

fix : pwn/glibc-heap/fastbin_attack #438

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix : pwn/glibc-heap/fastbin_attack #438

wants to merge 1 commit into from

Conversation

ly-test-fuzz
Copy link

2014 hack.lu oreo 's exp have a small mistake .
payload not work as note

2014 hack.lu oreo 's exp  have a small mistake . 
payload not work as  note
@iromise
Copy link
Member

iromise commented Oct 29, 2018

hi, when I write this wp, it works on ubuntu 16.04. Your environment is ?

@ly-test-fuzz
Copy link
Author

my fault; I mean the payload work don't like what you said in the comments.

hi, when I write this wp, it works on ubuntu 16.04. Your environment is ?

@ly-test-fuzz ly-test-fuzz reopened this Nov 2, 2018
@iromise
Copy link
Member

iromise commented Nov 6, 2018

oh, just leave it here, when I review the code, I will deal with issue.

@WinMin
Copy link
Contributor

WinMin commented Apr 21, 2019

@iromise 这个check 放太久了 - -

@iromise
Copy link
Member

iromise commented Apr 22, 2019

但我确实没时间看。。 @WinMin

@AngelKitty
Copy link
Member

@iromise 所以我觉得师傅应该看一下了,然后确定要不要解决一下这个conflict

@bash-c
Copy link
Contributor

bash-c commented May 2, 2020

This is such a long time. Sorry for the delay.
Anyway I checked this script and approve this pr. However this branch has some conflicts now, which should be resolved firstly.
@ly-test-fuzz @iromise

@bash-c
Copy link
Contributor

bash-c commented May 2, 2020

And I think there should be related modification in the EN version and ctf-challenges repo too

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

Successfully merging this pull request may close these issues.

None yet

5 participants