Skip to content

Commit

Permalink
Merge pull request #237 from kinvolk/iaguis/fix-runningFromSystemService
Browse files Browse the repository at this point in the history
util: fix runningFromSystemService
  • Loading branch information
lucab committed Jul 31, 2017
2 parents 24036eb + b703c57 commit d219646
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions util/util_cgo.go
Original file line number Diff line number Diff line change
Expand Up @@ -122,11 +122,12 @@ func runningFromSystemService() (ret bool, err error) {
errno := C.my_sd_pid_get_owner_uid(sd_pid_get_owner_uid, 0, &uid)
serrno := syscall.Errno(-errno)
// when we're running from a unit file, sd_pid_get_owner_uid returns
// ENOENT (systemd <220) or ENXIO (systemd >=220)
// ENOENT (systemd <220), ENXIO (systemd 220-223), or ENODATA
// (systemd >=234)
switch {
case errno >= 0:
ret = false
case serrno == syscall.ENOENT, serrno == syscall.ENXIO:
case serrno == syscall.ENOENT, serrno == syscall.ENXIO, serrno == syscall.ENODATA:
// Since the implementation of sessions in systemd relies on
// the `pam_systemd` module, using the sd_pid_get_owner_uid
// heuristic alone can result in false positives if that module
Expand Down

0 comments on commit d219646

Please sign in to comment.