Ack (reviewed only) /HansF > -----Original Message----- > From: [email protected] [mailto:[email protected]] > Sent: den 9 september 2014 13:02 > To: Hans Feldt; Hans Nordebäck; [email protected] > Cc: [email protected] > Subject: [PATCH 1 of 1] amfd: respond with ERR_INTERRUPT to su shutdown op > [#1048] > > osaf/services/saf/amf/amfd/su.cc | 4 +++- > 1 files changed, 3 insertions(+), 1 deletions(-) > > > Amf is responding with OK to su shutdown op if > su lock is issued before shutdown op completes. > Ideally, Amf should return ERR_INTERRUPT in this case. > > 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 > @@ -1156,7 +1156,9 @@ static void su_admin_op_cb(SaImmOiHandle > /* This means that shutdown was going on and lock has > been issued. In this case, response to shutdown > and then allow lock operation to proceed. */ > - su->complete_admin_op(SA_AIS_OK); > + report_admin_op_error(immoi_handle, > su->pend_cbk.invocation, > + SA_AIS_ERR_INTERRUPT, &su->pend_cbk, > + "SU lock has been issued '%s'", > su->name.value); > } > } > /* if Tolerance timer is running for any SI's withing this SG, then > return SA_AIS_ERR_TRY_AGAIN */
------------------------------------------------------------------------------ Want excitement? Manually upgrade your production database. When you want reliability, choose Perforce Perforce version control. Predictably reliable. http://pubads.g.doubleclick.net/gampad/clk?id=157508191&iu=/4140/ostg.clktrk _______________________________________________ Opensaf-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/opensaf-devel
