Hi!

Did you intend the fix for ticket [#1716] to go into the OpenSAF 5.0 
release? From what I can tell, it has so far only been pushed to the 
default branch (which will become OpenSAF 5.1). The fix didn't go into 
the RC1.

regards,
Anders Widell

On 04/11/2016 11:17 AM, Zoran Milinkovic wrote:
> Hi Neelakanta,
>
> Reviewed the patch,
> Ack from me.
>
> Thanks,
> Zoran
>
>
> -----Original Message-----
> From: [email protected] [mailto:[email protected]]
> Sent: Tuesday, April 05, 2016 3:13 PM
> To: Zoran Milinkovic
> Cc: [email protected]
> Subject: [PATCH 1 of 1] imm: Remove the assertion when object is modified 
> with 2PBE [#1716]
>
>   osaf/services/saf/immsv/immnd/immnd_evt.c |  4 +++-
>   1 files changed, 3 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
> @@ -6559,7 +6559,9 @@ static void immnd_evt_proc_object_modify
>                                               
> oi_cl_node->version.majorVersion == 0x2 &&
>                                               oi_cl_node->version.releaseCode 
> == 'A') {
>                                       if (applConnArr[ix] == pbeApplierConn) 
> { /* Slave pbe */
> -                                             
> osafassert(canonicalizedAttrMod); /* Must already be built for primary pbe */
> +                                             if(!canonicalizedAttrMod) { /* 
> Must already be built for primary pbe */
> +                                                     canonicalizedAttrMod = 
> immModel_canonicalizeAttrModification(cb, &(evt->info.objModify));       
> +                                             }
>                                               
> send_evt.info.imma.info.objModify.attrMods = canonicalizedAttrMod;
>                                       } else {
>                                               if (!allWritableAttr) { /* 
> Check if allWritableAttr is already built */
>
> ------------------------------------------------------------------------------
> Find and fix application performance issues faster with Applications Manager
> Applications Manager provides deep performance insights into multiple tiers of
> your business applications. It resolves application problems quickly and
> reduces your MTTR. Get your free trial! http://pubads.g.doubleclick.net/
> gampad/clk?id=1444514301&iu=/ca-pub-7940484522588532
> _______________________________________________
> Opensaf-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/opensaf-devel
>


------------------------------------------------------------------------------
Find and fix application performance issues faster with Applications Manager
Applications Manager provides deep performance insights into multiple tiers of
your business applications. It resolves application problems quickly and
reduces your MTTR. Get your free trial!
https://ad.doubleclick.net/ddm/clk/302982198;130105516;z
_______________________________________________
Opensaf-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/opensaf-devel

Reply via email to