android - ContentProvider 路径遍历漏洞

Question

Google 就 ContentProvider 中的路径遍历漏洞发出警告
https://support.google.com/faqs/answer/7496913

Implementations of openFile and openAssetFile in exported ContentProviders can be vulnerable if they do not properly validate incoming Uri parameters. A malicious app can supply a crafted Uri (for example, one that contains “/../”) to trick your app into returning a ParcelFileDescriptor for a file outside of the intended directory, thereby allowing the malicious app to access any file accessible to your app.

他们的例子(来自上面的链接):

public ParcelFileDescriptor openFile (Uri uri, String mode)
 throws FileNotFoundException {
 File f = new File(DIR, uri.getLastPathSegment());
  if (!f.getCanonicalPath().startsWith(DIR)) {
    throw new IllegalArgumentException();
  }
 return ParcelFileDescriptor.open(f, ParcelFileDescriptor.MODE_READ_ONLY);
}

DIR 指的是什么？我如何实现正确的修复？

Answer 1

DIR指您要从中共享文件的应用程序内部存储中的目录。
这可能是:

File fd = getContext().getFilesDir();
String DIR = fd.getAbsolutePath() + "/public";

(结果:/data/user/0/[APPLICATION_ID]/files/public)
修复:
要解决此问题，您必须确保攻击者无法使用包含路径遍历字符(例如“../”)的恶意 URI 访问内部存储中的意外文件。
(例如，/data/user/0/[APPLICATION_ID]/databases 中的数据库文件是“意外文件”。)
正如 Google 建议的那样，您可以通过检查文件的规范路径来做到这一点。规范路径是绝对的，不包含“。”或“..”了。 (如果文件的规范路径以您的目录路径开头，一切都很好。)
例子:

恶意 URI:content://[APPLICATION_ID]/public/../../databases/database.db

恶意路径:/data/user/0/[APPLICATION_ID]/files/public/../../databases/database.db

规范路径:/data/user/0/[APPLICATION_ID]/databases/database.db

所以，这是完整的修复:

private String DIR; 

@Override
public boolean onCreate() {
    File fd = getContext().getFilesDir();
    DIR = fd.getAbsolutePath() + "/public";
    return true;
}

@Override
public ParcelFileDescriptor openFile(Uri uri, String mode)
        throws FileNotFoundException {
    File f = new File(DIR, uri.getLastPathSegment());
    if (!f.getCanonicalPath().startsWith(DIR)) {
        throw new IllegalArgumentException();
    }
    return ParcelFileDescriptor.open(f, ParcelFileDescriptor.MODE_READ_ONLY);
}

(旁注: DIR 应该是 dir ，因为它不是常数。)