* [PATCH] python/sepolicy: Fix get_os_version except clause
@ 2023-06-19 6:32 Jeffery To
2023-06-29 15:07 ` Petr Lautrbach
0 siblings, 1 reply; 3+ messages in thread
From: Jeffery To @ 2023-06-19 6:32 UTC (permalink / raw)
To: selinux
This adds more exceptions to be handled by the except clause in
`get_os_version()`:
* If the `distro` package is not installed, then `import distro` raises
a `ModuleNotFoundError` exception.
* The distro documentation[1] lists `OSError` and `UnicodeError` as
exceptions that can be raised.
* Older versions of distro (<= 1.6.0) may also raise
`subprocessCalledProcessError`[2].
[1]: https://github.com/python-distro/distro/blob/v1.8.0/src/distro/distro.py#L749-L753
[2]: https://github.com/python-distro/distro/blob/v1.6.0/distro.py#L726-L728
Signed-off-by: Jeffery To <jeffery.to@gmail.com>
---
python/sepolicy/sepolicy/__init__.py | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/python/sepolicy/sepolicy/__init__.py b/python/sepolicy/sepolicy/__init__.py
index c177cdfc529d..2d526c94a0c5 100644
--- a/python/sepolicy/sepolicy/__init__.py
+++ b/python/sepolicy/sepolicy/__init__.py
@@ -1240,11 +1240,12 @@ def boolean_desc(boolean):
def get_os_version():
+ import subprocess
system_release = ""
try:
import distro
system_release = distro.name(pretty=True)
- except IOError:
+ except (ModuleNotFoundError, OSError, IOError, UnicodeError, subprocess.CalledProcessError):
system_release = "Misc"
return system_release
--
2.39.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] python/sepolicy: Fix get_os_version except clause
2023-06-19 6:32 [PATCH] python/sepolicy: Fix get_os_version except clause Jeffery To
@ 2023-06-29 15:07 ` Petr Lautrbach
2023-06-30 12:29 ` Petr Lautrbach
0 siblings, 1 reply; 3+ messages in thread
From: Petr Lautrbach @ 2023-06-29 15:07 UTC (permalink / raw)
To: Jeffery To, selinux
Jeffery To <jeffery.to@gmail.com> writes:
> This adds more exceptions to be handled by the except clause in
> `get_os_version()`:
>
> * If the `distro` package is not installed, then `import distro` raises
> a `ModuleNotFoundError` exception.
>
> * The distro documentation[1] lists `OSError` and `UnicodeError` as
> exceptions that can be raised.
>
> * Older versions of distro (<= 1.6.0) may also raise
> `subprocessCalledProcessError`[2].
>
> [1]: https://github.com/python-distro/distro/blob/v1.8.0/src/distro/distro.py#L749-L753
> [2]: https://github.com/python-distro/distro/blob/v1.6.0/distro.py#L726-L728
>
> Signed-off-by: Jeffery To <jeffery.to@gmail.com>
Acked-by: Petr Lautrbach <lautrbach@redhat.com>
> ---
> python/sepolicy/sepolicy/__init__.py | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/python/sepolicy/sepolicy/__init__.py b/python/sepolicy/sepolicy/__init__.py
> index c177cdfc529d..2d526c94a0c5 100644
> --- a/python/sepolicy/sepolicy/__init__.py
> +++ b/python/sepolicy/sepolicy/__init__.py
> @@ -1240,11 +1240,12 @@ def boolean_desc(boolean):
>
>
> def get_os_version():
> + import subprocess
> system_release = ""
> try:
> import distro
> system_release = distro.name(pretty=True)
> - except IOError:
> + except (ModuleNotFoundError, OSError, IOError, UnicodeError, subprocess.CalledProcessError):
> system_release = "Misc"
>
> return system_release
> --
> 2.39.2
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] python/sepolicy: Fix get_os_version except clause
2023-06-29 15:07 ` Petr Lautrbach
@ 2023-06-30 12:29 ` Petr Lautrbach
0 siblings, 0 replies; 3+ messages in thread
From: Petr Lautrbach @ 2023-06-30 12:29 UTC (permalink / raw)
To: Jeffery To, selinux
Petr Lautrbach <lautrbach@redhat.com> writes:
> Jeffery To <jeffery.to@gmail.com> writes:
>
>> This adds more exceptions to be handled by the except clause in
>> `get_os_version()`:
>>
>> * If the `distro` package is not installed, then `import distro` raises
>> a `ModuleNotFoundError` exception.
>>
>> * The distro documentation[1] lists `OSError` and `UnicodeError` as
>> exceptions that can be raised.
>>
>> * Older versions of distro (<= 1.6.0) may also raise
>> `subprocessCalledProcessError`[2].
>>
>> [1]: https://github.com/python-distro/distro/blob/v1.8.0/src/distro/distro.py#L749-L753
>> [2]: https://github.com/python-distro/distro/blob/v1.6.0/distro.py#L726-L728
>>
>> Signed-off-by: Jeffery To <jeffery.to@gmail.com>
>
> Acked-by: Petr Lautrbach <lautrbach@redhat.com>
merged, thanks!
>> ---
>> python/sepolicy/sepolicy/__init__.py | 3 ++-
>> 1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/python/sepolicy/sepolicy/__init__.py b/python/sepolicy/sepolicy/__init__.py
>> index c177cdfc529d..2d526c94a0c5 100644
>> --- a/python/sepolicy/sepolicy/__init__.py
>> +++ b/python/sepolicy/sepolicy/__init__.py
>> @@ -1240,11 +1240,12 @@ def boolean_desc(boolean):
>>
>>
>> def get_os_version():
>> + import subprocess
>> system_release = ""
>> try:
>> import distro
>> system_release = distro.name(pretty=True)
>> - except IOError:
>> + except (ModuleNotFoundError, OSError, IOError, UnicodeError, subprocess.CalledProcessError):
>> system_release = "Misc"
>>
>> return system_release
>> --
>> 2.39.2
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-06-30 12:30 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-06-19 6:32 [PATCH] python/sepolicy: Fix get_os_version except clause Jeffery To
2023-06-29 15:07 ` Petr Lautrbach
2023-06-30 12:29 ` Petr Lautrbach
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).