diff options
Diffstat (limited to 'chrome/browser/safe_browsing')
| -rw-r--r-- | chrome/browser/safe_browsing/download_protection_service.cc | 23 |
1 files changed, 18 insertions, 5 deletions
diff --git a/chrome/browser/safe_browsing/download_protection_service.cc b/chrome/browser/safe_browsing/download_protection_service.cc index 4217a21..c62bc4a 100644 --- a/chrome/browser/safe_browsing/download_protection_service.cc +++ b/chrome/browser/safe_browsing/download_protection_service.cc @@ -47,11 +47,24 @@ const char DownloadProtectionService::kDownloadRequestUrl[] = namespace { bool IsBinaryFile(const FilePath& file) { - return (file.MatchesExtension(FILE_PATH_LITERAL(".exe")) || - file.MatchesExtension(FILE_PATH_LITERAL(".cab")) || - file.MatchesExtension(FILE_PATH_LITERAL(".msi")) || - file.MatchesExtension(FILE_PATH_LITERAL(".crx")) || - file.MatchesExtension(FILE_PATH_LITERAL(".apk"))); + return ( + // Executable extensions for MS Windows. + file.MatchesExtension(FILE_PATH_LITERAL(".bas")) || + file.MatchesExtension(FILE_PATH_LITERAL(".bat")) || + file.MatchesExtension(FILE_PATH_LITERAL(".cab")) || + file.MatchesExtension(FILE_PATH_LITERAL(".cmd")) || + file.MatchesExtension(FILE_PATH_LITERAL(".com")) || + file.MatchesExtension(FILE_PATH_LITERAL(".exe")) || + file.MatchesExtension(FILE_PATH_LITERAL(".hta")) || + file.MatchesExtension(FILE_PATH_LITERAL(".msi")) || + file.MatchesExtension(FILE_PATH_LITERAL(".pif")) || + file.MatchesExtension(FILE_PATH_LITERAL(".reg")) || + file.MatchesExtension(FILE_PATH_LITERAL(".scr")) || + file.MatchesExtension(FILE_PATH_LITERAL(".vb")) || + file.MatchesExtension(FILE_PATH_LITERAL(".vbs")) || + // Chrome extensions and android APKs are also reported. + file.MatchesExtension(FILE_PATH_LITERAL(".crx")) || + file.MatchesExtension(FILE_PATH_LITERAL(".apk"))); } ClientDownloadRequest::DownloadType GetDownloadType(const FilePath& file) { |