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

heap_profiler.cpp has a few bugs

    • Service Arch
    • ALL
    • 2

      In file heap_profiler.cpp

      • member functions like size(), hash(), operator==, need const.
      • Consider replacing private HashTable implementation with standard one.
      • L479 bounds on j are infinite if numFrames < skipEndFrames. (This was fixed as a side-effect of SERVER-45357)
      • L486 first paren isn't necessarily start of function params.
        for example, "_Z1gIEPFviEv" demangles to "void (*g<>())(int)", a function returning a function pointer. Maybe just don't demangle.
      • L502 StringData receives a dangling reference.
        (This was fixed as a side-effect of SERVER-45357)
         
      • sortByActiveBytes comparator is not a stable order as the elements can shift position during the sort. Fix this comparator (SERVER-41787). 

       

       

       

       

            Assignee:
            backlog-server-servicearch [DO NOT USE] Backlog - Service Architecture
            Reporter:
            billy.donahue@mongodb.com Billy Donahue
            Votes:
            0 Vote for this issue
            Watchers:
            9 Start watching this issue

              Created:
              Updated: