Re: [PATCH] libselinux: selinux_status_open: return 1 in fallback mode

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

 



Petr Lautrbach <plautrba@xxxxxxxxxx> writes:

> Christian Göttsche <cgzones@xxxxxxxxxxxxxx> writes:
>
>> In case of a recurring call to `selinux_status_open(3)`, which
>> previously has been opened in fallback mode, return `1` according to its
>> documentation.
>>
>> Fixes: c5a699046f4 ("libselinux: make selinux_status_open(3) reentrant")
>>
>> Signed-off-by: Christian Göttsche <cgzones@xxxxxxxxxxxxxx>
>>
>
> Acked-by: Petr Lautrbach <plautrba@xxxxxxxxxx>

This is merged now. Thanks!


>
>> ---
>>  libselinux/src/sestatus.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/libselinux/src/sestatus.c b/libselinux/src/sestatus.c
>> index 531a522c..89c1f621 100644
>> --- a/libselinux/src/sestatus.c
>> +++ b/libselinux/src/sestatus.c
>> @@ -283,7 +283,7 @@ int selinux_status_open(int fallback)
>>  	uint32_t	seqno;
>>  
>>  	if (selinux_status != NULL) {
>> -		return 0;
>> +		return (selinux_status == MAP_FAILED) ? 1 : 0;
>>  	}
>>  
>>  	if (!selinux_mnt) {
>> -- 
>> 2.32.0.rc2





[Index of Archives]     [Selinux Refpolicy]     [Linux SGX]     [Fedora Users]     [Fedora Desktop]     [Yosemite Photos]     [Yosemite Camping]     [Yosemite Campsites]     [KDE Users]     [Gnome Users]

  Powered by Linux