summaryrefslogtreecommitdiffstats
path: root/sandbox/win/src/process_thread_interception.cc
Commit message (Collapse)AuthorAgeFilesLines
* Intercept CreateThread inside renderer and proxy out to CreateRemoteThread ↵liamjm2016-02-091-1/+78
| | | | | | | | | | | | | | | | | | 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}
* Make TargetCreateProcessA consistent with TargetCreateProcessW.wfh2016-01-251-1/+2
| | | | | | | | | | 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}
* Switch to standard integer types in sandbox/.avi2015-12-221-6/+8
| | | | | | | | | | | | 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}
* Revert of Switch to standard integer types in sandbox/. (patchset #3 ↵petewil2015-12-211-8/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | 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}
* Switch to standard integer types in sandbox/.avi2015-12-211-6/+8
| | | | | | | | | BUG=138542 TBR=cpu@chromium.org Review URL: https://codereview.chromium.org/1538283002 Cr-Commit-Position: refs/heads/master@{#366413}
* Fixes to possible GetLastError bugsbrucedawson2015-09-171-0/+4
| | | | | | | | | | | | | | | | | 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}
* close ALPC ports in sbox integrations tests. This is the first step in ↵liamjm2015-07-161-1/+2
| | | | | | | | | | | | | | | | | | | 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}
* Remove HandleCloser::SetupHandleInterceptionsjschuh@chromium.org2014-06-101-46/+0
| | | | | | | | | | 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
* This patch addresses two issues:zturner@chromium.org2013-10-211-3/+5
| | | | | | | | | | | | | | | | | | 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
* Move the Windows sandbox to sandbox/winjln@chromium.org2012-07-181-0/+447
| | | | | | | | | | | | | 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
* Emergency revert; rietveld broke; tree brokeerg@google.com2012-07-131-447/+0
| | | | git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146646 0039d316-1c4b-4281-b951-d872f2087c98
* sandbox/src -> sandbox/win/srcjln@chromium.org2012-07-131-10/+10
| | | | git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146626 0039d316-1c4b-4281-b951-d872f2087c98
* Move Windows sandboxjln@chromium.org2012-07-131-0/+447
- 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
generated by cgit v1.1 at 2024-09-06 22:01:59 +0000