Re: [Patch] add check to /proc/devices read routines

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

 



On Tue, Apr 26, 2005 at 09:08:27PM -0400, Neil Horman wrote:
 > Patch to add check to get_chrdev_list and get_blkdev_list to prevent reads of
 > /proc/devices from spilling over the provided page if more than 4096 bytes of
 > string data are generated from all the registered character and block devices in
 > a system
 > 
 > Signed-off-by: Neil Horman <[email protected]>
 > 
 > 
 >  fs/char_dev.c         |   13 ++++++++++++-
 >  fs/proc/proc_misc.c   |    2 +-
 >  include/linux/genhd.h |    2 +-
 >  3 files changed, 14 insertions(+), 3 deletions(-)

Missing changes to drivers/block/genhd.c perhaps ?
You changed the prototype of get_blkdev_list(), but not
the implementation, which still takes a single arg.

I've not looked at this code at all, but it smells like
something that perhaps needs converting to use seq_file() and friends ?

		Dave

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux