osaf/libs/core/mds/mds_dt_common.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
MDS while deleting a down process completely, fist determines the process
doesn't exist forever,
if process not exist any more , then only it deletes the PID onfo ,to avoid
race with a client that re-registers immediately after unregistered.
Currently this validation is not fulfilling NON-ROOT case , where we do get :
If PID exist :
OpenSAF process running as non-root - kill() returns -1 errno is EPERM
OpenSAF process running as root - kill() returns 0 errno is NONE
If PID NOT exist :
OpenSAF process running as as non-root - kill() returns -1 errno is ESRCH
OpenSAF process running as as root - kill() returns -1 errno is ESRCH
So added additional validation to fulfill the not root case.
diff --git a/osaf/libs/core/mds/mds_dt_common.c
b/osaf/libs/core/mds/mds_dt_common.c
--- a/osaf/libs/core/mds/mds_dt_common.c
+++ b/osaf/libs/core/mds/mds_dt_common.c
@@ -857,7 +857,7 @@ uint32_t mds_tmr_mailbox_processing(void
tmr_req_info->info.down_event_tmr_info.svc_id);
/* only delete if process not exist to avoid
race with a client
* that re-registers immediately after
unregister */
- if ((info != NULL) && (kill(info->pid, 0) ==
-1)) {
+ if ((info != NULL) && ((kill(info->pid, 0) ==
-1) && (errno != EPERM))) {
TRACE("TIMEOUT, deleting entry for
%"PRIx64", pid:%d",
info->mds_dest, info->pid);
(void)mds_process_info_del(info);
------------------------------------------------------------------------------
What NetFlow Analyzer can do for you? Monitors network bandwidth and traffic
patterns at an interface-level. Reveals which users, apps, and protocols are
consuming the most bandwidth. Provides multi-vendor support for NetFlow,
J-Flow, sFlow and other flows. Make informed decisions using capacity
planning reports. https://ad.doubleclick.net/ddm/clk/305295220;132659582;e
_______________________________________________
Opensaf-devel mailing list
Opensaf-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/opensaf-devel
