* is_my_tty returns a bool.

* Avoid implicit conversion of integers to booleans.
* Add brackets.
This commit is contained in:
nekral-guest 2008-05-25 21:33:38 +00:00
parent 06d2a32a3e
commit b94825bbad
2 changed files with 18 additions and 9 deletions

View File

@ -1,3 +1,10 @@
2008-05-25 Nicolas François <nicolas.francois@centraliens.net>
* libmisc/chowntty.c: is_my_tty returns a bool.
* libmisc/chowntty.c: Avoid implicit conversion of integers to
booleans.
* libmisc/chowntty.c: Add brackets.
2008-05-25 Nicolas François <nicolas.francois@centraliens.net>
* libmisc/chowndir.c: Avoid assignment in comparisons, implicit

View File

@ -46,17 +46,19 @@
/*
* is_my_tty -- determine if "tty" is the same as TTY stdin is using
*/
static int is_my_tty (const char *tty)
static bool is_my_tty (const char *tty)
{
struct stat by_name, by_fd;
if (stat (tty, &by_name) || fstat (0, &by_fd))
return 0;
if ((stat (tty, &by_name) != 0) || (fstat (0, &by_fd) != 0)) {
return false;
}
if (by_name.st_rdev != by_fd.st_rdev)
return 0;
else
return 1;
if (by_name.st_rdev != by_fd.st_rdev) {
return false;
} else {
return true;
}
}
/*
@ -102,8 +104,8 @@ void chown_tty (const char *tty, const struct passwd *info)
exit (1);
}
if (chown (tty, info->pw_uid, gid) ||
chmod (tty, getdef_num ("TTYPERM", 0600))) {
if ((chown (tty, info->pw_uid, gid) != 0)||
(chmod (tty, getdef_num ("TTYPERM", 0600)) != 0)) {
int err = errno;
snprintf (buf, sizeof buf, _("Unable to change tty %s"), tty);