Skip to content

Commit

Permalink
Fix permissions for everything inside the installation folder (#342)
Browse files Browse the repository at this point in the history
* add ci to iacls

* make sure files get affected too

Co-authored-by: Joao Moreno <joao.moreno@microsoft.com>
  • Loading branch information
sbatten and joaomoreno committed Apr 11, 2022
1 parent 82affae commit dfd34e8
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions build/win32/code.iss
Original file line number Diff line number Diff line change
Expand Up @@ -1491,10 +1491,10 @@ var
ResultCode: Integer;
Permissions: string;
begin
Permissions := '/grant:r "*S-1-5-18:F" /grant:r "*S-1-5-32-544:F" /grant:r "*S-1-5-11:RX" /grant:r "*S-1-5-32-545:RX"';
Permissions := '/grant:r "*S-1-5-18:(OI)(CI)F" /grant:r "*S-1-5-32-544:(OI)(CI)F" /grant:r "*S-1-5-11:(OI)(CI)RX" /grant:r "*S-1-5-32-545:(OI)(CI)RX"';
#if "user" == InstallTarget
Permissions := Permissions + ' /grant:r "*S-1-3-0:F"';
Permissions := Permissions + ' /grant:r "*S-1-3-0:(OI)(CI)F"';
#endif
Exec(ExpandConstant('{sys}\icacls.exe'), ExpandConstant('"{app}" /inheritancelevel:r ') + Permissions, '', SW_HIDE, ewWaitUntilTerminated, ResultCode);
Expand Down

1 comment on commit dfd34e8

@kingfrost987
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good

Please sign in to comment.