search

barrykn / android-cm7-bkndh1

CyanogenMod personal dirty(?) hacks repo. Actually CM10.x as well as CM7, despite the name
1 stars 0 forks source link

fix CYAN-1602 for CM7.2 #11

Closed barrykn closed 11 years ago

barrykn commented 11 years ago

https://jira.cyanogenmod.org/browse/CYAN-1602

This issue is for fixing this in CM7.2, since that's what I have running in production. I might file separate issues for CM10.x.

barrykn commented 11 years ago

This should be fixed by commit 18e1eeb846ccc110fe343bdfbffd2cd75594eca1 (but as of this writing I have not even compiled it yet, never mind testing).

barrykn commented 11 years ago

Well... it compiles, and it boots on jordan, and it seems OK.

I think I'll close this now and I'll reopen it if anything goes wrong.

barrykn commented 11 years ago

Testing with Bluebox Secuirty Scanner shows my CM7.2-based ROM (after fixing this issue) to be vulnerable to neither 8219321 nor 9695860. That's a nice surprise!

Although, testing on a Samsung Droid Charge running Gingerbread shows that it's vulnerable to 8219321 but not 9695860, so that would indicate that Gingerbread was never vulnerable to 9695860 in the first place.

Anyway, I'm just glad that I don't have to do another build to fix 9695860 on Gingerbread. (For the Jellybean version of my ROM, merging CM 10.1.2 should be sufficient.)