Date: Sat, 23 Apr 2011 13:57:12 +0000 (UTC) From: "Simon L. Nielsen" <simon@FreeBSD.org> To: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: svn commit: r220970 - head/usr.sbin/timed/timedc Message-ID: <201104231357.p3NDvCk1092033@svn.freebsd.org>
next in thread | raw e-mail | index | archive | help
Author: simon Date: Sat Apr 23 13:57:12 2011 New Revision: 220970 URL: http://svn.freebsd.org/changeset/base/220970 Log: Check return code of setuid() in timedc. While it will not fail in normal circumstances, better safe than sorry. MFC after: 3 days Modified: head/usr.sbin/timed/timedc/timedc.c Modified: head/usr.sbin/timed/timedc/timedc.c ============================================================================== --- head/usr.sbin/timed/timedc/timedc.c Sat Apr 23 13:42:03 2011 (r220969) +++ head/usr.sbin/timed/timedc/timedc.c Sat Apr 23 13:57:12 2011 (r220970) @@ -75,7 +75,8 @@ main(argc, argv) */ if (priv_resources() < 0) errx(1, "could not get privileged resources"); - (void) setuid(getuid()); + if (setuid(getuid()) != 0) + err(1, "setuid()"); if (--argc > 0) { c = getcmd(*++argv);
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201104231357.p3NDvCk1092033>