Ah,  this file handle is passed into PosixMmapFile.  PosixMmapFile performs Read/Write mappings … that does not match WRONLY.  Maybe everything "could" be read only.  But that is a ton more work than this bug fix release should being changing.

On Jul 22, 2013, at 9:03 AM, Dave Parfitt <notifications@github.com> wrote:

> In util/env_posix.cc:
> 
> > @@ -509,6 +545,18 @@ class PosixEnv : public Env {
> >      return s;
> >    }
> >  
> > +  virtual Status NewWriteOnlyFile(const std::string& fname,
> > +                                 WritableFile** result) {
> > +    Status s;
> > +    const int fd = open(fname.c_str(), O_CREAT | O_RDWR | O_TRUNC, 0644);
> wouldn't you want O_WRONLY instead of O_RDWR?
> 
> —
> Reply to this email directly or view it on GitHub.
>