Re: [PATCH] Harden check for used modules in hostonly mode

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

 



On Sunday 09 of May 2010 22:23:58 Andrey Borzenkov wrote:
> Make sure
that we do not accept module name which is substring of
> some other
module name. This resulted in piix being mistakenly loaded
> together
with ata_piix. It completely broke DVD access here.
> 
>
Signed-off-by: Andrey Borzenkov <arvidjaar@xxxxxxx>
> 

Ping. In
2.6.35 it became much worse. In 2.6.33 I had DVD as IDE and HD as PATA
which was OK. In 2.6.35 using exactly the same setup I lost any access
to HD. I.e. with these modules:

{pts/1}% lsmod | grep piix
ata_piix  
            22460  0 
libata                193241  4
ata_generic,pata_acpi,ata_piix,ahci
piix                    5408  1

ide_core              116373  3
ide_cd_mod,ide_pci_generic,piix

2.6.33 detects DVD as hda and HDD as
sda; with 2.6.35 only hda is detected. If I unload IDE stack and
reload ata_piix it correctly detects both HDD as sda and DVD as
sr0.

So without this patch dracut generates initrd that cannot find
root device.

Thank you!

> ---
> 
>  dracut-functions |    2 +-
>  1
files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git
a/dracut-functions b/dracut-functions
> index fdcfb7e..ebf09c0
100755
> --- a/dracut-functions
> +++ b/dracut-functions
> @@ -502,7
+502,7 @@ instmods() {
>  	        [[ -f $initdir/$1 ]] && { shift;
continue; }
>  		# If we are building a host-specific initramfs and
this
>  		# module is not already loaded, move on to the next one.
>
-		[[ $hostonly ]] && ! grep -q "${mod//-/_}" /proc/modules && \
>
+		[[ $hostonly ]] && ! grep -qe "\<${mod//-/_}\>" /proc/modules &&
\
>  	        ! echo $add_drivers | grep -qe "\<${mod}\>" &&	{
>  		  
 shift; continue;
>  		}
> 
> --
> To unsubscribe from this list: send
the line "unsubscribe initramfs"
> in the body of a message to
majordomo@xxxxxxxxxxxxxxx
> More majordomo info at 
http://vger.kernel.org/majordomo-info.html

Attachment: signature.asc
Description: This is a digitally signed message part.


[Index of Archives]     [Linux Kernel]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux SCSI]

  Powered by Linux