Re: [rhel5-branch] Make execWithRedirect() append to the files. (#702024)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Aug 30, 2011 at 08:31:04AM +0200, Ales Kozumplik wrote:
> On 08/29/2011 08:50 PM, Brian C. Lane wrote:
> >Is this any safer? Seems like Popen is still going to have the same file
> >open twice. In RHEL6 we seem to be using pipes and tee to manage the
> >output streams from the process.
> 
> Yes, but I wouldn't like to fix this particular bug at the cost of
> merging all of that.
> 
> I think in fact both of the ways are 'safe': it is just the
> operating system doesn't advance the seek position of each of the
> (underlying) descriptors (which are indeed used by subprocess.py by
> calling .fileno() on the file object) in the old version, but it
> does that in the second, just as described by man 2 open, section
> O_APPEND.
> 
> Does that convince you? We might see a bug or two about this in the
> future but it is already broken at the moment.

Yep :) Ack.

-- 
Brian C. Lane | Anaconda Team | IRC: bcl #anaconda | Port Orchard, WA (PST8PDT)

Attachment: pgp_iSZII_M7H.pgp
Description: PGP signature

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list

[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux