Dont use isPotentialEmergency for our emergency logic.
isPotentialEmergency number is meant to prevent 3rd party apps from circumventing our restriction on dialing emergency numbers (only default dialer can do that). 3rd parties could potentially circumvent it by dialing an emergency-prefixed number like 911234 since some carriers will dial emergency services for those numbers as well. isPotentialEmergency checks prefixes as well to prevent this. A bug turned up where we were treating too many calls as emergency and this is because we were too liberal with our use of isPotentialEmergency(). We needed to only use in the case above but we ended up using it all over telecom which caused our 'ask every time' feature to be disabled 911234 even in markets where that is a valid non-emergency number. The fix is to replace our isPotentialEmergency check with isLocalEmergency for the majority of telecom logic. This puts things at par with the correct behavior from kitkat. Bug: 21650423 Change-Id: I075ea9937b54e0b4bc4e158943652c30319e62fd
Loading
Please register or sign in to comment