Support for recording registered LSPs

chrome/common/safe_browsing/csd.proto

 // 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.
 //
 // Client side phishing and malware detection request and response
 // protocol buffers.  Those protocol messages should be kept in sync
 // with the server implementation.
 //
 // If you want to change this protocol definition or you have questions
 // regarding its format please contact chrome-anti-phishing@googlegroups.com.
@@ skipping 92 matching lines @@
   // where the referrer uses HTTPS.
   optional string referrer_url = 4;
 
   // Field 5 and 6 are only used on the server.
 
   message UrlInfo {
     required string ip = 1;
     required string url = 2;
     optional string method = 3;
     optional string referrer = 4;
-    // Resource type, the int value is a direct cast from the Type enum 
+    // Resource type, the int value is a direct cast from the Type enum
     // of ResourceType class defined in //src/webkit/commom/resource_type.h
     optional int32 resource_type = 5;
   }
 
   // List of resource urls that match the malware IP list.
   repeated UrlInfo bad_ip_url_info = 7;
 }
 
 message ClientMalwareResponse {
   required bool blacklist = 1;
@@ skipping 249 matching lines @@
     }
     optional OS os = 1;
     message Machine {
       optional string cpu_architecture = 1;
       optional string cpu_vendor = 2;
       optional uint32 cpuid = 3;
     }
     optional Machine machine = 2;
     message Process {
       optional string version = 1;
-      repeated string dlls = 2;
+      repeated string OBSOLETE_dlls = 2;
       message Patch {
         optional string function = 1;
         optional string target_dll = 2;
       }
       repeated Patch patches = 3;
       message NetworkProvider {}
       repeated NetworkProvider network_providers = 4;
       enum Channel {
         CHANNEL_UNKNOWN = 0;
         CHANNEL_CANARY = 1;
         CHANNEL_DEV = 2;
         CHANNEL_BETA = 3;
         CHANNEL_STABLE = 4;
       }
       optional Channel chrome_update_channel = 5;
       optional int64 uptime_msec = 6;
+      message Dll {
+        enum Feature {
+          UNKNOWN = 0;
+          LSP = 1;
+        }
+        optional string path = 1;
+        optional uint32 base_address = 2;

[mattm, 2014/06/17 01:28:18]

what about 64bit chrome?

[grt (UTC plus 2), 2014/06/17 10:22:05]

AddressOfEntryPoint (base_address) and SizeOfCode (length) are both DWORDs in
the PE image headers for 32-bit and 64-bit images (see
http://msdn.microsoft.com/en-us/gg463119.aspx).

[grt (UTC plus 2), 2014/06/17 13:10:06]

Ah, but now I see that the base_address isn't AddressOfEntryPoint, but is the
location in memory. This should be a uint64. Apologies for the misdirection,
Patrick.

[pmonette_google.com, 2014/06/17 17:48:22]

Done.

+        optional uint32 length = 3;
+        repeated Feature feature = 4;
+      }
+      repeated Dll dll = 9;

[grt (UTC plus 2), 2014/06/16 20:30:38]

note to matt: i've asked patrick to use 9 here since i'm adding 7 and 8 in
another CL elsewhere.

     }
     optional Process process = 3;
   }
 
   optional EnvironmentData environment = 3;
 }
 
 message ClientIncidentResponse {
   optional bytes token = 1;
   optional bool download_requested = 2;
 
   message EnvironmentRequest { optional int32 dll_index = 1; }
 
   repeated EnvironmentRequest environment_requests = 3;
 }