1
0
Fork 0
mirror of https://github.com/moby/moby.git synced 2022-11-09 12:21:53 -05:00

Merge pull request #39849 from tklauser/uname-unix-no-exec

Use unix.Uname instead of shelling out to uname on darwin/freebsd
This commit is contained in:
Sebastiaan van Stijn 2019-09-11 20:52:23 +02:00 committed by GitHub
commit 871994ce2a
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 13 additions and 31 deletions

View file

@ -3,19 +3,19 @@
package operatingsystem // import "github.com/docker/docker/pkg/parsers/operatingsystem" package operatingsystem // import "github.com/docker/docker/pkg/parsers/operatingsystem"
import ( import (
"bytes"
"errors" "errors"
"os/exec"
"strings" "golang.org/x/sys/unix"
) )
// GetOperatingSystem gets the name of the current operating system. // GetOperatingSystem gets the name of the current operating system.
func GetOperatingSystem() (string, error) { func GetOperatingSystem() (string, error) {
cmd := exec.Command("uname", "-s") utsname := &unix.Utsname{}
osName, err := cmd.Output() if err := unix.Uname(utsname); err != nil {
if err != nil {
return "", err return "", err
} }
return strings.TrimSpace(string(osName)), nil return string(utsname.Machine[:bytes.IndexByte(utsname.Sysname[:], 0)]), nil
} }
// GetOperatingSystemVersion gets the version of the current operating system, as a string. // GetOperatingSystemVersion gets the version of the current operating system, as a string.

View file

@ -1,18 +0,0 @@
// Package platform provides helper function to get the runtime architecture
// for different platforms.
package platform // import "github.com/docker/docker/pkg/platform"
import (
"bytes"
"golang.org/x/sys/unix"
)
// runtimeArchitecture gets the name of the current architecture (x86, x86_64, …)
func runtimeArchitecture() (string, error) {
utsname := &unix.Utsname{}
if err := unix.Uname(utsname); err != nil {
return "", err
}
return string(utsname.Machine[:bytes.IndexByte(utsname.Machine[:], 0)]), nil
}

View file

@ -1,20 +1,20 @@
// +build freebsd darwin // +build !windows
// Package platform provides helper function to get the runtime architecture // Package platform provides helper function to get the runtime architecture
// for different platforms. // for different platforms.
package platform // import "github.com/docker/docker/pkg/platform" package platform // import "github.com/docker/docker/pkg/platform"
import ( import (
"os/exec" "bytes"
"strings"
"golang.org/x/sys/unix"
) )
// runtimeArchitecture gets the name of the current architecture (x86, x86_64, i86pc, sun4v, ...) // runtimeArchitecture gets the name of the current architecture (x86, x86_64, i86pc, sun4v, ...)
func runtimeArchitecture() (string, error) { func runtimeArchitecture() (string, error) {
cmd := exec.Command("/usr/bin/uname", "-m") utsname := &unix.Utsname{}
machine, err := cmd.Output() if err := unix.Uname(utsname); err != nil {
if err != nil {
return "", err return "", err
} }
return strings.TrimSpace(string(machine)), nil return string(utsname.Machine[:bytes.IndexByte(utsname.Machine[:], 0)]), nil
} }