Re: A PRIV_* flag for /dev/mem?

From: Jamie Gritton <jamie_at_FreeBSD.org>
Date: Sun, 16 Jun 2013 09:21:08 -0600
On 06/16/13 00:20, Konstantin Belousov wrote:
> On Sat, Jun 15, 2013 at 05:23:50PM -0600, Jamie Gritton wrote:
>> Index: sys/dev/mem/memdev.c
>> ===================================================================
>> --- sys/dev/mem/memdev.c	(revision 251793)
>> +++ sys/dev/mem/memdev.c	(working copy)
>> _at__at_ -67,8 +67,14 _at__at_
>>   {
>>   	int error = 0;
>>
>> -	if (flags&  FWRITE)
>> -		error = securelevel_gt(td->td_ucred, 0);
>> +	if (flags&  FREAD)
>> +		error = priv_check(td, PRIV_KMEM_READ);
>> +	if (flags&  FWRITE) {
>> +		if (error != 0)
>> +			error = priv_check(td, PRIV_KMEM_WRITE);
>> +		if (error != 0)
> Shouldn't this be 'if (error == 0)' ?

Indeed it should. None of this has even been compiled yet, let alone
tested - still in the thought phase.

- Jamie
Received on Sun Jun 16 2013 - 13:21:12 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:40:38 UTC