Stack sampling profiler: fix module read validation failures

components/metrics/public/cpp/call_stack_profile_struct_traits.h

 // Copyright 2016 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.
 
 // Defines StructTraits specializations for translating between mojo types and
 // base::StackSamplingProfiler types, with data validity checks.
 
 #ifndef COMPONENTS_METRICS_CALL_STACK_PROFILE_STRUCT_TRAITS_H_
 #define COMPONENTS_METRICS_CALL_STACK_PROFILE_STRUCT_TRAITS_H_
 
@@ skipping 16 matching lines @@
       const base::StackSamplingProfiler::Module& module) {
     return module.id;
   }
   static const base::FilePath& filename(
       const base::StackSamplingProfiler::Module& module) {
     return module.filename;
   }
 
   static bool Read(metrics::mojom::CallStackModuleDataView data,
                    base::StackSamplingProfiler::Module* out) {
-    // Linux has the longest build id at 40 bytes.
-    static const size_t kMaxIDSize = 40;
-
     std::string id;
     base::FilePath filename;
-    if (!data.ReadId(&id) || id.size() > kMaxIDSize ||
-        !data.ReadFilename(&filename))
+    if (!data.ReadId(&id) || !data.ReadFilename(&filename))
       return false;
 
     *out =
         base::StackSamplingProfiler::Module(data.base_address(), id, filename);
     return true;
   }
 };
 
 template <>
 struct StructTraits<metrics::mojom::CallStackFrameDataView,
@@ skipping 310 matching lines @@
             metrics::CallStackProfileParams::SampleOrderingSpec::PRESERVE_ORDER;
         return true;
     }
     return false;
   }
 };
 
 }  // mojo
 
 #endif  // COMPONENTS_METRICS_CALL_STACK_PROFILE_STRUCT_TRAITS_H_