| Index: Shared/PluginFilter.cpp |
| =================================================================== |
| --- a/Shared/PluginFilter.cpp |
| +++ b/Shared/PluginFilter.cpp |
| @@ -824,32 +824,13 @@ |
| { |
| int filterType = 0; |
| - // We need to categorize the filters |
| - // We have three options, whitelist, block or element hiding |
| // See http://adblockplus.org/en/filters for further documentation |
| - // @@ indicates white listing rule |
| - if (filter.Find(L"@@") == 0) |
| - { |
| - filterType = CFilter::filterTypeWhiteList; |
| - |
| - filter.Delete(0, 2); |
| - } |
| // If a filter contains ## then it is a element hiding rule |
| - else if (filter.Find(L"#") >= 0) |
| + if (filter.Find(L"#") >= 0) |
| { |
| filterType = CFilter::filterTypeElementHide; |
| } |
| - //Anything we do not support here |
| - else if (filter.Find(L"*") == 0) |
| - { |
| - filterType = CFilter::filterTypeUnknown; |
| - } |
| - // Else, it is a general rule |
| - else |
| - { |
| - filterType = CFilter::filterTypeBlocking; |
| - } |
| try |
| { |
| @@ -860,7 +841,9 @@ |
| } |
| catch(...) |
| { |
| - //just ignore all errors we might get when adding filters |
| +#ifdef ENABLE_DEBUG_RESULT |
| + CPluginDebug::DebugResult(L"Error loading hide filter: " + filter); |
| +#endif |
| } |
| } |
| } |
| @@ -893,273 +876,6 @@ |
| } |
| -bool CPluginFilter::IsMatchFilter(const CFilter& filter, CString src, const CString& srcDomain, const CString& domain) const |
| -{ |
| - // Initial checks |
| - |
| - // $match_case |
| - if (!filter.m_isMatchCase) |
| - { |
| - src.MakeLower(); |
| - } |
| - |
| - // $domain |
| - if (!filter.m_domains.empty()) |
| - { |
| - bool bFound = false; |
| - |
| - for (std::set<CString>::const_iterator it = filter.m_domains.begin(); !bFound && it != filter.m_domains.end(); ++it) |
| - { |
| - bFound = domain == *(it) || IsSubdomain(domain, *it); |
| - } |
| - |
| - if (!bFound) |
| - { |
| - return false; |
| - } |
| - } |
| - |
| - // $domain ~ |
| - if (!filter.m_domainsNot.empty()) |
| - { |
| - for (std::set<CString>::const_iterator it = filter.m_domainsNot.begin(); it != filter.m_domainsNot.end(); ++it) |
| - { |
| - if (domain == *(it) || IsSubdomain(domain, *it)) |
| - { |
| - return false; |
| - } |
| - } |
| - } |
| - |
| - // $third_party |
| - if (filter.m_isThirdParty) |
| - { |
| - if (srcDomain == domain || IsSubdomain(srcDomain, domain)) |
| - { |
| - return false; |
| - } |
| - } |
| - |
| - // $third_party ~ |
| - if (filter.m_isFirstParty) |
| - { |
| - if (srcDomain != domain && !IsSubdomain(srcDomain, domain)) |
| - { |
| - return false; |
| - } |
| - } |
| - |
| - // "regex" checks |
| - |
| - int startPos = 0; |
| - int srcLength = src.GetLength(); |
| - UINT indexEnd = filter.m_stringElements.size() - 1; |
| - |
| - for (UINT index = 0; index <= indexEnd; index++) |
| - { |
| - if (index == 0 && filter.m_isFromStartDomain) |
| - { |
| - CString loweredDomain = srcDomain; |
| - int domainPos = src.Find(loweredDomain.MakeLower()); |
| - int lastPos = src.Find('/', domainPos); |
| - |
| - bool bFoundDomain = false; |
| - bool bContinueDomainSearch = true; |
| - |
| - while (bContinueDomainSearch) |
| - { |
| - if (domainPos == FindMatch(src, filter.m_stringElements[index])) |
| - { |
| - bContinueDomainSearch = false; |
| - bFoundDomain = true; |
| - } |
| - else |
| - { |
| - domainPos = src.Find('.', domainPos + 1) + 1; |
| - if (domainPos == 0 || (domainPos >= lastPos && lastPos >= 0)) |
| - { |
| - bContinueDomainSearch = false; |
| - } |
| - } |
| - } |
| - |
| - if (!bFoundDomain) |
| - { |
| - return false; |
| - } |
| - } |
| - |
| - startPos = FindMatch(src, filter.m_stringElements[index], startPos); |
| - if (startPos < 0) |
| - { |
| - return false; |
| - } |
| - |
| - int length = filter.m_stringElements[index].GetLength(); |
| - |
| - // Check from start |
| - if (index == 0 && filter.m_isFromStart && startPos > 0) |
| - { |
| - return false; |
| - } |
| - |
| - // Check from end |
| - if (index == indexEnd && filter.m_isFromEnd && startPos + length != srcLength) |
| - { |
| - return false; |
| - } |
| - |
| - startPos += length; |
| - } |
| - |
| - return true; |
| -} |
| - |
| - |
| -const CFilter* CPluginFilter::MatchFilter(int filterType, const CString& src, int contentType, const CString& domain) const |
| -{ |
| - const CFilter* filter = NULL; |
| - |
| - int startCharacter = 0; |
| - int keyLength = 4; |
| - |
| - CString srcLower = src; |
| - srcLower.MakeLower(); |
| - int srcLowerLength = srcLower.GetLength(); |
| - |
| - // Extract src domain |
| - DWORD length = 2048; |
| - CString srcDomain; |
| - |
| - if (SUCCEEDED(::UrlGetPart(src, srcDomain.GetBufferSetLength(2048), &length, URL_PART_HOSTNAME, 0))) |
| - { |
| - srcDomain.ReleaseBuffer(); |
| - |
| - if (srcDomain.Left(4) == L"www.") |
| - { |
| - srcDomain = srcDomain.Right(srcDomain.GetLength() - 4); |
| - } |
| - else if (srcDomain.Left(5) == L"www2." || srcDomain.Left(5) == L"www3.") |
| - { |
| - srcDomain = srcDomain.Right(srcDomain.GetLength() - 5); |
| - } |
| - } |
| - else |
| - { |
| - srcDomain.ReleaseBuffer(); |
| - srcDomain.Empty(); |
| - } |
| - |
| - // Search in filter map |
| - s_criticalSectionFilterMap.Lock(); |
| - { |
| - const TFilterMap* filterMap = m_filterMap[filterType]; |
| - |
| - if (srcLowerLength >= 7) |
| - { |
| - if (srcLower.Find(L"http://") == 0) |
| - { |
| - startCharacter = 7; |
| - } |
| - else if (srcLower.Find(L"https://") == 0) |
| - { |
| - startCharacter = 8; |
| - } |
| - } |
| - |
| - DWORD dwKey = 0; |
| - |
| - while (filter == NULL && srcLowerLength >= startCharacter + keyLength) |
| - { |
| - if (dwKey == 0) |
| - { |
| - dwKey = (srcLower.GetAt(startCharacter) << 24) | (srcLower.GetAt(startCharacter+1) << 16) | (srcLower.GetAt(startCharacter+2) << 8) | srcLower.GetAt(startCharacter+3); |
| - } |
| - else |
| - { |
| - dwKey <<= 8; |
| - dwKey |= srcLower.GetAt(startCharacter+3); |
| - } |
| - |
| - TFilterMap::const_iterator foundEntry = filterMap[0].find(dwKey); |
| - if (foundEntry != filterMap[0].end()) |
| - { |
| - if (((foundEntry->second.m_contentType & contentType) || foundEntry->second.m_contentType == CFilter::contentTypeAny) && IsMatchFilter(foundEntry->second, src, srcDomain, domain)) |
| - { |
| - filter = &(foundEntry->second); |
| - break; |
| - } |
| - } |
| - |
| - // No match - increment the start character |
| - startCharacter++; |
| - } |
| - |
| - // Second list |
| - if (filter == NULL) |
| - { |
| - dwKey = 0; |
| - startCharacter = 0; |
| - |
| - if (srcLowerLength >= 7) |
| - { |
| - if (srcLower.Find(L"http://") == 0) |
| - { |
| - startCharacter = 7; |
| - } |
| - else if (srcLower.Find(L"https://") == 0) |
| - { |
| - startCharacter = 8; |
| - } |
| - } |
| - |
| - while (filter == NULL && srcLowerLength >= startCharacter + keyLength) |
| - { |
| - if (dwKey == 0) |
| - { |
| - dwKey = (srcLower.GetAt(startCharacter) << 24) | (srcLower.GetAt(startCharacter+1) << 16) | (srcLower.GetAt(startCharacter+2) << 8) | srcLower.GetAt(startCharacter+3); |
| - } |
| - else |
| - { |
| - dwKey <<= 8; |
| - dwKey |= srcLower.GetAt(startCharacter+3); |
| - } |
| - |
| - TFilterMap::const_iterator foundEntry = filterMap[1].find(dwKey); |
| - if (foundEntry != filterMap[1].end()) |
| - { |
| - if (((foundEntry->second.m_contentType & contentType) || foundEntry->second.m_contentType == CFilter::contentTypeAny) && IsMatchFilter(foundEntry->second, src, srcDomain, domain)) |
| - { |
| - filter = &(foundEntry->second); |
| - break; |
| - } |
| - } |
| - |
| - // No match - increment the start character |
| - startCharacter++; |
| - } |
| - } |
| - |
| - // Search in default filter map (try all filters) |
| - if (filter == NULL) |
| - { |
| - for (TFilterMapDefault::const_iterator it = m_filterMapDefault[filterType].begin(); it != m_filterMapDefault[filterType].end(); ++it) |
| - { |
| - if (((it->m_contentType & contentType) || it->m_contentType == CFilter::contentTypeAny) && IsMatchFilter(*it, src, srcDomain, domain)) |
| - { |
| - filter = &(*it); |
| - break; |
| - } |
| - } |
| - } |
| - |
| - } |
| - s_criticalSectionFilterMap.Unlock(); |
| - |
| - return filter; |
| -} |
| - |
| - |
| bool CPluginFilter::ShouldWhiteList(CString src) const |
| { |
| // We should not block the empty string, so all filtering does not make sense |
| @@ -1169,9 +885,8 @@ |
| return false; |
| } |
| - const CFilter* filter = MatchFilter(CFilter::filterTypeWhiteList, src, CFilter::contentTypeDocument, ""); |
| - |
| - return filter ? true : false; |
| + //TODO: Implement whitelisting check from libadblockplus here |
| + return false; |
| } |
| @@ -1201,12 +916,13 @@ |
| CPluginClient* client = CPluginClient::GetInstance(); |
| AdblockPlus::FilterEngine* filterEngine = client->GetFilterEngine(); |
| -// src.OemToCharA(); |
| + //TODO: Make sure if the content type names are in sync with libadblockplus |
| + std::string contentTypeString = CT2A(type, CP_UTF8); |
| - std::string contentTypeString = ""; |
| + CT2CA srcAnsi(src, CP_UTF8); |
| + std::string url(srcAnsi); |
| - CT2CA srcAnsi(src); |
| - std::string url(srcAnsi); |
| + //TODO: figure out domain passing for whitelisting |
| if (filterEngine->Matches(url, contentTypeString)) |
| { |
| if (addDebug) |
| @@ -1220,36 +936,6 @@ |
| return true; |
| } |
| return false; |
| - |
| - //The following is for reference only |
| - |
| - const CFilter* blockFilter = MatchFilter(CFilter::filterTypeBlocking, src, contentType, domain); |
| - if (blockFilter) |
| - { |
| - const CFilter* whiteFilter = MatchFilter(CFilter::filterTypeWhiteList, src, contentType, domain); |
| - if (whiteFilter) |
| - { |
| - if (addDebug) |
| - { |
| - DEBUG_FILTER("Filter::ShouldBlock " + type + " NO src:" + src + " - whitelist:\"" + whiteFilter->m_filterText + "\""); |
| - } |
| - blockFilter = NULL; |
| - } |
| - else if (addDebug) |
| - { |
| - DEBUG_FILTER("Filter::ShouldBlock " + type + " YES src:" + src + " - \"" + blockFilter->m_filterText + "\"") |
| - |
| -#ifdef ENABLE_DEBUG_RESULT |
| - CPluginDebug::DebugResultBlocking(type, src); |
| -#endif |
| - } |
| - } |
| - else if (addDebug) |
| - { |
| - DEBUG_FILTER("Filter::ShouldBlock " + type + " NO src:" + src) |
| - } |
| - |
| - return blockFilter ? true : false; |
| } |
| int CPluginFilter::FindMatch(const CString& src, CString filterPart, int srcStartPos) const |