Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 8 Jul 2009 05:56:14 +0000 (UTC)
From:      Marcel Moolenaar <marcel@FreeBSD.org>
To:        cvs-src-old@freebsd.org
Subject:   cvs commit: src/sys/dev/ata atapi-cd.c src/sys/geom geom_dev.c src/sys/geom/part g_part.c g_part_if.m src/sys/sys disk.h
Message-ID:  <200907080558.n685wIng085743@repoman.freebsd.org>

next in thread | raw e-mail | index | archive | help
marcel      2009-07-08 05:56:14 UTC

  FreeBSD src repository

  Modified files:
    sys/dev/ata          atapi-cd.c 
    sys/geom             geom_dev.c 
    sys/geom/part        g_part.c g_part_if.m 
    sys/sys              disk.h 
  Log:
  SVN rev 195436 on 2009-07-08 05:56:14Z by marcel
  
  Revert revisions 188839 and 188868. Use of the ioctl in geom_dev.c
  is invalid because the ioctl happens without prior open. The ioctl
  got introduced to provide backward compatibility for extended
  partitions, but it ended up not being used because it didn't work
  as expected. Since there are no consumers of the ioctl and the
  implementation is broken, the best fix is to remove the code
  entirely.
  
  Spotted by:     phk
  Approved by:    re (kensmith)
  
  Revision  Changes    Path
  1.206     +1 -4      src/sys/dev/ata/atapi-cd.c
  1.102     +0 -12     src/sys/geom/geom_dev.c
  1.39      +0 -28     src/sys/geom/part/g_part.c
  1.9       +0 -9      src/sys/geom/part/g_part_if.m
  1.45      +0 -6      src/sys/sys/disk.h



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200907080558.n685wIng085743>