This change seems to be the same one that you just pushed as part of 767? If so what should we do with this defect? Thanks, Hans
On 4 February 2014 11:36, <praveen.malv...@oracle.com> wrote: > osaf/services/saf/amf/amfnd/comp.cc | 7 +++++++ > 1 files changed, 7 insertions(+), 0 deletions(-) > > > When AMFND responds to AMFD for the completion of removal of assignments, it > deletes the SUSI and COMPCSI > records. Reported problem appears when a component receiving multiple CSIs > responds to AMFND for the removal > of assignments and AMFND responds to AMFD for the completion of assignment. > Since component has multiple > CSIs assigned to it, AMFND tries to again generate removal completion > indication and access csi_list in the > component which has no nodes in it. This leads to AMFND crash. > This patch assures that AMFND will not access csi_list in component after > deletion of comp_csi record. > > diff --git a/osaf/services/saf/amf/amfnd/comp.cc > b/osaf/services/saf/amf/amfnd/comp.cc > --- a/osaf/services/saf/amf/amfnd/comp.cc > +++ b/osaf/services/saf/amf/amfnd/comp.cc > @@ -1761,6 +1761,13 @@ uint32_t avnd_comp_csi_remove_done(AVND_ > break; > } > } > + > + /* When AMFND responds to AMFD for removal of > assignments for the SIs in the any SU, > + it also deletes all the SUSI and COMPCSI records. > In such a case component will > + not have any CSI in its csi_list. If removal is > completed break the loop. > + */ > + if (comp->csi_list.n_nodes == 0) > + break; > } > > /* This is removal with TARGET_ALL. So if all CSIs in all SIs > of SU are moved > > ------------------------------------------------------------------------------ > Managing the Performance of Cloud-Based Applications > Take advantage of what the Cloud has to offer - Avoid Common Pitfalls. > Read the Whitepaper. > http://pubads.g.doubleclick.net/gampad/clk?id=121051231&iu=/4140/ostg.clktrk > _______________________________________________ > Opensaf-devel mailing list > Opensaf-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/opensaf-devel ------------------------------------------------------------------------------ Android apps run on BlackBerry 10 Introducing the new BlackBerry 10.2.1 Runtime for Android apps. Now with support for Jelly Bean, Bluetooth, Mapview and more. Get your Android app in front of a whole new audience. Start now. http://pubads.g.doubleclick.net/gampad/clk?id=124407151&iu=/4140/ostg.clktrk _______________________________________________ Opensaf-devel mailing list Opensaf-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/opensaf-devel
