Skip to content

Commit

Permalink
os/user: small fixes for Solaris
Browse files Browse the repository at this point in the history
Change-Id: I56149ef6607fb4d9baff9047cb3a47d71cad6fa6
Reviewed-on: https://go-review.googlesource.com/8261
Reviewed-by: Minux Ma <minux@golang.org>
  • Loading branch information
4ad committed May 6, 2015
1 parent 121489c commit 24396da
Showing 1 changed file with 12 additions and 2 deletions.
14 changes: 12 additions & 2 deletions src/os/user/lookup_unix.go
Expand Up @@ -17,14 +17,20 @@ import (
)

/*
#cgo solaris CFLAGS: -D_POSIX_PTHREAD_SEMANTICS
#include <unistd.h>
#include <sys/types.h>
#include <pwd.h>
#include <stdlib.h>
static int mygetpwuid_r(int uid, struct passwd *pwd,
char *buf, size_t buflen, struct passwd **result) {
return getpwuid_r(uid, pwd, buf, buflen, result);
return getpwuid_r(uid, pwd, buf, buflen, result);
}
static int mygetpwnam_r(const char *name, struct passwd *pwd,
char *buf, size_t buflen, struct passwd **result) {
return getpwnam_r(name, pwd, buf, buflen, result);
}
*/
import "C"
Expand Down Expand Up @@ -67,7 +73,11 @@ func lookupUnix(uid int, username string, lookupByName bool) (*User, error) {
if lookupByName {
nameC := C.CString(username)
defer C.free(unsafe.Pointer(nameC))
rv = C.getpwnam_r(nameC,
// mygetpwnam_r is a wrapper around getpwnam_r to avoid
// passing a size_t to getpwnam_r, because for unknown
// reasons passing a size_t to getpwnam_r doesn't work on
// Solaris.
rv = C.mygetpwnam_r(nameC,
&pwd,
(*C.char)(buf),
C.size_t(bufSize),
Expand Down

0 comments on commit 24396da

Please sign in to comment.