Uploaded image for project: 'Core Server'
  1. Core Server
  2. SERVER-102845

procparser and processinfo parsers need unit tests

    • Type: Icon: Task Task
    • Resolution: Unresolved
    • Priority: Icon: Major - P3 Major - P3
    • None
    • Affects Version/s: None
    • Component/s: None
    • Server Programmability
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      Part of our "all parsers need unit tests" effort, given the multiple recent CVEs that have come out of parser bugs.

      Specifically:

      • procparser::parseGenericStats()
      • MountRecord::parseLine()
      • LinuxSysHelper
      • ProcessInfo::SystemInfo::collectSystemInfo() in processinfo_solaris.cpp

      Note that there is also some "sysctl binary output to integer value" conversion in processinfo for osx and freebsd, but I suspect the combination of simple direct conversion and non-production platforms pushes that out of scope.

            Assignee:
            Unassigned Unassigned
            Reporter:
            ronald.steinke@mongodb.com Ronald Steinke
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              None
              None
              None
              None