Re: The open() system call in f8 really broken...

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

 



On Friday, 17 August 2007 at 01:06, Dave Jones wrote:
> On Thu, Aug 16, 2007 at 04:36:37PM -0400, Dave Jones wrote:
>  > On Thu, Aug 16, 2007 at 04:21:38PM -0400, Dave Jones wrote:
>  >  > On Thu, Aug 16, 2007 at 05:08:15PM +0200, Oliver Falk wrote:
>  >  > 
>  >  >  > If you compile the whole Fedora tree, how many warnings will you see?
>  >  >  
>  >  > So I grepped across a make prep'd tree of devel (all 63 gig of it).
>  >  > Here's the fallout..
>  >  > 
>  >  > openmpi/openmpi-1.2.3/orte/runtime/orte_abort.c         fd = open(abort_file, O_CREAT);
>  >  > jython/jython-svn-Release_2_2beta1/CPythonLib/test/test_unicode_file.py:f = os.open(TESTFN_ENCODED, os.O_CREAT)
>  >  > perl/perl-5.8.8/t/op/taint.t:   eval { sysopen(my $cr, $evil, &O_CREAT) };
>  >  > proftpd/proftpd-1.3.0a/contrib/mod_rewrite.c:    if ((fifo_lockfd = open(fifo_lockname, O_CREAT)) < 0)
>  >  > pwlib/pwlib-1.10.7/configure:sem_t *s = sem_open("test", O_CREAT)
>  >  > pwlib/pwlib-1.10.7/configure.ac:                 [sem_t *s = sem_open("test", O_CREAT)],
>  >  > python/Python-2.5.1/Lib/test/test_unicode_file.py:        f = os.open(filename, os.O_CREAT)
>  >  > python-docs/Python-2.5.1/Lib/test/test_unicode_file.py:        f = os.open(filename, os.O_CREAT)
>  >  > xca/xca-0.6.3/_tmp_root/usr/lib/python2.5/test/test_unicode_file.py:        f = os.open(filename, os.O_CREAT)
>  >  > 
>  >  > Not too bad considering.
>  > 
>  > Bah, my regexp was imperfect.  I'll fix up and rerun.
> 
> More like it. 267 hits.
> 
> 	Dave
> 
[...]
> Checking ./dx/dx-4.4.4/src/exec/libdx/fileio.c
> Found open with O_CREAT and no mode.
> 	    fd = open(name, O_WRONLY | O_CREAT);

I'll patch this up and send fix upstream. Thanks!

Regards,
R.

-- 
Fedora contributor http://fedoraproject.org/wiki/DominikMierzejewski
Livna contributor http://rpm.livna.org MPlayer developer http://mplayerhq.hu
"Faith manages."
        -- Delenn to Lennier in Babylon 5:"Confessions and Lamentations"

--
Fedora-maintainers mailing list
Fedora-maintainers@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/fedora-maintainers

--
Fedora-maintainers-readonly mailing list
Fedora-maintainers-readonly@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/fedora-maintainers-readonly

[Index of Archives]     [Fedora Users]     [Fedora Development]     [Fedora Devel Java]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux