commit bbd056ac4ac206bea36e6e8b40bf40c4a2d44f25
author Yangster-mac <yanglu@google.com> Mon Jan 22 13:37:02 2018 -0800
committer Yang Lu <yanglu@google.com> Tue Jan 23 16:41:52 2018 +0000
tree 6848813c5efe289ecaea32854b6a429c6086ea43
parent 4953824b89bfbdd5a85a2a7182c0903d9283e1d1

Fix the crash when parsing empty config.

Test: statsd test passed.
Change-Id: Ice4bcb7368ba416a9ffe6e9c89129b307302980e

cmds/statsd/src/StatsService.cpp

diff --git a/cmds/statsd/src/StatsService.cpp b/cmds/statsd/src/StatsService.cpp
index ca097d0..ba628b8 100644
--- a/cmds/statsd/src/StatsService.cpp
+++ b/cmds/statsd/src/StatsService.cpp
@@ -725,7 +725,7 @@
     if (checkCallingPermission(String16(kPermissionDump))) {
         ConfigKey configKey(ipc->getCallingUid(), key);
         StatsdConfig cfg;
-        if (!cfg.ParseFromArray(&config[0], config.size())) {
+        if (config.empty() || !cfg.ParseFromArray(&config[0], config.size())) {
             *success = false;
             return Status::ok();
         }