From 360ee295a8c5ac1f693e38160302c1ea93bb9964 Mon Sep 17 00:00:00 2001
From: Karen Arutyunov <karen@codesynthesis.com>
Date: Sat, 24 Jun 2017 20:46:46 +0300
Subject: Only examine first 512 log line bytes for warnings detection

Fixes
---
 bbot/worker.cxx | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

(limited to 'bbot')

diff --git a/bbot/worker.cxx b/bbot/worker.cxx
index a0e2ad7..81d9a50 100644
--- a/bbot/worker.cxx
+++ b/bbot/worker.cxx
@@ -116,7 +116,13 @@ run_bpkg (tracer& t,
         {
           for (const auto& re: warn_detect)
           {
-            if (regex_search (l, re))
+            // Only examine the first 512 bytes. Long lines (e.g., linker
+            // command lines) could trigger implementation-specific limitations
+            // (like stack overflow). Plus, it is a performance concern.
+            //
+            if (regex_search (l.begin (),
+                              l.size () < 512 ? l.end () : l.begin () + 512,
+                              re))
             {
               r = result_status::warning;
               break;
-- 
cgit v1.1