linux-kernel - Re: [PATCH] SCSI: userspace cannot use scsi_command_size_tbl, COMMAND_SIZE and scsi_device

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Sun, 28 Jun 2009 10:56:27 +0300
From:	Boaz Harrosh <bharrosh@...asas.com>
To:	Jaswinder Singh Rajput <jaswinder@...nel.org>,
	James Smart <james.smart@...lex.com>,
	Olaf Hering <olh@...e.de>,
	Andrew Morton <akpm@...ux-foundation.org>
CC:	James Bottomley <James.Bottomley@...senPartnership.com>,
	Sam Ravnborg <sam@...nborg.org>,
	Benny Halevy <bhalevy@...asas.com>,
	LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] SCSI: userspace cannot use scsi_command_size_tbl, COMMAND_SIZE
 and scsi_device_type

On 06/27/2009 08:05 PM, Jaswinder Singh Rajput wrote:
> userspace cannot use scsi_command_size_tbl, COMMAND_SIZE
> and scsi_device_type defined in kernel
> 
> fix the following 'make headers_check' warnings:
> 
>   usr/include/scsi/scsi.h:159: userspace cannot call function or variable defined in the kernel
>   usr/include/scsi/scsi.h:285: userspace cannot call function or variable defined in the kernel
> 
> Signed-off-by: Jaswinder Singh Rajput <jaswinderrajput@...il.com>

include/scsi/scsi.h was always provided by glibc and was never exported from Kernel.
It was only recently added to include/scsi/Kbuild. (In this merge window, was never released)
(see: [9e4f5e29] [SCSI] FC Pass Thru support)

Two questions. 
1. Are we sure this will work. Is the Header fully compatible with glibc's header.
2. What will this do to distor's package managers having the same file in two packages?
   Don't we need a followup patch sent to glibc to remove that Header from there?

James Smart
  What is the new definitions needed in user-mode in scsi.h that is not available in glibc's
supplied header?

(For reference see this patch from 2007: [e629a7dd] do not export /usr/include/scsi in make headers_install)

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