Skip to content

Commit

Permalink
fix security issue (#91)
Browse files Browse the repository at this point in the history
  • Loading branch information
mitaki28 committed May 11, 2021
1 parent ce7bb35 commit cdbf676
Show file tree
Hide file tree
Showing 6 changed files with 2,580 additions and 17 deletions.
15 changes: 14 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,20 @@ Please update as follows:

## Command

- `Show Executable and Compile Options`: Showing Clang executable and compile options for the active editor.
- `ClangCommandAdapter: Show Executable and Compile Options`: Showing Clang executable and compile options for the active editor.
- `ClangCommandAdapter: Trust Workspace`: Trust the current workspace. (see [#Security](#Security))
- `ClangCommandAdapter: Untrust Workspace`: Untrust the current workspace. (see [#Security](#Security))

## Security

The following settings may cause security issue, if you are opening a malicious workspece.

- `clang.executable`: Arbitrary executables specified in the workspace-level setting will be executed.
- `clang.cflags`, `clang.cxxflags`, `clang.objcflags`: Clang command is executed with artibrary options specified in the workspace-level setting.

Therefore, these workspace-level settings are disabled by default.
- To enable these workspace-level settings, please trust the workspace from dialog that appears when you open the file.
- To change trust/untrust status of the current workspace, please use `ClangCommandAdapter: Trust Workspace` or `ClangCommandAdapter: Untrust Workspace`.

## Acknowledgements

Expand Down
Loading

0 comments on commit cdbf676

Please sign in to comment.