diff options
author | 2014-03-24 14:28:36 +0900 | |
---|---|---|
committer | 2014-03-24 14:28:36 +0900 | |
commit | 4c9377043060d1b68ac408fd99adc91be4c99484 (patch) | |
tree | fbef90a8ea3d6465ff4c54e9de250ee9e9e1577c | |
parent | 7c20f140e0ab3efa25dd0708e1373b231339f5e2 (diff) | |
download | latinime-4c9377043060d1b68ac408fd99adc91be4c99484.tar.gz latinime-4c9377043060d1b68ac408fd99adc91be4c99484.tar.xz latinime-4c9377043060d1b68ac408fd99adc91be4c99484.zip |
Fix: wrong size checking in SparseTable.
Bug: 13592288
Change-Id: I52f03081c5abf25e3c160e2b8fff5e98f8448499
-rw-r--r-- | native/jni/src/suggest/policyimpl/dictionary/utils/sparse_table.cpp | 12 |
1 files changed, 5 insertions, 7 deletions
diff --git a/native/jni/src/suggest/policyimpl/dictionary/utils/sparse_table.cpp b/native/jni/src/suggest/policyimpl/dictionary/utils/sparse_table.cpp index c380429d9..810827dd0 100644 --- a/native/jni/src/suggest/policyimpl/dictionary/utils/sparse_table.cpp +++ b/native/jni/src/suggest/policyimpl/dictionary/utils/sparse_table.cpp @@ -41,13 +41,11 @@ uint32_t SparseTable::get(const int id) const { bool SparseTable::set(const int id, const uint32_t value) { const int posInIndexTable = getPosInIndexTable(id); // Extends the index table if needed. - if (mIndexTableBuffer->getTailPosition() < posInIndexTable) { - int tailPos = mIndexTableBuffer->getTailPosition(); - while(tailPos < posInIndexTable) { - if (!mIndexTableBuffer->writeUintAndAdvancePosition(NOT_EXIST, INDEX_SIZE, &tailPos)) { - AKLOGE("cannot extend index table. tailPos: %d to: %d", tailPos, posInIndexTable); - return false; - } + int tailPos = mIndexTableBuffer->getTailPosition(); + while (tailPos <= posInIndexTable) { + if (!mIndexTableBuffer->writeUintAndAdvancePosition(NOT_EXIST, INDEX_SIZE, &tailPos)) { + AKLOGE("cannot extend index table. tailPos: %d to: %d", tailPos, posInIndexTable); + return false; } } if (contains(id)) { |