vusec / drammer

Native binary for testing Android phones for the Rowhammer bug
484 stars 164 forks source link

Permission denied error in Galaxy S7 (G930F) #15

Open ThallesAraujo opened 6 years ago

ThallesAraujo commented 6 years ago

Device information: Android Version: 8.0.0 Oreo UX Version: Samsung Experience 9.0 Android Security Patch Level: April 1, 2018 (Occurs both in default and aggressive modes) screenshot_20180628-164006_drammer

vvdveen commented 6 years ago

This may have been an issue in how we invoke the native binary from the app. I pushed an updated version of the apk on Friday morning (June 29) that should address this. Maybe you can remove the app and redownload it?

Thanks, Victor

ThallesAraujo commented 6 years ago

Ok, I did it and I guess this time it worked. I was able to see the output. I did the test with apps closed and rebooting the device, too (as recomended in output. This was a result after restarting).

screenshot_20180701-175952_drammer

vvdveen commented 6 years ago

Yes, this looks better. I have seen similar output before and it seems that (some) galaxy devices disabled access to some of the files in /proc/. This makes it harder for our app to figure out when to start hammering. I will have to think about how to address this.

In general, I think that it is a good strategy from Samsung to disable read access to those files.