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

assert.soon() with hang analyzer enabled inside a try/finally can lead to an invariant failure in ProgramRegistry

    XMLWordPrintable

Details

    • Fully Compatible
    • ALL
    • v4.4
    • Hide

      The sleep(5000) is to ensure the parallel shell has already exited by the time MongoRunner.runningChildPids() is called.

      ./mongo --nodb --eval 'var cleanup = startParallelShell("1", undefined, true); sleep(5000); try { MongoRunner.runningChildPids(); throw new Error('Simulating assert.soon() failure'); } finally { cleanup() }'

      Show
      The sleep(5000) is to ensure the parallel shell has already exited by the time MongoRunner.runningChildPids() is called. ./mongo --nodb --eval 'var cleanup = startParallelShell("1", undefined, true); sleep(5000); try { MongoRunner.runningChildPids(); throw new Error('Simulating assert.soon() failure'); } finally { cleanup() }'
    • STM 2020-06-29, STM 2020-07-13
    • 50

    Description

      The MongoRunner.runningChildPids() function calls through to getRunningMongoChildProcessIds() in shell_utils_launcher.cpp, which in turn calls wait_for_pid() on all processes in ProgramRegistry::_registeredPids. A JavaScript test of the form

      let cleanup;
      		 
      try {
      		 
          cleanup = startParallelShell(...);
      		 
          assert.soon(...);
      		 
          ...
      		 
      } finally {
      		 
          cleanup();
      		 
      }

      (for example) would end up calling wait_for_pid() multiple times. The first call to wait_for_pid() via getRunningMongoChildProcessIds() would unregister the pid from ProgramRegistry::_registeredPids and the second call would trigger this invariant.

      Attachments

        Issue Links

          is caused by

          Improvement - An improvement or enhancement to an existing feature or task. SERVER-43153 Expose pids of spawned processes in the shell

          • Major - P3 - Major loss of function.
          • Closed
          related to

          Bug - A problem which impairs or prevents the functions of the product. SERVER-49347 Fix locking in shell's ProgramRegistry

          • Major - P3 - Major loss of function.
          • Closed

          Activity

            People

              richard.samuels@mongodb.com Richard Samuels
              max.hirschhorn@mongodb.com Max Hirschhorn
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: