svn commit: r350158 - head/lib/libproc

Brooks Davis brooks at FreeBSD.org
Fri Jul 19 23:10:43 UTC 2019


Author: brooks
Date: Fri Jul 19 23:10:43 2019
New Revision: 350158
URL: https://svnweb.freebsd.org/changeset/base/350158

Log:
  Remove an unneeded temporary variable in two functions.
  
  There is no need to convert an intptr_t to a long just to cast it to a
  (void *).
  
  Obtained from:	CheriBSD
  MFC after:	1 week
  Sponsored by:	DARPA, AFRL

Modified:
  head/lib/libproc/proc_bkpt.c

Modified: head/lib/libproc/proc_bkpt.c
==============================================================================
--- head/lib/libproc/proc_bkpt.c	Fri Jul 19 22:54:09 2019	(r350157)
+++ head/lib/libproc/proc_bkpt.c	Fri Jul 19 23:10:43 2019	(r350158)
@@ -102,7 +102,6 @@ proc_bkptset(struct proc_handle *phdl, uintptr_t addre
     unsigned long *saved)
 {
 	struct ptrace_io_desc piod;
-	unsigned long caddr;
 	int ret = 0, stopped;
 	instr_t instr;
 
@@ -125,10 +124,9 @@ proc_bkptset(struct proc_handle *phdl, uintptr_t addre
 	/*
 	 * Read the original instruction.
 	 */
-	caddr = address;
 	instr = 0;
 	piod.piod_op = PIOD_READ_I;
-	piod.piod_offs = (void *)caddr;
+	piod.piod_offs = (void *)address;
 	piod.piod_addr = &instr;
 	piod.piod_len  = BREAKPOINT_INSTR_SZ;
 	if (ptrace(PT_IO, proc_getpid(phdl), (caddr_t)&piod, 0) < 0) {
@@ -141,10 +139,9 @@ proc_bkptset(struct proc_handle *phdl, uintptr_t addre
 	/*
 	 * Write a breakpoint instruction to that address.
 	 */
-	caddr = address;
 	instr = BREAKPOINT_INSTR;
 	piod.piod_op = PIOD_WRITE_I;
-	piod.piod_offs = (void *)caddr;
+	piod.piod_offs = (void *)address;
 	piod.piod_addr = &instr;
 	piod.piod_len  = BREAKPOINT_INSTR_SZ;
 	if (ptrace(PT_IO, proc_getpid(phdl), (caddr_t)&piod, 0) < 0) {
@@ -167,7 +164,6 @@ proc_bkptdel(struct proc_handle *phdl, uintptr_t addre
     unsigned long saved)
 {
 	struct ptrace_io_desc piod;
-	unsigned long caddr;
 	int ret = 0, stopped;
 	instr_t instr;
 
@@ -189,10 +185,9 @@ proc_bkptdel(struct proc_handle *phdl, uintptr_t addre
 	/*
 	 * Overwrite the breakpoint instruction that we setup previously.
 	 */
-	caddr = address;
 	instr = saved;
 	piod.piod_op = PIOD_WRITE_I;
-	piod.piod_offs = (void *)caddr;
+	piod.piod_offs = (void *)address;
 	piod.piod_addr = &instr;
 	piod.piod_len  = BREAKPOINT_INSTR_SZ;
 	if (ptrace(PT_IO, proc_getpid(phdl), (caddr_t)&piod, 0) < 0) {


More information about the svn-src-head mailing list