Hi Thang, ACK
Best Regards, ThuanTr -----Original Message----- From: thang.d.nguyen <thang.d.ngu...@dektech.com.au> Sent: Thursday, January 2, 2020 10:06 AM To: gary....@dektech.com.au; minh.c...@dektech.com.au; thuan.t...@dektech.com.au Cc: opensaf-devel@lists.sourceforge.net; thang.d.nguyen <thang.d.ngu...@dektech.com.au> Subject: [PATCH 1/1] amfd: not allow unlock/lock if su is under restarting [#3132] Not allow unlock/lock if su is under restarting. --- src/amf/amfd/su.cc | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/src/amf/amfd/su.cc b/src/amf/amfd/su.cc index e1da8f726..5a6c69c33 100644 --- a/src/amf/amfd/su.cc +++ b/src/amf/amfd/su.cc @@ -1479,6 +1479,15 @@ static void su_admin_op_cb(SaImmOiHandleT immoi_handle, goto done; } + if ((su->pend_cbk.admin_oper == SA_AMF_ADMIN_RESTART) && + ((op_id == SA_AMF_ADMIN_UNLOCK) || (op_id == SA_AMF_ADMIN_LOCK))) { + report_admin_op_error( + immoi_handle, invocation, SA_AIS_ERR_TRY_AGAIN, nullptr, + "SU'%s', undergoing admin operation'%u'", + su->name.c_str(), su->pend_cbk.admin_oper); + goto done; + } + /* Validation has passed and admin operation should be done. Proceed with * it... */ switch (op_id) { -- 2.17.1 _______________________________________________ Opensaf-devel mailing list Opensaf-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/opensaf-devel