Added Debugging Logs: (#4130)

Logs have been added to debug and resolve the following issues:
1. AttributeError: 'NoneType' object has no attribute 'stdin'
2. Line is too long for a stacktrace

src/clusterfuzz/_internal/crash_analysis/stack_parsing/stack_symbolizer.py

@@ -244,6 +244,7 @@ def __init__(self, symbolizer_path, default_arch, system, dsym_hints=[]):
 
   def open_llvm_symbolizer(self):
     if not os.path.exists(self.symbolizer_path):
+      logs.info(f'os.path() does not have {self.symbolizer_path}')
       return None
 
     # Setup symbolizer command line.
@@ -274,6 +275,7 @@ def open_llvm_symbolizer(self):
     global pipes
     pipes.append(pipe)
 
+    logs.info(f'created pipe - {pipe}')
     return pipe
 
   def symbolize(self, addr, binary, offset):

src/clusterfuzz/_internal/system/environment.py

@@ -24,6 +24,7 @@
 import yaml
 
 from clusterfuzz._internal import fuzzing
+from clusterfuzz._internal.metrics import logs
 
 # Tools supporting customization of options via ADDITIONAL_{TOOL_NAME}_OPTIONS.
 # FIXME: Support ADDITIONAL_UBSAN_OPTIONS and ADDITIONAL_LSAN_OPTIONS in an
@@ -355,6 +356,7 @@ def get_llvm_symbolizer_path():
         stderr=subprocess.DEVNULL)
     if return_code == 0:
       # llvm-symbolize works, return it.
+      logs.info(f'Environment Symbolizer Path :- {llvm_symbolizer_path}')
       return llvm_symbolizer_path
 
   # Either
@@ -365,10 +367,12 @@ def get_llvm_symbolizer_path():
 
   # Make sure that we have a default llvm-symbolizer for this platform.
   if not os.path.exists(llvm_symbolizer_path):
+    logs.info(f'None Symbolizer Path :- {llvm_symbolizer_path}')
     return None
 
   # Make sure that llvm symbolizer binary is executable.
   os.chmod(llvm_symbolizer_path, 0o750)
+  logs.info(f'Default Symbolizer Path :- {llvm_symbolizer_path}')
   return llvm_symbolizer_path
 
 

src/clusterfuzz/stacktraces/__init__.py

@@ -1407,7 +1407,7 @@ def should_ignore_line_for_crash_processing(line, state):
     return True
 
   if len(line) > 1024**2:
-    logs.error('Line is too long for a stacktrace.')
+    logs.error('Line is too long for a stacktrace', engine_output=line)
     return True
 
   return False