golang/go

cmd/link: suspicious use of (*cmd/internal/bio/buf.Reader).Seek

Open

#28,556 建立於 2018年11月2日

在 GitHub 查看
 (3 留言) (0 反應) (0 負責人)Go (19,008 fork)batch import
NeedsInvestigationcompiler/runtimehelp wanted

倉庫指標

Star
 (133,883 star)
PR 合併指標
 (30 天內沒有已合併 PR)

描述

While a running a recent test of vet, it flagged the (*buf.Reader).Seek method as not having a proper return type (int64, error). Instead it handles errors by calling log.Fatal, yet most callers of this function seem to assume it returns a negative number to indicate failure.

Someone who knows this code should probably audit it.

貢獻者指南

研究方向
該問題指向 cmd/link 套件中 (*cmd/internal/bio/buf.Reader).Seek 的可疑用法。該方法定義在 cmd/internal/bio 中，透過呼叫 log.Fatal 來處理錯誤，但許多呼叫者似乎假設它在失敗時回傳負數。需要進行稽核，以確定呼叫者是否正確處理錯誤，或者是否應變更方法的實作。相關檔案包括 cmd/internal/bio/buf.go（用於 Seek 方法）以及呼叫 Seek 的 cmd/link/internal/ld/ 和其他連結器套件。該問題沒有關聯的 PR 或分配者，因此研究應從檢查呼叫點並理解預期的錯誤處理開始。
技術棧
go
領域
backend
議題類型
錯誤
難度面向新貢獻者的預計實作難度，1 表示很小改動，5 表示專家級工作。
3
預計時間有經驗貢獻者完成調查、實作、測試並準備 pull request 的粗略時間範圍。
半天
活動狀態議題目前的可參與程度：新鮮、活躍、陳舊、阻塞或等待維護者輸入。
活躍
清晰度議題是否清楚說明預期改動、驗收標準和下一步。
清晰
前置要求
GoGit
新手友善度1-100 的估計分數，表示該議題對首次貢獻者的友善程度。
65