Hi Hung,

yes, immnd_evt_proc_ccb_obj_modify_rsp needs to be corrected .
IMMND_EVT_A2ND_CCB_OBJ_CREATE_RSP is present instead of 
IMMND_EVT_A2ND_CCB_OBJ_MODIFY_RSP.

/Neel.
On Tuesday 22 September 2015 12:55 PM, Hung Nguyen wrote:
> Thanks, I got it.
>
> I think immnd_evt_proc_ccb_obj_modify_rsp() also needs to be corrected.
> It should be IMMND_EVT_A2ND_CCB_OBJ_MODIFY_RSP in stead of 
> IMMND_EVT_A2ND_CCB_OBJ_MODIFY_RSP_2.
>
> Best Regards,
>
> Hung Nguyen - DEK Technologies
>
>
> --------------------------------------------------------------------------------
>  
>
> From: Neelakanta Reddy [email protected]
> Sent: Tuesday, September 22, 2015 2:19PM
> To: Hung Nguyen, Anders Bjornerstedt, Zoran Milinkovic
>     [email protected], [email protected], 
> [email protected]
> Cc: Opensaf-devel
>     [email protected]
> Subject: Re: [devel] [PATCH 1 of 1] imm: corrected missmatch in the 
> event type for CCB create rsp [#1494]
>
>
> Hi Hung,
>
> The IMMND_EVT_A2ND_CCB_OBJ_CREATE_RSP is the message type comes from 
> OI when error string is not set.
>
> /Neel.
>
> On Tuesday 22 September 2015 12:44 PM, Hung Nguyen wrote:
>> Hi Neelakanta,
>>
>> 'errStrList' is populated in immnd_evt_proc_ccb_obj_create_rsp(), 
>> it's not from the OI.
>> Why we have to assert the type of incoming message (from OI) here?
>>
>> Best Regards,
>>
>> Hung Nguyen - DEK Technologies
>>
>>
>> --------------------------------------------------------------------------------
>>  
>>
>> From: Neelakanta Reddy [email protected]
>> Sent: Tuesday, September 22, 2015 2:08PM
>> To: Anders Bjornerstedt, Zoran Milinkovic
>>     [email protected], [email protected]
>> Cc: Opensaf-devel
>>     [email protected]
>> Subject: [devel] [PATCH 1 of 1] imm: corrected missmatch in the event 
>> type for CCB create rsp [#1494]
>>
>>
>>  osaf/services/saf/immsv/immnd/immnd_evt.c |  2 +-
>>  1 files changed, 1 insertions(+), 1 deletions(-)
>>
>>
>> diff --git a/osaf/services/saf/immsv/immnd/immnd_evt.c 
>> b/osaf/services/saf/immsv/immnd/immnd_evt.c
>> --- a/osaf/services/saf/immsv/immnd/immnd_evt.c
>> +++ b/osaf/services/saf/immsv/immnd/immnd_evt.c
>> @@ -3736,7 +3736,7 @@ static void immnd_evt_proc_ccb_obj_creat
>>                  send_evt.info.imma.info.errRsp.errStrings = &(strList);
>>                  send_evt.info.imma.type = IMMA_EVT_ND2A_IMM_ERROR_2;
>>              } else if(errStrList.name.buf) {
>> -                osafassert(evt->type == 
>> IMMND_EVT_A2ND_CCB_OBJ_MODIFY_RSP_2);
>> +                osafassert(evt->type == 
>> IMMND_EVT_A2ND_CCB_OBJ_CREATE_RSP);
>>                  send_evt.info.imma.info.errRsp.errStrings = 
>> &(errStrList);
>>                  send_evt.info.imma.type = IMMA_EVT_ND2A_IMM_ERROR_2;
>>              }
>>
>> ------------------------------------------------------------------------------
>>  
>>
>> _______________________________________________
>> Opensaf-devel mailing list
>> [email protected]
>> https://lists.sourceforge.net/lists/listinfo/opensaf-devel
>>
>>
>
>


------------------------------------------------------------------------------
_______________________________________________
Opensaf-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/opensaf-devel

Reply via email to