CVSROOT: /cvs Module name: src Changes by: chel...@cvs.openbsd.org 2024/02/23 11:19:03
Modified files: sys/ddb : db_access.h sys/arch/alpha/alpha: db_interface.c sys/arch/amd64/amd64: db_memrw.c sys/arch/arm/arm: db_interface.c sys/arch/arm64/arm64: db_interface.c sys/arch/hppa/hppa: db_interface.c sys/arch/i386/i386: db_memrw.c sys/arch/m88k/m88k: db_interface.c sys/arch/mips64/mips64: db_machdep.c sys/arch/powerpc/ddb: db_memrw.c sys/arch/powerpc64/powerpc64: db_memrw.c sys/arch/riscv64/riscv64: db_interface.c sys/arch/sh/sh : db_memrw.c sys/arch/sparc64/sparc64: db_interface.c Log message: ddb(4): db_read_bytes/db_write_bytes: change dst/src parameter to void* Almost all db_read_bytes() callers cast the destination buffer argument to char*, which suggests the API's prototype is incompatible with how the API is actually used. Change db_read_bytes() and db_write_bytes() to take a void* as the destination/source buffer parameter so callers don't need to cast the argument. With input from bluhm@. Bugs caught by Clemens Gossnitzer (ASCII approximation of name). Thread: https://marc.info/?l=openbsd-tech&m=170740813021636&w=2 ok bluhm@