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)
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.


No differences found