diff mbox series

[v3] hvf: Report HV_DENIED error

Message ID 20230608123014.28715-1-quic_acaggian@quicinc.com (mailing list archive)
State New, archived
Headers show
Series [v3] hvf: Report HV_DENIED error | expand

Commit Message

Antonio Caggiano June 8, 2023, 12:30 p.m. UTC
On MacOS 11 and subsequent versions, in case the resulting binary is not
signed with the proper entitlement, handle and report the HV_DENIED
error.

Signed-off-by: Antonio Caggiano <quic_acaggian@quicinc.com>
---
v2: Use architecture specific defines from AvailabilityMacros.h to enable the
    HV_DENIED case only on MacOS 11 and subsequent versions.
v3: Fix ifdef guard.

 accel/hvf/hvf-all.c | 6 ++++++
 1 file changed, 6 insertions(+)

Comments

Philippe Mathieu-Daudé June 13, 2023, 9:33 a.m. UTC | #1
On 8/6/23 14:30, Antonio Caggiano wrote:
> On MacOS 11 and subsequent versions, in case the resulting binary is not
> signed with the proper entitlement, handle and report the HV_DENIED
> error.
> 
> Signed-off-by: Antonio Caggiano <quic_acaggian@quicinc.com>
> ---
> v2: Use architecture specific defines from AvailabilityMacros.h to enable the
>      HV_DENIED case only on MacOS 11 and subsequent versions.
> v3: Fix ifdef guard.
> 
>   accel/hvf/hvf-all.c | 6 ++++++
>   1 file changed, 6 insertions(+)

Queued, thanks!
diff mbox series

Patch

diff --git a/accel/hvf/hvf-all.c b/accel/hvf/hvf-all.c
index 754707dbfb..4920787af6 100644
--- a/accel/hvf/hvf-all.c
+++ b/accel/hvf/hvf-all.c
@@ -38,6 +38,12 @@  void assert_hvf_ok(hv_return_t ret)
     case HV_UNSUPPORTED:
         error_report("Error: HV_UNSUPPORTED");
         break;
+#if defined(MAC_OS_VERSION_11_0) && \
+    MAC_OS_X_VERSION_MIN_REQUIRED >= MAC_OS_VERSION_11_0
+    case HV_DENIED:
+        error_report("Error: HV_DENIED");
+        break;
+#endif
     default:
         error_report("Unknown Error");
     }