tree 1316238fde8ff925cb72800c3934355c3311bec8
parent 2aabeed1fd68d119986146d74ffc6e5a7b9fdefa
author John Stultz <john.stultz@linaro.org> 1572999814 +0000
committer Yifan Hong <elsk@google.com> 1573004418 +0000

healthd: Fix incorrect return check

In commit 1d4368b4943b ("[REFACTOR] healthd: BatteryMonitor
use health 2.1 types"), the mapSysfsString() function was
changed to return NULL instead of -1 on error.

Everywher else that function is used commit 1d4368b4943b
tweaked the return value check from (ret < 0) to (!ret),
but one spot was missed.

This patch corrects the one missed check so that we compare
against null rather then negative value, since the return
value can no longer be negative.

This issue was found by inspection, and I've not seen an actual
problem with it, but thought it should be corrected.

Change-Id: I0a6564274d01276352a5dc0660b8ffbe748b5fde
Signed-off-by: John Stultz <john.stultz@linaro.org>