tree 5357bdeff5614a6b11d33672531d49cd79e1b6ac
parent dfccb0c7959ecbfe86eff772d73a0d57af66c847
author Ben Chan <benchan@chromium.org> 1401851947 -0700
committer chrome-internal-fetch <chrome-internal-fetch@google.com> 1401932207 +0000

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>