Ack but see inline

> -----Original Message-----
> From: Hans Nordebäck
> Sent: den 26 augusti 2014 10:12
> To: [email protected]; Hans Feldt
> Cc: [email protected]
> Subject: [PATCH 1 of 1] AMFND: avnd_evt_avd_operation_request_evh don't 
> handle bcast msg correct [#1010]
> 
>  osaf/services/saf/amf/amfnd/di.cc |  7 +++++--
>  1 files changed, 5 insertions(+), 2 deletions(-)
> 
> 
> diff --git a/osaf/services/saf/amf/amfnd/di.cc 
> b/osaf/services/saf/amf/amfnd/di.cc
> --- a/osaf/services/saf/amf/amfnd/di.cc
> +++ b/osaf/services/saf/amf/amfnd/di.cc
> @@ -209,8 +209,11 @@ uint32_t avnd_evt_avd_operation_request_
>       if (!m_AVND_CB_IS_AVD_UP(cb))
>               goto done;
> 
> -     avnd_msgid_assert(info->msg_id);
> -     cb->rcv_msg_id = info->msg_id;
> +     // If message was not broadcasted, (msg_id == 0)
> +     if (info->msg_id != 0) {
[Hans] this could be hidden in a "set_msg_id()" but which is more intrusive so 
for now you could just leave a TODO in the code.

> +             avnd_msgid_assert(info->msg_id);
> +             cb->rcv_msg_id = info->msg_id;
> +     }
>       switch (param->class_id) {
>       case AVSV_SA_AMF_NODE:
>               rc = avnd_node_oper_req(cb, param);

------------------------------------------------------------------------------
Slashdot TV.  
Video for Nerds.  Stuff that matters.
http://tv.slashdot.org/
_______________________________________________
Opensaf-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/opensaf-devel

Reply via email to