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