Huy Le <[EMAIL PROTECTED]> writes:
> There's another intermittent bug when I specify a Fcc
> field that requires a folder to be created, and then another
> folder under it to be created.  I guess it's send's fault?
> 
> With "send -verbose", and a "Fcc: s/feed/nmh" where +s
> exists, but +s/feed does not exist, here's what I get
> when I send email:
> 
> What now? s
>  -- Posting for All Recipients --
>   -- Network Recipients --
>   jon at dnai.com: address ok
>  -- Recipient Copies Posted --
>  -- Filing Folder Copies --
>   Fcc s/feed/nmh: Create folder "/home/huyle/Mail/s/feed/nmh"? y
> refile: unable to create directory /home/huyle/Mail/s/feed/nmh: Permission denied
> refile: unable to create folder /home/huyle/Mail/s/feed/nmh
> exit 1
>  -- Folder Copies Filed --
> Message Processed
> 
> 
> It turns out that the ~/Mail/s/feed directory is created (by send?) with the
> drw------- permissions instead of drwx------, so naturally
> a subfolder cannot be created.

Thanks for finding that, Huy.  That bug apparently dates back to MH.
Surprising no one caught it before.  The reason the directory was missing
its execute permission was that in makedir() the default folder mode "0700"
was being passed to atoi(), which interpreted it as a decimal constant.
I've changed it to call strtoul() so it's interpreted properly.  I also
fixed a bug where when creating nested directories it didn't even try to set
the correct permissions except on the innermost directory.

While I was in makedir() I added a comment relating my confusion as to why
we go through a separate code path where the mkdir _command_ is called if
the effective and real user IDs don't match.  Maybe a few other folks could
look at that code -- my inclination is to delete it, but I don't want to do
that if there's a good reason for its existence.

Huy, do you have CVS installed?  If so, go ahead and update to the latest
source.  If not, you can get the latest version from cvsweb on
http://www.mhost.com/nmh/cvs.html (the changes are a bit long for a patch).

Another fix I just made is that repeat 'make install's would fail because
the check for the existence of mh_profile.5 was written incorrectly.

-----------------------------------------------------------------------
Dan Harkless                   | To prevent SPAM contamination, please 
[EMAIL PROTECTED]      | do not post this private email address
SpeedGate Communications, Inc. | to the USENET or WWW.  Thank you.     

Reply via email to