d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Correct pointer subtraction in KASSERT().

Browse Source 
The assertion would never fire without truly spectacular future
programming errors.

Reported by:	Coverity
CID:		1391367, 1391368
Sponsored by:	DARPA, AFRL
This commit is contained in:
Brooks Davis 2018-05-29 17:49:03 +00:00
parent c0fd148b5d
commit d8b2f0790b
1 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
sys/kern/imgact_elf.c
View File

@ -1145,8 +1145,7 @@ __elfN(freebsd_fixup)(register_t **stack_base, struct image_params *imgp)
free(imgp->auxargs, M_TEMP); free(imgp->auxargs, M_TEMP);
imgp->auxargs = NULL; imgp->auxargs = NULL;
KASSERT((pos - argarray) / sizeof(*pos) <= AT_COUNT, KASSERT(pos - argarray <= AT_COUNT, ("Too many auxargs"));
("Too many auxargs"));
error = copyout(argarray, auxbase, sizeof(*argarray) * AT_COUNT); error = copyout(argarray, auxbase, sizeof(*argarray) * AT_COUNT);
free(argarray, M_TEMP); free(argarray, M_TEMP);