commit 9df20155f635d91e62d679274906e437150765d2
author Gwendal Grignou <gwendal@google.com> Tue May 03 19:33:20 2016 -0700
committer Gwendal Grignou <gwendal@google.com> Wed Jun 15 16:53:50 2016 -0700
tree 0d63af5b7f7dc2f70d11a7175464ff29de795498
parent 9681d5700a5b429c8fb40f8a5e8865d250dc2cd2

sensors: Fix error at init if VPD table is not present.

Initialize the variable array before filling it.
Found the problem on cyan-cheets, dragon uses similar sensor_hal.

BUG=chrome-os-partner:45679
TEST=On cyan, if VPD is missing, check error message is gone.

Change-Id: Ide273ad1be0f4016eb2ff53e250990b470d75600
Signed-off-by: Gwendal Grignou <gwendal@google.com>

sensor_hub/sensors.cpp

diff --git a/sensor_hub/sensors.cpp b/sensor_hub/sensors.cpp
index 0b7cd20..3ec531d 100644
--- a/sensor_hub/sensors.cpp
+++ b/sensor_hub/sensors.cpp
@@ -355,6 +355,8 @@
     for (int i = X ; i < MAX_AXIS; i++) {
         snprintf(calib_key[i], sizeof(calib_key[i]), "%s_%c_calibbias",
                 cros_ec_iio_axis_names[sensor_type], 'x' + i);
+    }
+    for (int i = X ; i < MAX_AXIS; i++) {
         if (cros_ec_sysfs_get_attr(vpd_path, calib_key[i], calib_value[i])) {
             ALOGI("Calibration key %s missing.\n", calib_key[i]);
             calib_data_valid = false;