cromo: Fix a crash due to setting a DBus error more than once.

DBus::Error::set() can only be called once to set the error, or
otherwise, the underlying call to dbus_set_error() would fail and abort
the process. This CL changes SmsCache::List() to report only the first
error when iterating through the list of SMS messages.

BUG=chromium:380494
TEST=Tested the following:
1. `USE='gobi cellular' FEATURES=test emerge-$BOARD platform2`
2. Test receiving SMS messages on a Gobi3k modem.

Change-Id: I1e3d886717dec9ac92624216a184c6f7bce5aec1
Reviewed-on: https://chromium-review.googlesource.com/202560
Tested-by: Ben Chan <benchan@chromium.org>
Reviewed-by: Prathmesh Prabhu <pprabhu@chromium.org>
Commit-Queue: Ben Chan <benchan@chromium.org>
1 file changed