Fixed a glitch in the ComponentCloudPolicyUpdater where a protobuf field was read before being vali…

chrome/browser/policy/component_cloud_policy_updater.cc

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/policy/component_cloud_policy_updater.h"
 
 #include <string>
 
 #include "base/bind.h"
 #include "base/location.h"
@@ skipping 333 matching lines @@
 
   // Validate the policy before doing anything else.
   PolicyNamespace ns;
   em::ExternalPolicyData data;
   if (!store_->ValidatePolicy(response.Pass(), &ns, &data)) {
     LOG(ERROR) << "Failed to validate component policy fetched from DMServer";
     return;
   }
 
   // Maybe the data for this hash has already been downloaded and cached.
-  if (data.secure_hash() == store_->GetCachedHash(ns))
+  if (data.has_secure_hash() &&
+      data.secure_hash() == store_->GetCachedHash(ns)) {
     return;
+  }
 
   // TODO(joaodasilva): implement the other two auth methods.
   if (data.download_auth_method() != em::ExternalPolicyData::NONE)
     return;
 
   // Check for an existing job for this component.
   FetchJob* job = fetch_jobs_[ns];
   if (job) {
     // Check if this data has already been seen.
     if (job->ParamsEquals(data))
@@ skipping 50 matching lines @@
   DCHECK(fetch_jobs_[job->policy_namespace()] == job);
   DCHECK(!job_queue_.empty() && job_queue_.front() == job);
   // The job isn't deleted when it fails because a retry attempt may have been
   // scheduled. It's also kept so that UpdateExternalPolicy() can see the
   // current data, and avoid a new fetch if the data hasn't changed.
   job_queue_.pop();
   StartNextJob();
 }
 
 }  // namespace policy