Skip to content
This repository has been archived by the owner on Sep 6, 2019. It is now read-only.

Commit

Permalink
Limit parameters to 250 characters
Browse files Browse the repository at this point in the history
  • Loading branch information
M66B committed Feb 21, 2014
1 parent d0c4369 commit 56b5a71
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 0 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@ Changelog
* *storage/open* will also check media storage
* Clear restrictions will clear whitelists ([issue](/../../issues/1375))
* Whitelist management requires a [Pro license](http://www.xprivacy.eu/)
* Limit parameters to 250 characters (some applications use long Javascript for loadUrl)
* Updated Dutch translation
* Updated Lithuanian translation
* Updated Russian translation
Expand Down
4 changes: 4 additions & 0 deletions src/biz/bokhorst/xprivacy/PrivacyManager.java
Original file line number Diff line number Diff line change
Expand Up @@ -267,6 +267,10 @@ public static boolean getRestrictionExtra(final XHook hook, int uid, String rest
} else if (getHook(restrictionName, methodName) == null)
Util.log(hook, Log.WARN, "Unknown method=" + methodName);

// Check extra
if (extra != null && extra.length() > 250)
extra = extra.substring(0, 250) + "...";

// Check cache
boolean cached = false;
CRestriction key = new CRestriction(uid, restrictionName, methodName, extra);
Expand Down

0 comments on commit 56b5a71

Please sign in to comment.