diff options
author | Darren Tucker <dtucker@zip.com.au> | 2016-10-26 08:43:25 +1100 |
---|---|---|
committer | Darren Tucker <dtucker@zip.com.au> | 2016-10-26 08:43:25 +1100 |
commit | b4e96b4c9bea4182846e4942ba2048e6d708ee54 (patch) | |
tree | bf113d6252dd709577e8a5310faf1aef289f776c | |
parent | 1c4ef0b808d3d38232aeeb1cebb7e9a43def42c5 (diff) |
Use !=NULL instead of >0 for getdefaultproj.
getdefaultproj() returns a pointer so test it for NULL inequality
instead of >0. Fixes compiler warning and is more correct. Patch from
David Binderman.
-rw-r--r-- | openbsd-compat/port-solaris.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/openbsd-compat/port-solaris.c b/openbsd-compat/port-solaris.c index bb8fccb41..0e89dc326 100644 --- a/openbsd-compat/port-solaris.c +++ b/openbsd-compat/port-solaris.c | |||
@@ -213,7 +213,7 @@ solaris_set_default_project(struct passwd *pw) | |||
213 | 213 | ||
214 | /* get default project, if we fail just return gracefully */ | 214 | /* get default project, if we fail just return gracefully */ |
215 | if ((defaultproject = getdefaultproj(pw->pw_name, &tempproject, &buf, | 215 | if ((defaultproject = getdefaultproj(pw->pw_name, &tempproject, &buf, |
216 | sizeof(buf))) > 0) { | 216 | sizeof(buf))) != NULL) { |
217 | /* set default project */ | 217 | /* set default project */ |
218 | if (setproject(defaultproject->pj_name, pw->pw_name, | 218 | if (setproject(defaultproject->pj_name, pw->pw_name, |
219 | TASK_NORMAL) != 0) | 219 | TASK_NORMAL) != 0) |