From faadda040d49c581a82fc459c34186c8bff58a2b Mon Sep 17 00:00:00 2001 From: Roberto Clapis Date: Thu, 2 Aug 2018 15:27:14 +0200 Subject: [PATCH] os: add check for empty executable path on darwin os used to panic (access out of bounds) if executablePath was left empty Fixes #22529 Change-Id: Iead5e60a3b000dbde421a8e8612c3690340879ce Reviewed-on: https://go-review.googlesource.com/127546 Run-TryBot: Ian Lance Taylor TryBot-Result: Gobot Gobot Reviewed-by: Brad Fitzpatrick --- src/os/executable_darwin.go | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/src/os/executable_darwin.go b/src/os/executable_darwin.go index ce5b8140a4..dae9f4ee18 100644 --- a/src/os/executable_darwin.go +++ b/src/os/executable_darwin.go @@ -4,12 +4,17 @@ package os +import "errors" + var executablePath string // set by ../runtime/os_darwin.go var initCwd, initCwdErr = Getwd() func executable() (string, error) { ep := executablePath + if len(ep) == 0 { + return ep, errors.New("cannot find executable path") + } if ep[0] != '/' { if initCwdErr != nil { return ep, initCwdErr -- 2.50.0