aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorKeisuke Kuroyanagi <ksk@google.com>2014-05-28 20:20:39 +0900
committerKeisuke Kuroyanagi <ksk@google.com>2014-05-28 20:20:39 +0900
commit7852765a503fe6dab54e1c4ab9e5e6b7bbdc70f2 (patch)
tree4eb16b4f02948a829fd5ddda2591d9dbba83e7df
parent6ee2cfa66c81162c2a42572ea63af1b5e38e339f (diff)
downloadlatinime-7852765a503fe6dab54e1c4ab9e5e6b7bbdc70f2.tar.gz
latinime-7852765a503fe6dab54e1c4ab9e5e6b7bbdc70f2.tar.xz
latinime-7852765a503fe6dab54e1c4ab9e5e6b7bbdc70f2.zip
Add length check for attaching a BoS marker.
Bug: 15297872 Change-Id: I7b0a27409b81b76561d19c6caa18a5011d3e90f6
-rw-r--r--native/jni/src/suggest/core/session/prev_words_info.h5
1 files changed, 4 insertions, 1 deletions
diff --git a/native/jni/src/suggest/core/session/prev_words_info.h b/native/jni/src/suggest/core/session/prev_words_info.h
index a58000abb..56c53c1c2 100644
--- a/native/jni/src/suggest/core/session/prev_words_info.h
+++ b/native/jni/src/suggest/core/session/prev_words_info.h
@@ -92,7 +92,7 @@ class PrevWordsInfo {
const DictionaryStructureWithBufferPolicy *const dictStructurePolicy,
const int *const wordCodePoints, const int wordCodePointCount,
const bool isBeginningOfSentence, const bool tryLowerCaseSearch) {
- if (!dictStructurePolicy || !wordCodePoints) {
+ if (!dictStructurePolicy || !wordCodePoints || wordCodePointCount > MAX_WORD_LENGTH) {
return NOT_A_DICT_POS;
}
int codePoints[MAX_WORD_LENGTH];
@@ -122,6 +122,9 @@ class PrevWordsInfo {
const DictionaryStructureWithBufferPolicy *const dictStructurePolicy,
const int *const wordCodePoints, const int wordCodePointCount,
const bool isBeginningOfSentence) {
+ if (!dictStructurePolicy || !wordCodePoints || wordCodePointCount > MAX_WORD_LENGTH) {
+ return NOT_A_DICT_POS;
+ }
int codePoints[MAX_WORD_LENGTH];
int codePointCount = wordCodePointCount;
memmove(codePoints, wordCodePoints, sizeof(int) * codePointCount);