From 01848872763ab231108419e9e0ece7bdcd252e34 Mon Sep 17 00:00:00 2001
From: Leo Custodio <73857187+lcustodio-krg@users.noreply.github.com>
Date: Fri, 5 Feb 2021 20:10:26 -0600
Subject: [PATCH] Removing redundant logic and correcting debug directive

Two debug directives were disabled in what appears to be a mistake - updated code to correct entries.
Removed logic that was always true based on switch case.
---
 minix/kernel/proc.c | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/minix/kernel/proc.c b/minix/kernel/proc.c
index 05af25f95..19783a8d1 100644
--- a/minix/kernel/proc.c
+++ b/minix/kernel/proc.c
@@ -506,7 +506,7 @@ static int do_sync_ipc(struct proc * caller_ptr, /* who made the call */
   {
 	if (call_nr != RECEIVE)
 	{
-#if 0
+#if DEBUG_ENABLE_IPC_WARNINGS
 		printf("sys_call: %s by %d with bad endpoint %d\n", 
 			callname,
 			proc_nr(caller_ptr), src_dst_e);
@@ -519,7 +519,7 @@ static int do_sync_ipc(struct proc * caller_ptr, /* who made the call */
   {
 	/* Require a valid source and/or destination process. */
 	if(!isokendpt(src_dst_e, &src_dst_p)) {
-#if 0
+#if DEBUG_ENABLE_IPC_WARNINGS
 		printf("sys_call: %s by %d with bad endpoint %d\n", 
 			callname,
 			proc_nr(caller_ptr), src_dst_e);
@@ -576,10 +576,8 @@ static int do_sync_ipc(struct proc * caller_ptr, /* who made the call */
 		break;				/* done, or SEND failed */
 	/* fall through for SENDREC */
   case RECEIVE:			
-	if (call_nr == RECEIVE) {
-		caller_ptr->p_misc_flags &= ~MF_REPLY_PEND;
-		IPC_STATUS_CLEAR(caller_ptr);  /* clear IPC status code */
-	}
+	caller_ptr->p_misc_flags &= ~MF_REPLY_PEND;
+	IPC_STATUS_CLEAR(caller_ptr);  /* clear IPC status code */
 	result = mini_receive(caller_ptr, src_dst_e, m_ptr, 0);
 	break;
   case NOTIFY: