[ofw][IBAL] patch for pool corruption

[Yossi Leybovich]

Hi

Do you know why ven_id 0x66a ( former SilverStorm ) need special
treatment for the location query?
Can you use id_string in the query_description ?

Can we just use the same code for all IOCs

Yossi 
 

> -----Original Message-----
> From: Alex Estrin [mailto:alex.estrin at qlogic.com] 
> Sent: Friday, March 02, 2007 5:30 PM
> To: Yossi Leybovich
> Cc: ofw at lists.openfabrics.org
> Subject: [ofw][IBAL] patch for pool corruption
> 
> Hi Yossi,
> 
> This patch should fix potential pool corruption.
> Please review.
> 
> Thanks,
> Alex 
> 
> Index: iou_ioc_mgr.c
> ===================================================================
> --- iou_ioc_mgr.c	(revision 599)
> +++ iou_ioc_mgr.c	(working copy)
> @@ -1142,7 +1142,9 @@
>  		return STATUS_NO_SUCH_DEVICE;
>  	}
>  
> -	p_string = ExAllocatePool( PagedPool, IOC_LOCATION_SIZE );
> +	p_string = ExAllocatePool( PagedPool, 
> +		max( IOC_LOCATION_SIZE,
> +		sizeof( WCHAR ) * ( sizeof(
> p_ext->info.profile.id_string ) + 1 )));
>  	if( !p_string )
>  	{
>  		IOU_PRINT_EXIT( TRACE_LEVEL_ERROR, IOU_DBG_ERROR,
> 
>