makedir(): Removed the strange RUID!=EUID code and the access(2) call.
authormarkus schnalke <meillo@marmaro.de>
Fri, 23 Mar 2012 08:43:17 +0000 (09:43 +0100)
committermarkus schnalke <meillo@marmaro.de>
Fri, 23 Mar 2012 08:43:17 +0000 (09:43 +0100)
commit076d7cc09eccff3d01d153434d89455db375ac44
treecbd9a812878aa4e2fabad0e54e1ecfbe0b640ca9
parent7ffb36d28e517a6f3a10272056fc127592ab1c19
makedir(): Removed the strange RUID!=EUID code and the access(2) call.
Dan couldn't find a reason for the code neither, hence I assume it's save
to simply remove it.
    Ummm, why do we want to avoid creating directories
    with the effective user ID?  None of the nmh tools are
    installed such that the effective should be different
    from the real, and if some parent process made the two
    be different, I don't see why it should be our job to
    enforce the real UID.  Also, why the heck do we call
    the mkdir executable rather than the library function in
    this case??  If we do want to call the mkdir executable,
    we should at least be giving it -p (and change the single
    chmod() call below) so it can successfully create nested
    directories like the above code can.
    -- Dan Harkless <dan-nmh@dilvish.speed.net>
Concerning access(2): We should better always just try the action instead
of first asking if we are allowed to do it.
sbr/makedir.c