GitHub CodeQL reports for overly permissive file permissions

オープン

#3,267 2026年1月12日に作成コメント 2 件リアクション 0 件担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度: 4/5
見積もり時間: 3〜5日
初心者へのやさしさ: 35/100

issue の種類: バグ
明瞭さ: おおむね明確
活発さ: 停滞
技術スタック: python
領域: build-system, security

調査の方向性

gyp/pylib/gyp/common.py:561、gyp/pylib/gyp/flock_tool.py:43、gyp/pylib/gyp/mac_tool.py:250 にあるフラグ付きの呼び出しを確認してください。まず、これらの権限とプロセスの umask がどのように使用されているかを追跡し、次にその動作を CodeQL ルールと比較します。各操作が意図的なもので安全に抑制できるかどうかを文書化するか、必要な変更を特定できれば完了です。

索引モデルが issue の本文から書いたものです。

説明

CodeQL has a Python rule with ID py/overly-permissive-file which flags several chmod/open calls in the node-gyp Python code. I would like to confirm that these operations are intentionally more permissive and safe to suppress.

Flagged locations

File	Line	Code	CodeQL concern
`gyp/pylib/gyp/common.py`	561	`os.chmod(tool_path, 0o755)`	World readable
`gyp/pylib/gyp/flock_tool.py`	43	`os.open(lockfile, ..., 0o666)`	World readable/writable
`gyp/pylib/gyp/mac_tool.py`	250	`os.open(lockfile, ..., 0o666)`	World readable/writable

0o755 on common.py:561 is set because you are unsure whether all build processes are run with the same user
0o666 on lock files in flock_tool.py and mac_tool.py is intentional, with umask expected to restrict actual permissions

Could you confirm these permissions are intentional and by design?

主要言語: Python
スター: 10.7k
フォーク: 1.9k
平均マージ: 1日 4時間
マージ済み PR（30日）: 5

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
リポジトリをフォークし、ブランチを切って変更します。
issue 番号を参照したプルリクエストを送ります。

nodejs/node-gyp のほかの issue

ci: GitHub Action seanmiddleditch/gha-setup-ninja is now archived オープン

dependencies github_actions

難易度 2/5 1〜3時間初心者へのやさしさ 68/100

nodejs/node-gyp#3359 · 2026年8月14日に作成
Support authentication for private `--dist-url` Node.js distribution mirrors オープン

難易度 4/5 3〜5日初心者へのやさしさ 48/100

nodejs/node-gyp#3372 · 2026年9月18日に作成
gyp: 'Undefined variable android_ndk_path' when building native modules on Android (Termux) without NDK オープン

難易度 4/5 3〜5日初心者へのやさしさ 48/100

nodejs/node-gyp#3355 · コメント 1 件 · 2026年8月13日に作成
Allow MSYS2 GCC to be used in Windows version of node-gyp オープン

MSYS2

難易度 4/5 3〜5日初心者へのやさしさ 35/100

nodejs/node-gyp#3350 · コメント 1 件 · 2026年7月21日に作成
macOS: gyp omits -isysroot when binding.gyp doesn't set SDKROOT, causing clang to resolve SDK via a path that ignores DEVELOPER_DIR/xcode-select オープン

難易度 4/5 3〜5日初心者へのやさしさ 48/100

nodejs/node-gyp#3344 · コメント 3 件 · 2026年7月9日に作成

nodejs/node-gyp の issue をすべて見る

似ている issue

validate-structure inspecte challenge/work/.terraform et rend faux les labs qui ont un provider en cache オープン

bug

難易度 2/5 1〜3時間初心者へのやさしさ 75/100

stephrobert/dsoxlab#238 · 2026年9月24日に作成
Uniform probability densities use the upper bound as an interval width オープン

難易度 2/5 1〜3時間初心者へのやさしさ 75/100

nasa/fmdtools#40 · 2026年9月24日に作成
_common_folder() treats a single file as the common folder; single-file archives can't be installed (4.2.8) / crash (master) オープン

難易度 2/5 1〜3時間初心者へのやさしさ 75/100

sublimehq/package_control#1780 · 2026年9月24日に作成
Set 'Sample' input in 'autorun.py' to utilize 'Sample' Tab in eLog オープン

難易度 2/5 1〜3時間初心者へのやさしさ 65/100

pcdshub/mfx#309 · コメント 2 件 · 2026年9月24日に作成
niri config file addition does not respect existing optional include オープン

難易度 2/5 1〜3時間初心者へのやさしさ 70/100

nwg-piotr/nwg-displays#145 · 2026年9月24日に作成

Python の issue をもっと見る