Hi, Ack from me after the following comment is fixed. As a general guideline, *please avoid* modifying the copyright part of the code. The 'authors' may be edited if any significant architectural/design changes are made to the OpenSAF services.
Thanks, Mathi. On Tue, Sep 25, 2018 at 2:03 PM Meenakshi TK <[email protected]> wrote: > --- > src/plm/plmd/plms_imm.c | 11 ++++++++--- > 1 file changed, 8 insertions(+), 3 deletions(-) > > diff --git a/src/plm/plmd/plms_imm.c b/src/plm/plmd/plms_imm.c > index 9fbba6c..bfdba45 100644 > --- a/src/plm/plmd/plms_imm.c > +++ b/src/plm/plmd/plms_imm.c > @@ -12,7 +12,7 @@ > * licensing terms. > * > * Author(s): Emerson Network Power > - * > + * High Availability Solutions Pvt Ltd > */ > > > > /***************************************************************************** > @@ -600,10 +600,10 @@ void plms_modify_objects(SaInt32T obj_type, SaNameT > *obj_name, > } > static void plms_delete_objects(SaInt32T obj_type, SaNameT *obj_name) > { > - PLMS_GROUP_ENTITY *child_list, *temp; > + PLMS_GROUP_ENTITY *child_list = NULL, *temp; > PLMS_ENTITY *plm_ent, *parent, *tmp; > SaUint32T rc; > - TRACE_ENTER(); > + TRACE_ENTER2("%s", obj_name->value); > switch (obj_type) { > case PLMS_HE_OBJ_TYPE: > case PLMS_EE_OBJ_TYPE: > @@ -612,6 +612,10 @@ static void plms_delete_objects(SaInt32T obj_type, > SaNameT *obj_name) > &plms_cb->entity_info, (SaUint8T *)obj_name); > /* No need to check if plm_ent is null */ > /* Update the parent/child relationship */ > + if (plm_ent == NULL) { > + TRACE_2("%s not found", obj_name->value); > + goto end; > + } > parent = plm_ent->parent; > if (parent == NULL) { > /* Parent is domain object */ > @@ -686,6 +690,7 @@ static void plms_delete_objects(SaInt32T obj_type, > SaNameT *obj_name) > plms_delete_vm_obj(obj_name); > break; > } > +end: > TRACE_LEAVE(); > } > static SaAisErrorT plms_imm_ccb_obj_create_cbk(SaImmOiHandleT imm_oi_hdl, > -- > 2.7.4 > > _______________________________________________ Opensaf-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/opensaf-devel
