commit 5c3148251a73c0c85a7e3f07de32603e4cffcf94
author Keisuke Kuroyanagi <ksk@google.com> Thu Oct 03 17:33:34 2013 +0900
committer Keisuke Kuroyanagi <ksk@google.com> Thu Oct 03 17:33:34 2013 +0900
tree 0a79e2abf66b323697f6e0161a782cd10f8a6a7e
parent 723da79c93c7b0d575cd87974b419f0f23cd5654

Fix: Native crash when an empty attribute is written. DO NOT MERGE

Bug: 10964805
Change-Id: I862a6b920a7a09eac4e012bfe75f087b2d7b4fe6

native/jni/src/suggest/policyimpl/dictionary/header/header_read_write_utils.cpp

diff --git a/native/jni/src/suggest/policyimpl/dictionary/header/header_read_write_utils.cpp b/native/jni/src/suggest/policyimpl/dictionary/header/header_read_write_utils.cpp
index 2694ce8..5ded8f6 100644
--- a/native/jni/src/suggest/policyimpl/dictionary/header/header_read_write_utils.cpp
+++ b/native/jni/src/suggest/policyimpl/dictionary/header/header_read_write_utils.cpp
@@ -139,6 +139,9 @@
         int *const writingPos) {
     for (AttributeMap::const_iterator it = headerAttributes->begin();
             it != headerAttributes->end(); ++it) {
+        if (it->first.empty() || it->second.empty()) {
+            continue;
+        }
         // Write a key.
         if (!buffer->writeCodePointsAndAdvancePosition(&(it->first.at(0)), it->first.size(),
                 true /* writesTerminator */, writingPos)) {