[Bug]: CDI spec WriteTo leaks the temporary descriptor on save errors
メンテナーはふだん 6 日以内に返信
評価
調査の方向性
Start at spec.Interface.WriteTo in the pkg/nvcdi/spec package and trace the temporary-file cleanup and close paths when saving fails. The issue gives a reproducer using an invalid CDI version and io.Discard; use it to confirm that failed writes close the descriptor and remove the temporary file. An open linked pull request (#2140) is already addressing this issue.
索引モデルが issue の本文から書いたものです。
説明
Describe the bug
spec.Interface.WriteTo leaves the temporary file descriptor open when saving the CDI specification fails. The temporary pathname is removed, but the descriptor is only reclaimed by Go's file finalizer.
On current main (023f35906ee23384b48dc3db3f2d1fb75d29f6b0), eight failed writes leave eight open, deleted nvcdi-spec-*.yaml descriptors when garbage collection is disabled. This affects callers that reuse the public library after a validation or filesystem error.
To Reproduce
Run the following from a module that imports the toolkit:
package main
import (
"io"
"runtime/debug"
"github.com/NVIDIA/nvidia-container-toolkit/pkg/nvcdi/spec"
)
func main() {
debug.SetGCPercent(-1) // Make descriptor cleanup independent of finalizers.
s, err := spec.New(spec.WithVersion("invalid"))
if err != nil {
panic(err)
}
for i := 0; i < 8; i++ {
_, err := s.WriteTo(io.Discard)
if err == nil {
panic("expected CDI validation failure")
}
}
select {} // Inspect /proc/<pid>/fd in another terminal.
}
Each failed write reports failed to write spec: invalid CDI Spec: invalid version "invalid". /proc/<pid>/fd contains eight temporary CDI files with the (deleted) suffix. No temporary pathname remains on disk.
Expected behavior
Every return from WriteTo should close the temporary file descriptor, including failures before the existing explicit tmpFile.Close() call. The original save error and temporary-file cleanup should be preserved.
Environment
- Toolkit: current main at
023f35906ee23384b48dc3db3f2d1fb75d29f6b0 - Linux amd64, Go 1.26.0
- Reproduction uses the public Go API; GPU, driver, and container-runtime operations are not involved.
Additional information
The error is reproducible without GPU hardware. I plan to add a focused regression that separately checks open descriptors and remaining temporary files, and ensure the temporary descriptor is closed on the save-error path.
- 主要言語
- Go
- スター
- 4.6k
- フォーク
- 601
- 平均マージ
- 7日 14時間
- マージ済み PR（30日）
- 10
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
NVIDIA/nvidia-container-toolkit のほかの issue
-
Address TODO comments in internal/platform-support/dgpu/dgpu.go対応中かも @truongbo17 が 17 日前に担当しました。 オープンdsx-ws-0930 good-first-issue
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
NVIDIA/nvidia-container-toolkit#2114 · コメント 1 件 ·
メンテナーはふだん 6 日以内に返信
-
難易度 3/5 1〜2日 初心者へのやさしさ 78/100
NVIDIA/nvidia-container-toolkit#2147 · コメント 1 件 ·
メンテナーはふだん 6 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 47/100
NVIDIA/nvidia-container-toolkit#2137 ·
メンテナーはふだん 6 日以内に返信
-
[Bug]: containerd cleanup removes imports for unrelated drop-ins対応中かも @git-jxj が 8 日前に担当しました。 オープン
難易度 3/5 1〜2日 初心者へのやさしさ 68/100
NVIDIA/nvidia-container-toolkit#2134 ·
メンテナーはふだん 6 日以内に返信
-
[Bug]: create-symlinks and update-ldcache hooks fail container create when the container rootfs is read-only対応中かも @sahanasreenath が 10 日前に担当しました。 オープン
難易度 3/5 半日 初心者へのやさしさ 25/100
NVIDIA/nvidia-container-toolkit#2131 · コメント 2 件 ·
メンテナーはふだん 6 日以内に返信
NVIDIA/nvidia-container-toolkit の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
scaleway/scaleway-cli#6492 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 67/100
taskcluster/taskcluster#9284 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
oasdiff/oasdiff#1337 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
splunk/terraform-provider-splunk#256 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信