tree c8a4ce8527106bb9bb2f20155cd35475541a4552
parent d7131ffc9a9f24c130fe169bbb73aa1eca7d998b
author Andre Furtado <afurtado@google.com> 1471541317 -0700
committer Liisa Anttonen <liisa.anttonen@nedevicesw.com> 1481809465 +0200

30481342: Security Vulnerability - TOCTOU in MmsProvider allows access to files as phone (radio) uid

Problem: MmsProvider.openFile validated the current _data column
in the DB and then called ContentProvider.openFileHelper which was again
reading from the DB. A race condition could cause the second DB read to
read an updated, malicious value.

Fix: instead of doing the first DB check and calling
ContentProvider.openFileHelper, we're now just calling
MmsProvider.safeOpenFileHelper which does a single check.

Test: used the POC provided for this incident.

b/30481342

Change-Id: I653129359130b9fae59d4c355320b266c158a698
(cherry picked from commit 5bc7f9682d72c89ba252be6471b2db9b7e7815e3)