Safebrowsing: Honor the metadata from malware fullhash results in SB API 3.0.

chrome/browser/safe_browsing/protocol_parser.cc

 // Copyright (c) 2012 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.
 //
 // Parse the data returned from the SafeBrowsing v2.1 protocol response.
 
 // TODOv3(shess): Review these changes carefully.
 
 #include <stdlib.h>
 
@@ skipping 113 matching lines @@
     return true;
   }
 
  private:
   const char* data_;
   size_t length_;
 
   DISALLOW_COPY_AND_ASSIGN(BufferReader);
 };
 
-bool ParseGetHashMetadata(size_t hash_count, BufferReader* reader) {
+bool ParseGetHashMetadata(size_t hash_count,
+                          BufferReader* reader,
+                          std::vector<SBFullHashResult>* full_hashes) {
   for (size_t i = 0; i < hash_count; ++i) {
     base::StringPiece line;
     if (!reader->GetLine(&line))
       return false;
 
     size_t meta_data_len;
     if (!base::StringToSizeT(line, &meta_data_len))
       return false;
 
     const void* meta_data;
     if (!reader->RefData(&meta_data, meta_data_len))
       return false;
+
+    if (full_hashes) {
+      (*full_hashes)[full_hashes->size() - hash_count + i].metadata.assign(
+          reinterpret_cast<const char*>(meta_data), meta_data_len);
+    }
   }
   return true;
 }
 
 }  // namespace
 
 namespace safe_browsing {
 
 // BODY          = CACHELIFETIME LF HASHENTRY* EOF
 // CACHELIFETIME = DIGIT+
@@ skipping 59 matching lines @@
     size_t hash_count;
     if (!base::StringToSizeT(hash_count_string, &hash_count))
       return false;
 
     if (hash_len * hash_count > reader.length())
       return false;
 
     // Ignore hash results from lists we don't recognize.
     if (full_hash.list_id < 0) {
       reader.Advance(hash_len * hash_count);
-      if (has_metadata && !ParseGetHashMetadata(hash_count, &reader))
+      if (has_metadata && !ParseGetHashMetadata(hash_count, &reader, NULL))
         return false;
       continue;
     }
 
     for (size_t i = 0; i < hash_count; ++i) {
       if (!reader.GetData(&full_hash.hash, hash_len))
         return false;
       full_hashes->push_back(full_hash);
     }
 
-    // Discard the metadata for now.
-    // TODO(mattm): handle the metadata (see crbug.com/176648).
-    if (has_metadata && !ParseGetHashMetadata(hash_count, &reader))
+    if (has_metadata && !ParseGetHashMetadata(hash_count, &reader, full_hashes))
       return false;
   }
 
   return reader.empty();
 }
 
 // BODY       = HEADER LF PREFIXES EOF
 // HEADER     = PREFIXSIZE ":" LENGTH
 // PREFIXSIZE = DIGIT+         # Size of each prefix in bytes
 // LENGTH     = DIGIT+         # Size of PREFIXES in bytes
@@ skipping 132 matching lines @@
   if (!list.adds.empty() && !list.subs.empty())
     formatted_results.append(":");
   if (!list.subs.empty())
     formatted_results.append("s:").append(list.subs);
   formatted_results.append("\n");
 
   return formatted_results;
 }
 
 }  // namespace safe_browsing