Details

    • Type: Sub-task Sub-task
    • Status: Resolved Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Fix Version/s: 0.4.4
    • Component/s: Services
    • Labels:
      None

      Description

      It is required that the environment block passed to the services processes, be they started using the LocalSystem account or using another account (e.g. LocalService, NetworkService, ...), contains the user-related environment variables: ALLUSERSPROFILE and USERPROFILE (note that USERNAME is optional).

        Issue Links

          Activity

          HBelusca
          made changes -
          Field Original Value New Value
          Fix Version/s 0.4.4 [ 10600 ]
          HBelusca
          made changes -
          Component/s Services [ 10211 ]
          HBelusca
          made changes -
          Status Untriaged [ 10000 ] Open [ 1 ]
          Module services
          HBelusca
          made changes -
          Link This issue blocks CORE-12279 [ CORE-12279 ]
          HBelusca
          made changes -
          Status Open [ 1 ] In Progress [ 3 ]
          HBelusca committed 73430 (3 files)
          Reviews: none

          [ADVAPI32_APITEST]: Add a small helper lib 'svchlp' for common routines for test-services embedded in api tests. Taken & heavily adapted from Jacek Caban's services_service test and Thomas Faber's ServiceArgs.c (the latter will be able to use 'svchlp' when it'll be ready).
          CORE-12414

          HBelusca
          made changes -
          Description It is required that the environment block passed to the services processes, be they started using the LocalSystem account or using another account (e.g. LocalService, NetworkService, ...), contains the user-related environment variables: {{ALLUSERSPROFILE}}, {{USERPROFILE}} and {{USERNAME}}.
          HBelusca
          made changes -
          Description It is required that the environment block passed to the services processes, be they started using the LocalSystem account or using another account (e.g. LocalService, NetworkService, ...), contains the user-related environment variables: {{ALLUSERSPROFILE}}, {{USERPROFILE}} and {{USERNAME}}. It is required that the environment block passed to the services processes, be they started using the LocalSystem account or using another account (e.g. LocalService, NetworkService, ...), contains the user-related environment variables: {{ALLUSERSPROFILE}} and {{USERPROFILE}} (note that {{USERNAME}} is optional).
          HBelusca committed 73431 (3 files)
          Reviews: none

          [ADVAPI32_APITEST]: Test for a (correctly initialized) service process environment block, that should contain both ALLUSERSPROFILE and USERPROFILE environment variables.
          CORE-12414

          Thomas Faber
          made changes -
          Link This issue relates to CORE-10381 [ CORE-10381 ]
          Eric Kohl committed 73433 (2 files)
          Reviews: none

          [SERVICES]
          Create a new environment block when a service process is started.
          Patch by Hermès BÉLUSCA - MAÏTO.
          CORE-12414

          Hide
          HBelusca
          added a comment - - edited

          Tests committed in r73430 and r73431; fix committed from a patch from me, by Eric Kohl in r73433.

          Testbot results: http://www.reactos.org/testman/compare.php?ids=50603,50607 and http://www.reactos.org/testman/diff.php?id1=30859865&id2=30861801&type=1&strip=0

          Show
          HBelusca
          added a comment - - edited Tests committed in r73430 and r73431; fix committed from a patch from me, by Eric Kohl in r73433. Testbot results: http://www.reactos.org/testman/compare.php?ids=50603,50607 and http://www.reactos.org/testman/diff.php?id1=30859865&id2=30861801&type=1&strip=0
          HBelusca
          made changes -
          Status In Progress [ 3 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          HBelusca committed 73435 (1 file)
          Reviews: none

          [SERVICES]: Addendum to r73433 for ScmStartUserModeService:
          - Use the correct capitalization for the desktop name (to be consistent with all the rest of our code);
          - Fix DPRINT1 order of arguments;
          - Reorganize a bit the code to avoid the memory leak if ImpersonateLoggedOnUser fails and the environment block was already allocated (caught by Ged Murphy).
          CORE-12414

            People

            • Assignee:
              HBelusca
              Reporter:
              HBelusca
            • Votes:
              1 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: