Fix stack collection with size limit

src/client/linux/minidump_writer/minidump_writer.cc

Index: src/client/linux/minidump_writer/minidump_writer.cc
diff --git a/src/client/linux/minidump_writer/minidump_writer.cc b/src/client/linux/minidump_writer/minidump_writer.cc
index f407caa72c3b4ba8517dc9e85f7d06f7e1da8e67..abd74508d7cda6a2a73d75e6caa25b77b343d880 100644
--- a/src/client/linux/minidump_writer/minidump_writer.cc
+++ b/src/client/linux/minidump_writer/minidump_writer.cc
@@ -275,6 +275,12 @@ class MinidumpWriter {
       if (max_stack_len >= 0 &&
           stack_len > static_cast<unsigned int>(max_stack_len)) {
         stack_len = max_stack_len;
+        // Skip empty chunks of length max_stack_len.
+        uintptr_t int_stack = reinterpret_cast<uintptr_t>(stack);
+        while (int_stack +  max_stack_len < stack_pointer) {

[ivanpe, 2016/05/17 01:03:10]

Please, remove the extra blank after '+'

[lv, 2016/05/23 15:54:57]

Done.

+          int_stack += max_stack_len;

[ivanpe, 2016/05/17 01:03:10]

This will be an infinite loop when max_stack_len = 0.

[lv, 2016/05/23 15:54:57]

I added an additional check. I don't think anyone uses a value of 0 here, so I
could also re-purpose 0 instead of -1 to indicate no cap, which would simplify
the code a bit. Let me know which way you prefer.

+        }
+        stack = reinterpret_cast<const void*>(int_stack);
       }
       if (!memory.Allocate(stack_len))
         return false;