Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/sys/compat/freebsd



Hm, looks like this can skate by because compat_freebsd isn't supported on amd64?

> On Apr 6, 2019, at 10:30 AM, Jason Thorpe <thorpej%me.com@localhost> wrote:
> 
> 
> 
>> On Apr 6, 2019, at 10:27 AM, Robert Elz <kre%netbsd.org@localhost> wrote:
>> 
>> Module Name:	src
>> Committed By:	kre
>> Date:		Sat Apr  6 17:27:59 UTC 2019
>> 
>> Modified Files:
>> 	src/sys/compat/freebsd: freebsd_syscall.c
>> 
>> Log Message:
>> Revert previous.   Instead change ufetch_long() to ufetch_int() in
>> both calls (and also remove the cast on the earlier).
>> 
>> These replaced fuword, to fetch a register_t (ie: int).
> 
> This is wrong -- register_t is 64-bit on amd64 ... so u_long is the better choice of cast.
> 
>> 
>> 
>> To generate a diff of this commit:
>> cvs rdiff -u -r1.4 -r1.5 src/sys/compat/freebsd/freebsd_syscall.c
>> 
>> Please note that diffs are not public domain; they are subject to the
>> copyright notices on the relevant files.
>> 
> 
> -- thorpej
> 

-- thorpej



Home | Main Index | Thread Index | Old Index