diff options
| author | simon <simon@FreeBSD.org> | 2011-04-23 13:07:35 +0000 |
|---|---|---|
| committer | simon <simon@FreeBSD.org> | 2011-04-23 13:07:35 +0000 |
| commit | 5cb5f5de18dbcc0e7a5fcc9fffe53bc388328ecb (patch) | |
| tree | b767632380641dbd7124fc9107fe2dc6dc2d704f /contrib/traceroute/traceroute.c | |
| parent | 872195caf937fd6dc952eeeed83c61258243bb32 (diff) | |
| download | FreeBSD-src-5cb5f5de18dbcc0e7a5fcc9fffe53bc388328ecb.zip FreeBSD-src-5cb5f5de18dbcc0e7a5fcc9fffe53bc388328ecb.tar.gz | |
Check return code of setuid() in traceroute.
While it will not fail in normal circumstances, better safe than sorry.
Reported by: LLVM's clang static analyzer
MFC after: 3 days
Diffstat (limited to 'contrib/traceroute/traceroute.c')
| -rw-r--r-- | contrib/traceroute/traceroute.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/contrib/traceroute/traceroute.c b/contrib/traceroute/traceroute.c index db4d075..2a5fea2 100644 --- a/contrib/traceroute/traceroute.c +++ b/contrib/traceroute/traceroute.c @@ -509,7 +509,10 @@ main(int argc, char **argv) sockerrno = errno; } - setuid(getuid()); + if (setuid(getuid()) != 0) { + perror("setuid()"); + exit(1); + } #ifdef IPCTL_DEFTTL { |
