Unified Diff: lib/filterClasses.js
Issue 29861585:
Issue 6871 - Reject filters with blank CSPs (Closed)
Base URL: https://hg.adblockplus.org/adblockpluscore/
Patch Set: Address PS1 comments
Created Aug. 23, 2018, 5:12 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« no previous file
|
test/filterClasses.js » ('j')
|
test/filterClasses.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: lib/filterClasses.js |
| =================================================================== |
| --- a/lib/filterClasses.js |
| +++ b/lib/filterClasses.js |
| @@ -774,18 +774,22 @@ |
| if (contentType == null) |
| ({contentType} = RegExpFilter.prototype); |
| contentType &= ~type; |
| } |
| else |
| { |
| contentType |= type; |
| - if (type == RegExpFilter.typeMap.CSP && value) |
| + if (type == RegExpFilter.typeMap.CSP) |
| + { |
| + if (!value) |
| + return new InvalidFilter(origText, "filter_invalid_csp"); |
| csp = value; |
| + } |
| } |
| } |
| else |
| { |
| switch (option.toLowerCase()) |
| { |
| case "match-case": |
| matchCase = !inverse; |
« no previous file with comments
|
« no previous file
|
test/filterClasses.js » ('j')
|
test/filterClasses.js » ('J')
