From: Ben Gras <ben@minix3.org>
Date: Fri, 17 Apr 2009 13:46:37 +0000 (+0000)
Subject:  - enable ipc warnings by default
X-Git-Tag: v3.1.4~76
X-Git-Url: http://zhaoyanbai.com/repos/man.named-journalprint.html?a=commitdiff_plain;h=e0f3a5acf1b5b6582bfa83d690578a5bcdbd7274;p=minix.git

 - enable ipc warnings by default
 - ipc checking code in kernel didn't properly catch the
   sendrec() to self case; added special case check
 - triggered by PM using stock panic() - needs its own _exit()

reported by Joren l'Ami.
---

diff --git a/kernel/debug.h b/kernel/debug.h
index 87c674ca2..034a26e82 100644
--- a/kernel/debug.h
+++ b/kernel/debug.h
@@ -19,7 +19,7 @@
  * Of course the call still fails, but nothing is printed if these warnings
  * are disabled.
  */
-#define DEBUG_ENABLE_IPC_WARNINGS	0
+#define DEBUG_ENABLE_IPC_WARNINGS	1
 #define DEBUG_STACKTRACE		1
 #define DEBUG_VMASSERT			1
 #define DEBUG_SCHED_CHECK		1
diff --git a/kernel/proc.c b/kernel/proc.c
index 11960020a..0b5f43c36 100755
--- a/kernel/proc.c
+++ b/kernel/proc.c
@@ -201,6 +201,13 @@ long bit_map;			/* notification event set or flags */
   }
   else
   {
+	if(caller_ptr->p_endpoint == src_dst_e) {
+#if DEBUG_ENABLE_IPC_WARNINGS
+		kprintf("sys_call: trap %d by %d with self %d\n", 
+			call_nr, proc_nr(caller_ptr), src_dst_e);
+#endif
+		return EINVAL;
+	}
 	/* Require a valid source and/or destination process. */
 	if(!isokendpt(src_dst_e, &src_dst_p)) {
 #if DEBUG_ENABLE_IPC_WARNINGS
diff --git a/servers/pm/forkexit.c b/servers/pm/forkexit.c
index 63656ad6b..4bfe4ad30 100644
--- a/servers/pm/forkexit.c
+++ b/servers/pm/forkexit.c
@@ -417,3 +417,13 @@ register struct mproc *rmp;	/* tells which process is exiting */
   rmp->mp_child_stime = 0;
   procs_in_use--;
 }
+
+PUBLIC void _exit(int code)
+{
+	sys_exit(SELF);
+}
+
+PUBLIC void __exit(int code)
+{
+	sys_exit(SELF);
+}