wine/gecko
0
0
Fork 0
mirror of https://gitlab.winehq.org/wine/wine-gecko.git synced 2024-09-13 09:24:08 -07:00
Code Issues Packages Projects Releases Wiki Activity

Bug 765831 - Part 4: Clamp string lengths passed to getTextBeforeCursor/getTextAfterCursor. r=blassey

Browse Source
This commit is contained in:
Chris Peterson 2012-06-19 12:12:27 -07:00
parent 899a4bbb58
commit 7ea36bae56
1 changed files with 21 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
mobile/android/base/GeckoInputConnection.java
View File

@ -256,7 +256,16 @@ public class GeckoInputConnection
@Override
public CharSequence getTextBeforeCursor(int length, int flags) {
clampSelection();
// Avoid underrunning text buffer.
Span selection = clampSelection();
if (length > selection.start) {
length = selection.start;
}
if (length < 1) {
return "";
}
return super.getTextBeforeCursor(length, flags);
}
@ -268,7 +277,17 @@ public class GeckoInputConnection
@Override
public CharSequence getTextAfterCursor(int length, int flags) {
clampSelection();
// Avoid overrunning text buffer.
Span selection = clampSelection();
int contentLength = getEditable().length();
if (selection.end + length > contentLength) {
length = contentLength - selection.end;
}
if (length < 1) {
return "";
}
return super.getTextAfterCursor(length, flags);
}