search

nowsecure / dirtycow

radare2 IO plugin for Linux and Android. Modifies files owned by other users via dirtycow Copy-On-Write cache vulnerability
https://www.nowsecure.com/blog/2016/10/21/dirty-cow-vulnerability-mobile-impact/
GNU Lesser General Public License v3.0
93 stars 25 forks source link

Cowpy issue #8

Closed HemanthJabalpuri closed 5 years ago

HemanthJabalpuri commented 5 years ago

Sir I successfully changed the prop value in /system/build.prop with cowpy.. But after reboot it resets to the original value....

trufae commented 5 years ago

That’s exactly how dirtycow is suposed to work.

On 11 May 2019, at 04:59, HemanthJabalpuri notifications@github.com wrote:

Sir I successfully changed the prop value in /system/build.prop with cowpy.. But after reboot it resets to the original value....

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

HemanthJabalpuri commented 5 years ago

@trufae in build.sh, I think it is if ! [ -d "$T" ]; then... there should be not (!) right?

trufae commented 5 years ago

This path only exists in termux. And thats where the script is suposed to run. So the conditional is correct. For other platforms use the makefile

On 11 May 2019, at 13:05, HemanthJabalpuri notifications@github.com wrote:

@trufae in build.sh, I think it is if ! [ -d $T ]; then... there should be not (!) right?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

HemanthJabalpuri commented 5 years ago

This path only exists in termux. And thats where the script is suposed to run. So the conditional is correct. For other platforms use the makefile … On 11 May 2019, at 13:05, HemanthJabalpuri @.***> wrote: @trufae in build.sh, I think it is if ! [ -d $T ]; then... there should be not (!) right? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

If there is no NOT(!) then it will exit in Termux.... according to my basic scripting...

trufae commented 5 years ago

fixed it, yes you are right

HemanthJabalpuri commented 5 years ago

Sir @trufae I made a wrapper script for cowpy to correctly overwriting file which is less in size than destination file...

cowpy.sh.txt

trufae commented 5 years ago

Can you submit it in a PR to review the changes?

On 12 May 2019, at 06:06, HemanthJabalpuri notifications@github.com wrote:

Sir @trufae I made a wrapper script for cowpy to correctly overwriting file which is less in size than destination file...

cowpy.sh.txt

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

HemanthJabalpuri commented 5 years ago

Can you submit it in a PR to review the changes?

Done.