Hi Hoang Vo,
Can you please also make the fix more readable
replace `for (count = 0; count < ckpt_node->node_users_cnt; count++) {`
some thing like `for (node_user = ckpt_node->node_users; node_user !=
NULL; node_user = node_user->next) {`
then, we can remove variable `int count = 0;`, move
`cpd_msg.info.usr_info_2.node_users_cnt = ckpt_node->node_users_cnt;`
after for () loop.
-AVM
On 11/16/2016 8:51 AM, A V Mahesh wrote:
> Hi Hoang Vo,
>
> On 11/15/2016 12:57 PM, Hoang Vo wrote:
>> Testing Commands:
>> -----------------
>>
>>
>> Testing, Expected Results:
>> --------------------------
>>
> Can you please share test case .
>
> -AVM
>
> On 11/15/2016 12:57 PM, Hoang Vo wrote:
>> osaf/services/saf/cpsv/cpd/cpd_red.c | 5 +++++
>> 1 files changed, 5 insertions(+), 0 deletions(-)
>>
>>
>> diff --git a/osaf/services/saf/cpsv/cpd/cpd_red.c
>> b/osaf/services/saf/cpsv/cpd/cpd_red.c
>> --- a/osaf/services/saf/cpsv/cpd/cpd_red.c
>> +++ b/osaf/services/saf/cpsv/cpd/cpd_red.c
>> @@ -322,6 +322,11 @@ void cpd_a2s_ckpt_usr_info(CPD_CB *cb, C
>> memset(cpd_msg.info.usr_info_2.node_list, '\0',
>> (sizeof(CPD_NODE_USER_INFO) * ckpt_node->node_users_cnt));
>>
>> for (count = 0; count < ckpt_node->node_users_cnt; count++) {
>> + if (node_user == NULL) {
>> + ckpt_node->node_users_cnt = count;
>> + cpd_msg.info.usr_info_2.node_users_cnt = count;
>> + break;
>> + }
>> cpd_msg.info.usr_info_2.node_list[count].dest =
>> node_user->dest;
>> cpd_msg.info.usr_info_2.node_list[count].num_users =
>> node_user->num_users;
>> cpd_msg.info.usr_info_2.node_list[count].num_readers =
>> node_user->num_readers;
>
> ------------------------------------------------------------------------------
> _______________________________________________
> Opensaf-devel mailing list
> Opensaf-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/opensaf-devel
------------------------------------------------------------------------------
_______________________________________________
Opensaf-devel mailing list
Opensaf-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/opensaf-devel
