calling die or exit directly lead to segfault (#55)

* calling die or exit directly lead to segfault

* pre hook not needed

otel_observer.c

@@ -280,11 +280,13 @@ static void observer_end(zend_execute_data *execute_data, zval *retval,
         }
 
         if (UNEXPECTED(EG(exception))) {
+            // do not release params[3] if exit was called
+            if (exception && !zend_is_unwind_exit(exception)) {
+                OBJ_RELEASE(Z_OBJ(params[3]));
+            }
             if (exception) {
                 OBJ_RELEASE(exception);
-                OBJ_RELEASE(Z_OBJ(params[3]));
             }
-
             ZVAL_OBJ_COPY(&params[3], EG(exception));
         }
 

tests/calling_die_lead_to_segfault.phpt

@@ -0,0 +1,39 @@
+--TEST--
+Check if calling die or exit will finish gracefully
+--EXTENSIONS--
+opentelemetry
+--FILE--
+
+<?php
+
+use function OpenTelemetry\Instrumentation\hook;
+
+class TestClass {
+    public static function countFunction(): void
+    {
+       for ($i = 1; $i <= 300; $i++) {
+            if ($i === 200) {
+                die('exit!');
+            }
+       }
+    }
+}
+
+hook(
+    'TestClass',
+    'countFunction',
+    null,
+    post: static function ($object, array $params, mixed $ret, ?\Throwable $exception ) {}
+);
+
+try{
+TestClass::countFunction();
+}
+catch(Exception) {}
+// Comment out line below and revert fix in order to trigger segfault
+// reproduction frequency depends on platform
+catch(TypeError) {}
+?>
+
+--EXPECT--
+exit!