Uploaded image for project: 'Core ReactOS'
  1. Core ReactOS
  2. CORE-7113

Ldr: Actctx regression at Child process

    XMLWordPrintable

Details

    • Bug
    • Resolution: Fixed
    • Major
    • None
    • RosDlls
    • 52,446

    Description

      Hi!
      I'm tracking several bugs of apps which doesnt want to start and those apps leads to the LDR rewrite. Maybe the LDR rewrite didn't introduce the bug, but at least it has uncovered several others which are crashing a lots of apps (just try a query "ldr" or "actctx" in Jira, and you would be "surprised").

      Tests shows this regression/uncovered bugs too, so it would be a nice opportunity to understand what's happening:

      52445: Kernel32:actctx http://www.reactos.org/sites/all/modules/reactos/testman/detail.php?id=2272977&prev=2271225

      52446(LDR rewrite): Kernel32:actctx http://www.reactos.org/sites/all/modules/reactos/testman/detail.php?id=2273853&prev=2272977

      58492(Trunk): Kernel32:actctx http://www.reactos.org/sites/all/modules/reactos/testman/detail.php?id=6853168&prev=6817402

      Go to the bottom of those tests.

      In 52445 we can see that the test:
      actctx.c:1126: child process manifest1
      Has succeded.

      In 52446(the rewrite) we can see that the test:
      actctx.c:1126: child process manifest1
      Has failed

      In 58492(trunk) the situation is even worse:
      We aren't even hitting
      actctx.c:1126: child process manifest1

      (the only reason to jump the test is because:
      https://svn.reactos.org/svn/reactos/trunk/rostests/winetests/kernel32/actctx.c?revision=58492&view=markup#l1234 )

      And also it shows:
      actctx: 4 failures in child process

      So at least there are 2 regressions.
      I will try to find which revision introduced the second regression via Testman.

      Attachments

        Issue Links

          Activity

            People

              fireball fireball
              vicmarcal vicmarcal
              Votes:
              2 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: