fix false positive in phishing detection on some PDF files

PalmerAL · PalmerAL · commit 0b07e536a8da · 2016-05-05T19:41:01.000-05:00
diff --git a/js/webview/phishDetector.js b/js/webview/phishDetector.js
@@ -145,7 +145,7 @@ function checkPhishingStatus() {
 
 	if (loc.split("#")[0].split("/").length > 5) {
 		debug_phishing("long path found");
-		phishingScore += Math.max(loc.split("/").length * 0.05, 0.25);
+		phishingScore += Math.max(Math.min(loc.split("/").length * 0.05, 0.75), 0.25);
 	}
 
 	//CANTINA - penalize locations with lots of dots

Original file line number	Diff line number	Diff line change
`@@ -145,7 +145,7 @@ function checkPhishingStatus() {`
`145`	`145`
`146`	`146`	`if (loc.split("#")[0].split("/").length > 5) {`
`147`	`147`	`debug_phishing("long path found");`
`148`		`- phishingScore += Math.max(loc.split("/").length * 0.05, 0.25);`
	`148`	`+ phishingScore += Math.max(Math.min(loc.split("/").length * 0.05, 0.75), 0.25);`
`149`	`149`	`}`
`150`	`150`
`151`	`151`	`//CANTINA - penalize locations with lots of dots`