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

Jon Ciesla limb at jcomserv.net
Fri Aug 17 13:45:21 UTC 2007


> 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 ./ettercap/ettercap-NG-0.7.3/src/ec_log.c
> Found open with O_CREAT and no mode.
> 	fd->fd = open(filename, O_CREAT | O_TRUNC | O_RDWR | O_BINARY);

> Checking ./bombardier/bombardier-0.8.2/hof.c
> Found open with O_CREAT and no mode.
> 	fd = open("/var/games/bombardi

Wow.  Thanks, these are mine, and this made it easy to find, patch, build,
test, send upstream, and build for devel.

>
>
> --
> http://www.codemonkey.org.uk
>
> --
> Fedora-maintainers mailing list
> Fedora-maintainers at redhat.com
> https://www.redhat.com/mailman/listinfo/fedora-maintainers
>


-- 
novus ordo absurdum




More information about the Fedora-maintainers mailing list