--cidfile fails if file exists even if empty

オープン

#5,954 2025年3月22日に作成コメント 4 件リアクション 0 件担当者 0 名 GitHub で見る

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

評価

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

issue の種類: バグ
明瞭さ: おおむね明確
活発さ: 活発
技術スタック: docker, go
領域: cli

調査の方向性

docker run --cidfile の処理から始め、mktemp で作成したパスを使って報告されたケースを再現してください。既存の空の cidfile が失敗を引き起こさなくなり、PID を含む cidfile では意図された失敗動作が維持されれば完了です。このエントリポイントの CLI テストが配置されている箇所で、これらのケースのカバレッジを追加または更新してください。

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

説明

kind/enhancement status/0-triage

Description

The best practice when creating a temporary file is not only to generate a filename of a file that does not already exist, but also to create/open the file with O_EXCL, to avoid TOCTOU, typically that's what mkstemp(3) does.
Unfortunately, docker-run's --cidfile prevents from passing it a safe temporary file, because docker-run will fail if the given file merely exists.

Reproduce

t=$(mktemp)
docker run --rm --cidfile=$t debian

Expected behavior

docker-run should fail only if the file given as --cidfile contains a PID (even better, check if the PID is alive by using kill(the_pid, 0) which is designed for that)

docker version

Client:
 Version:           20.10.24+dfsg1
 API version:       1.41
 Go version:        go1.19.8
 Git commit:        297e128
 Built:             Sat Oct 12 15:19:49 2024
 OS/Arch:           linux/amd64
 Context:           default
 Experimental:      true

Server:
 Engine:
  Version:          20.10.5+dfsg1
  API version:      1.41 (minimum version 1.12)
  Go version:       go1.15.15
  Git commit:       363e9a8
  Built:            Mon May 30 18:34:49 2022
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          1.6.20~ds1
  GitCommit:        1.6.20~ds1-1+deb12u1
 runc:
  Version:          1.1.5+ds1
  GitCommit:        1.1.5+ds1-1+deb12u1
 docker-init:
  Version:          0.19.0
  GitCommit:

docker info

too much identifying info

Additional Info

No response

主要言語: Go
スター: 6.1k
フォーク: 2.2k
平均マージ: 1日 10時間
マージ済み PR（30日）: 47

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

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

はじめの一歩

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

docker/cli のほかの issue

`docker logout docker.io` does not remove Docker Hub credentials オープン

kind/bug status/0-triage

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

docker/cli#7326 · 2026年9月23日に作成
Truncating log lines at 75 characters makes them useless. オープン

kind/bug status/0-triage

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

docker/cli#7176 · コメント 1 件 · 2026年8月11日に作成
cli-plugins/hooks: max-message enforcement is off by one オープン

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

docker/cli#7005 · コメント 2 件 · 2026年5月25日に作成
Path confinement for `secrets.file` and `configs.file` in stack deploy オープン

kind/feature status/0-triage

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

docker/cli#6919 · コメント 3 件 · 2026年4月7日に作成
`docker context import` TLS Entry Handling results in OOM オープン

kind/bug status/0-triage

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

docker/cli#6917 · コメント 2 件 · 2026年4月7日に作成

docker/cli の issue をすべて見る

似ている issue

Smoke Test: Copilot - 35930762956 オープン

難易度 1/5 1時間未満初心者へのやさしさ 60/100

github/gh-aw-mcpg#13748 · 2026年9月23日に作成
[aw] Daily Reliability Review failed オープン

agentic-workflows

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

github/gh-aw#63046 · 2026年9月23日に作成
OpenVEX statements carry no subcomponent package or version オープン

needs-triage

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

NVIDIA/aicr#2945 · 2026年9月23日に作成
[Bug]: Event executions make the originating request panic (nil JobInsertResult in queue log middleware) when the OTLP log exporter is enabled オープン

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

zitadel/zitadel#12811 · 2026年9月23日に作成
java: Librarian prepends duplicate java- prefix when library name already includes it オープン

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

googleapis/librarian#7670 · コメント 2 件 · 2026年9月23日に作成

Go の issue をもっと見る