commit b4fccd24acb3b2731a6ba62dd923f20d1d2a697d
author Darin Petkov <petkov@chromium.org> Fri Aug 10 11:59:26 2012 +0200
committer Gerrit <chrome-bot@google.com> Mon Aug 13 03:34:45 2012 -0700
tree eddf2fcf93b98682e3c3a96571df83a5d940d178
parent 3c508e19fecd24dd8872d493c5cc6dfdfbe70f17

shill: cellular: Avoid null pointer dereference if provider has no networks.

Also:

- Add a SetHomeProvider unit tests to CellularCapabilityUniversal.
- Free the test broadband provider database in CellularCapabilityUniversalTest.

BUG=chromium-os:33375
TEST=unit tests, tested on device

Change-Id: I37086954e51500840d6aa424964c973fccc6b22b
Reviewed-on: https://gerrit.chromium.org/gerrit/29872
Tested-by: Darin Petkov <petkov@chromium.org>
Reviewed-by: Ben Chan <benchan@chromium.org>
Commit-Ready: Darin Petkov <petkov@chromium.org>

cellular_capability_gsm.cc

diff --git a/cellular_capability_gsm.cc b/cellular_capability_gsm.cc
index af4e2df..a366ec2 100644
--- a/cellular_capability_gsm.cc
+++ b/cellular_capability_gsm.cc
@@ -473,7 +473,7 @@
   }
   home_provider_ = provider;
   Cellular::Operator oper;
-  if (provider->networks) {
+  if (provider->networks && provider->networks[0]) {
     oper.SetCode(provider->networks[0]);
   }
   if (provider->country) {