Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 1 Jul 2004 20:16:18 -0700
From:      David Schultz <das@FreeBSD.ORG>
To:        "Christian S.J. Peron" <csjp@FreeBSD.ORG>
Cc:        cvs-all@FreeBSD.ORG
Subject:   Re: cvs commit: src/libexec/tftpd tftpd.c
Message-ID:  <20040702031618.GA80812@VARK.homeunix.com>
In-Reply-To: <200406301858.i5UIwJkB047720@repoman.freebsd.org>

index | next in thread | previous in thread | raw e-mail

On Wed, Jun 30, 2004, Christian S.J. Peron wrote:
> csjp        2004-06-30 18:58:19 UTC
> 
>   FreeBSD src repository
> 
>   Modified files:
>     libexec/tftpd        tftpd.c 
>   Log:
>   The call to setuid(2) subsequently causes setgroups(2) to fail. setgroups(2)
>   requires super-user access in order to complete successfully.
>   Move setgroups(2) to execute before setuid(2) so that it is successful.

Error checks would also be appropriate here, and would have
caught the bug.


home | help

Want to link to this message? Use this
URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20040702031618.GA80812>