Re: [patch 02/19] acpi: fix single linked list manipulation

[Len Brown <lenb@kernel.org>]

Applied.

thanks,
-Len

On Tuesday 19 December 2006 15:56, akpm@osdl.org wrote:
> From: Akinobu Mita <akinobu.mita@gmail.com>
> 
> Fix single linked list manipulation for sub_driver.  If the remving entry
> is not on the head of the sub_driver list, it goes into infinate loop.
> 
> Though that infinite loop doesn't happen.  Because the only user of
> acpi_pci_register_dirver() is acpiphp.
> 
> Cc: Len Brown <len.brown@intel.com>
> Signed-off-by: Akinobu Mita <akinobu.mita@gmail.com>
> Signed-off-by: Andrew Morton <akpm@osdl.org>
> ---
> 
>  drivers/acpi/pci_root.c |    9 +++++----
>  1 files changed, 5 insertions(+), 4 deletions(-)
> 
> diff -puN drivers/acpi/pci_root.c~acpi-fix-single-linked-list-manipulation drivers/acpi/pci_root.c
> --- a/drivers/acpi/pci_root.c~acpi-fix-single-linked-list-manipulation
> +++ a/drivers/acpi/pci_root.c
> @@ -98,11 +98,12 @@ void acpi_pci_unregister_driver(struct a
>  
>  	struct acpi_pci_driver **pptr = &sub_driver;
>  	while (*pptr) {
> -		if (*pptr != driver)
> -			continue;
> -		*pptr = (*pptr)->next;
> -		break;
> +		if (*pptr == driver)
> +			break;
> +		pptr = &(*pptr)->next;
>  	}
> +	BUG_ON(!*pptr);
> +	*pptr = (*pptr)->next;
>  
>  	if (!driver->remove)
>  		return;
> _
>