[ofa-general] [PATCH] ibutils: fixing seg. fault in ibis_gsi_mad_ctrl.c

[Yevgeny Kliteynik]

Hi Hal,

Hal Rosenstock wrote:
> On Tue, 2008-06-17 at 07:26 -0700, Hal Rosenstock wrote:
>> On Thu, 2008-05-22 at 15:20 +0300, Yevgeny Kliteynik wrote:
>>> Hi Oren,
>>>
>>> Fixing seg fault in allocation of gsi management class vector.
>> Was this ever committed ? Should it be applied for both trunk and
>> ofed_1_3 ?
> 
> Looks like it's there for trunk but not ofed_1_3.

It's not needed for ofed_1_3 - the seg fault happens only
when adding a new feature to ibutils (support for CC MADs),
which doesn't exist in ofed_1_3.

-- Yevgeny

> -- Hal
> 
>> -- Hal
>>
>>> Signed-off-by: Yevgeny Kliteynik <kliteyn at dev.mellanox.co.il>
>>> ---
>>>  ibis/src/ibis_gsi_mad_ctrl.c |    2 +-
>>>  1 files changed, 1 insertions(+), 1 deletions(-)
>>>
>>> diff --git a/ibis/src/ibis_gsi_mad_ctrl.c b/ibis/src/ibis_gsi_mad_ctrl.c
>>> index 356d33d..bfb5fe6 100644
>>> --- a/ibis/src/ibis_gsi_mad_ctrl.c
>>> +++ b/ibis/src/ibis_gsi_mad_ctrl.c
>>> @@ -731,7 +731,7 @@ ibis_gsi_mad_ctrl_set_class_attr_cb(
>>>    size = cl_vector_get_size(&p_ctrl->class_vector);
>>>    if (size <= mad_class)
>>>    {
>>> -    cl_status = cl_vector_set_size(&p_ctrl->class_vector,mad_class);
>>> +    cl_status = cl_vector_set_size(&p_ctrl->class_vector,mad_class+1);
>>>
>>>      if( cl_status != CL_SUCCESS)
>>>      {
>> _______________________________________________
>> general mailing list
>> general at lists.openfabrics.org
>> http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general
>>
>> To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general
> 
>