Age | Commit message (Collapse) | Author | Files | Lines |
|
Pester tests to the function 'Test-ServiceDaclPermission' were added in
order to increase confidence in its reliability. In general, my
intention was to replace the current functionality of the service
management functions such as Invoke-ServiceStart, to not use blindly
'sc.exe start' but rather consult with the DACL permissions and base the
decision on that. Unforunately, further investigation lead me to the
conclusion that retrieval of the service's DACL permissions requires
that an additional DACL permission (RC) be set. This may lead to an edge
case that could miss a potential privilege escalation condition and
thereby the original idea was discarded. Nonetheless,
'Test-ServiceDaclPermission' can be used for less critical tasks.
Therefore, a 'CanRestart' property was added to the output of the
service enumeration functions such as 'Get-ServiceUnquoted' as I think
that it will add value to redteamers/pentesters by helping them
prioritise which service should be abused for escalation of privileges.
Services that can be restarted by a low privileged user will probably be
prioritised first. Additionally, manual checking whether the vulnerable
service can be restarted would not be required in most cases.
|
|
Merge 3.0 release changes
|
|
Also cleaned up some module manifest cruft.
|
|
|
|
|
|
|
|
This reverts commit 98ebc1b0b8b64d069d34d80c128aa226b5e8416f.
|
|
Affected test harness PEs were updated to work in XP. Addresses issue
#100
|
|
A module should maintain a version number not the individual scripts.
|
|
Scripts in a module should not be individually versioned. Only the
module should be versioned.
|
|
|
|
|
|
In some cases, the loaded module would show up as loaded after the check
occurred.
|
|
This fix addresses issue #99. While I'm not the biggest fan of
performing tests based on environment variables, I felt that making
additional Win32 API calls would have been overkill.
|
|
|
|
Addresses issue #80. This was a tricky fix because the script should
ideally handle Unicode and Ascii encoded scripts. I haven't tested
scripts with international characters but I would imagine those script
would get mangled since the decrypted output is ultimately normalized to
ascii.
|
|
Easy fix to issue #97
|
|
Update Invoke-TokenManipulation.ps1 to address Win 10 incompatibility
|
|
This bug fix addresses issue #96. As much as a hate dropping files to
disk, this was the easiest way to preserve objects in PSv2+. If someone
want to implement the [de]serialization themselves and keep everything
in memory, please submit a PR.
|
|
Addresses issue #95.
|
|
The test dll I now use is advpack.dll since that is present in all
versions of windows.
|
|
Updated embedded powerkatz.dll to the latest version of mimikatz -
[Commit
1b13057](https://github.com/gentilkiwi/mimikatz/commit/1b130574ed78d1d9bf6117b0839056900cb8f816)
This update addresses issue #94.
|
|
|
|
|
|
|
|
Removed the "EndAddress" parameter set since it was never used. This
should resolve any parameter set confusion.
|
|
|
|
|
|
for meta functions, in order to prevent failure when running on a non-domain joined machine
took out FQDN Pester tests from Recon.tests.ps1 that used $env:userdnsdomain
|
|
|
|
|
|
Fixes issue #70
|
|
Incorporates idea from @obscuresec in issue #50.
|
|
|
|
Added ConvertFrom-UACValue to convert binary UAC values to human readable format.
Corrected logic in Set-ADObject.
|
|
|
|
|
|
Updated README.md's
|
|
|
|
|
|
|
|
|
|
This reverts commit a0ab599810f8f05a9bf24850fb9104516b71abb7.
|
|
|
|
|
|
Removed extraneous parameters
Removed the following extraneous parameters:
-PEPath
-PEUrl
The functionality they provided can be easily replicated in code outside
of Invoke-ReflectivePEInjection. i.e. it should be up to the user how
they might want to download a PE before loading it. That should not be
dictated by Invoke-ReflectivePEInjection.
|
|
This reverts commit 0eb520e31f97bc0ca83bd2c1546a18dd97e09d79.
|
|
Removed the following extraneous parameters:
-PEPath
-PEUrl
-ComputerName
The functionality they provided can be easily replicated in code outside
of Invoke-ReflectivePEInjection. i.e. it should be up to the user how
they might want to download a PE before loading it. That should not be
dictated by Invoke-ReflectivePEInjection.
|
|
|
|
This was only ever intended to be a PoC. I'll bring this back if
requested but it exhibits duplicate functionality.
|