| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
in browser.
summary:
* Create new interception
* Create new IPC (IPC_CREATETHREAD_TAG)
* new tests in process_policy_test.cc
* Interception is installed, but passes call through by default. Proxy is only done for tests, when ALPC ports are closed.
Note: the IPC uses VOIDPTR_TYPE for a size_t param. These are not guaranteed to be the same (http://stackoverflow.com/questions/1464174/size-t-vs-intptr-t). Should we create a size_t IPC type?
BUG=464430
Review URL: https://codereview.chromium.org/1225183003
Cr-Commit-Position: refs/heads/master@{#374510}
|
|
|
|
|
|
|
|
|
|
| |
Add a test that would previously crash before this CL.
BUG=580916
Review URL: https://codereview.chromium.org/1621413003
Cr-Commit-Position: refs/heads/master@{#371293}
|
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=138542
TBR=cpu@chromium.org
Committed: https://crrev.com/b5032dd2fd694a9bd7a9e96ad8fefcaf29c24c9e
Cr-Commit-Position: refs/heads/master@{#366413}
Review URL: https://codereview.chromium.org/1538283002
Cr-Commit-Position: refs/heads/master@{#366520}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
id:40001 of https://codereview.chromium.org/1538283002/ )
Reason for revert:
This appears to be causing a compile failure (macros.h omitted)
Original issue's description:
> Switch to standard integer types in sandbox/.
>
> BUG=138542
> TBR=cpu@chromium.org
>
> Committed: https://crrev.com/b5032dd2fd694a9bd7a9e96ad8fefcaf29c24c9e
> Cr-Commit-Position: refs/heads/master@{#366413}
TBR=cpu@chromium.org,avi@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=138542
Review URL: https://codereview.chromium.org/1539423002
Cr-Commit-Position: refs/heads/master@{#366428}
|
|
|
|
|
|
|
|
|
| |
BUG=138542
TBR=cpu@chromium.org
Review URL: https://codereview.chromium.org/1538283002
Cr-Commit-Position: refs/heads/master@{#366413}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
GetLastError needs to be called immediately after the function whose
error code it is retrieving. This has been made particularly important
by VS 2015 which sometimes clears the error code when allocating memory.
See bug 528394 for details of the underlying issue.
These changes came from code inspection that looked for patterns that
appeared wrong. None of the issues look critical, although that depends
on what callers do with the error codes.
BUG=529981
Review URL: https://codereview.chromium.org/1337223002
Cr-Commit-Position: refs/heads/master@{#349481}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
locking down ALPC ports.
Closing ALPC ports causes the following tests to break, because they call CreateProcess(). The interception of CreateProcess() tries to call CreateProcess natively, which crashes.
ProcessPolicyTest.TestGetProcessTokenMaxAccess
ProcessPolicyTest.TestGetProcessTokenMaxAccessNoJob
PolicyTargetTest.OpenThread
The changes in process_thread_interception.cc check to see if CSRSS is disconnected, and doesn't use the native CreateProcess if CSRSS is disconnected. This connected flag is implicitly set in the HandleCloserAgent when type "ALPC Port" are requested to be closed.
Therefore this still only affects sbox_integration_tests, but adds some of the capability that will be needed to further lock down CSRSS.
BUG=464430
Review URL: https://codereview.chromium.org/1226383005
Cr-Commit-Position: refs/heads/master@{#338963}
|
|
|
|
|
|
|
|
|
|
| |
This code isn't in use and will need to be rewritten if we want to use it in the future.
NOTRY=true
Review URL: https://codereview.chromium.org/320313002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@276179 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
1) Some functions were using the wrong signatures, which
only happened to work by coincidence since the wrong
signatures had the same number of arguments as the right
signatures.
2) Don't use imported functions from system libraries in
patched functions until after TargetServices::InitCalled()
has been called, which guarantees that our IAT has been
initialized.
BUG=305815
Review URL: https://codereview.chromium.org/29573002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@229943 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is a rather large refactor to move the Windows sandbox to the right place.
BUG=
TEST=
NOTRY=true
TBR=sky@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10689170
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@147151 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146646 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146626 0039d316-1c4b-4281-b951-d872f2087c98
|
|
- Move Windows sandbox to sandbox/win
- Update sandbox_win.gypi
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146625 0039d316-1c4b-4281-b951-d872f2087c98
|