Add check to EnsureValidConfiguration to filter out some corner cases

Author: Jlalond

lldb/include/lldb/API/SBSaveCoreOptions.h

@@ -120,11 +120,11 @@ class LLDB_API SBSaveCoreOptions {
   SBThreadCollection GetThreadsToSave() const;
 
   /// Get the current total number of bytes the core is expected to be but not
-  /// including the overhead of the core file format. Requires a Process and 
+  /// including the overhead of the core file format. Requires a Process and
   /// Style to be specified.
-  /// 
+  ///
   /// \note
-  ///   This can cause some modification of the underlying data store 
+  ///   This can cause some modification of the underlying data store
   ///   as regions with no permissions, or invalid permissions will be removed
   ///   and stacks will be minified up to their stack pointer + the redzone.
   ///

lldb/source/Symbol/SaveCoreOptions.cpp

@@ -151,13 +151,17 @@ uint64_t SaveCoreOptions::GetCurrentSizeInBytes(Status &error) {
     return 0;
   }
 
+  error = EnsureValidConfiguration(m_process_sp);
+  if (error.Fail())
+    return 0;
+
   CoreFileMemoryRanges ranges;
   error = m_process_sp->CalculateCoreFileSaveRanges(*this, ranges);
   if (error.Fail())
     return 0;
 
   uint64_t total_in_bytes = 0;
-  for (auto& core_range : ranges)
+  for (auto &core_range : ranges)
     total_in_bytes += core_range.data.range.size();
 
   return total_in_bytes;