Ack
/Hans

> -----Original Message-----
> From: Gary Lee [mailto:[email protected]]
> Sent: den 22 maj 2014 09:20
> To: Hans Feldt
> Cc: [email protected]
> Subject: [PATCH 1 of 1] amfd: use correct SU when calling set_su_failover 
> [#713]
> 
>  osaf/services/saf/amf/amfd/su.cc |  2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> 
> A typo was introduced when some SU logic was
> moved from comp.cc to su.cc. This may have caused
> a segfault when set_su_failover() is called
> on a null pointer.
> 
> set_su_failover() should be called on "this" SU object.
> 
> diff --git a/osaf/services/saf/amf/amfd/su.cc 
> b/osaf/services/saf/amf/amfd/su.cc
> --- a/osaf/services/saf/amf/amfd/su.cc
> +++ b/osaf/services/saf/amf/amfd/su.cc
> @@ -169,7 +169,7 @@ void AVD_SU::remove_comp(AVD_COMP *comp)
>               /* If SU becomes NPI then set saAmfSUFailover flag
>                * Sec 3.11.1.3.2 AMF-B.04.01 spec */
>               if (saAmfSUPreInstantiable == false) {
> -                     comp->su->set_su_failover(true);
> +                     set_su_failover(true);
>               }
>       }
>  }

------------------------------------------------------------------------------
"Accelerate Dev Cycles with Automated Cross-Browser Testing - For FREE
Instantly run your Selenium tests across 300+ browser/OS combos.
Get unparalleled scalability from the best Selenium testing platform available
Simple to use. Nothing to install. Get started now for free."
http://p.sf.net/sfu/SauceLabs
_______________________________________________
Opensaf-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/opensaf-devel

Reply via email to