Skip to content

Commit f14161d

Browse files
committed
use logger in test-no-escalate
Signed-off-by: Caleb Lloyd <caleblloyd@gmail.com>
1 parent cb0f768 commit f14161d

1 file changed

Lines changed: 11 additions & 7 deletions

File tree

test-no-escalate/test-no-escalate.go

Lines changed: 11 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,31 +1,35 @@
11
package main
22

33
import (
4-
"fmt"
4+
"log"
55
"os"
66
"syscall"
77
)
88

9+
var logger = log.New(os.Stderr, "", 0)
10+
911
func main() {
10-
fmt.Printf("Current UID: %d, GID: %d\n", os.Getuid(), os.Getgid())
11-
fmt.Printf("Current EUID: %d, EGID: %d\n", os.Geteuid(), os.Getegid())
12+
logger.SetPrefix("test-no-escalate: ")
13+
14+
logger.Printf("Current UID: %d, GID: %d", os.Getuid(), os.Getgid())
15+
logger.Printf("Current EUID: %d, EGID: %d", os.Geteuid(), os.Getegid())
1216

1317
// Test that both seteuid(0) and setegid(0) fail as expected
1418
euidError := syscall.Seteuid(0)
1519
egidError := syscall.Setegid(0)
1620

1721
if euidError != nil && egidError != nil {
18-
fmt.Printf("Got expected error when setting EUID to 0: %v\n", euidError)
19-
fmt.Printf("Got expected error when setting EGID to 0: %v\n", egidError)
22+
logger.Printf("Got expected error when setting EUID to 0: %v", euidError)
23+
logger.Printf("Got expected error when setting EGID to 0: %v", egidError)
2024
// This is the expected behavior - exit with success
2125
os.Exit(0)
2226
} else {
2327
// At least one of them succeeded, which is a security vulnerability
2428
if euidError == nil {
25-
fmt.Printf("ERROR: Successfully set EUID to 0. New EUID: %d\n", os.Geteuid())
29+
logger.Printf("ERROR: Successfully set EUID to 0. New EUID: %d", os.Geteuid())
2630
}
2731
if egidError == nil {
28-
fmt.Printf("ERROR: Successfully set EGID to 0. New EGID: %d\n", os.Getegid())
32+
logger.Printf("ERROR: Successfully set EGID to 0. New EGID: %d", os.Getegid())
2933
}
3034
// Exit with failure
3135
os.Exit(1)

0 commit comments

Comments
 (0)