search

tihmstar / futurerestore

A hacked up idevicerestore wrapper, which allows specifying SEP and Baseband for restoring
GNU Lesser General Public License v3.0
857 stars 278 forks source link

Futurerestore assumes all 64-bit devices have a SEP #193

Open kirb opened 6 years ago

kirb commented 6 years ago

In this block that determines whether the right set of flags have been used:

https://github.com/tihmstar/futurerestore/blob/4a8cabbe556d264df17379372f01f0523ba8d6c4/futurerestore/main.cpp#L189-L194

the logic assumes all 64-bit devices have a SEP. Also, futurerestore.cpp assumes an img4 device has a SEP. This isn’t true for iPad Air 1, iPad Mini 2, iPod touch 6th gen. (Maybe Apple TV as well? Watch? HomePod? Etc.) So maybe there should be a --no-sep flag.

(I guess this is another bug, but this logic should also be clearer about what arguments are missing.)

kirb commented 6 years ago

I’m mistaken, those iPads do have SEP, presumably iPod too, maybe Apple TV, and well, watch is 32-bit (Wikipedia says SEP was added in Series 2 though). Still, errors should be more clear about what’s missing.