Merge pull request #138 from sj-i/0.3.x

0.3.x to 0.4.x

composer.json

@@ -38,7 +38,7 @@
     "squizlabs/php_codesniffer": "3.6.2",
     "mockery/mockery": "1.4.4",
     "psalm/plugin-mockery": "0.9.1",
-    "jetbrains/phpstorm-stubs": "2021.2",
+    "jetbrains/phpstorm-stubs": "2021.3",
     "php-coveralls/php-coveralls": "2.5.2"
   },
   "autoload": {

src/Lib/FFI/CannotAllocateBufferException.php

@@ -0,0 +1,18 @@
+<?php
+
+/**
+ * This file is part of the sj-i/php-profiler package.
+ *
+ * (c) sji <sji@sj-i.dev>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+declare(strict_types=1);
+
+namespace PhpProfiler\Lib\FFI;
+
+final class CannotAllocateBufferException extends \Exception
+{
+}

src/Lib/FFI/CannotCastCDataException.php

@@ -0,0 +1,18 @@
+<?php
+
+/**
+ * This file is part of the sj-i/php-profiler package.
+ *
+ * (c) sji <sji@sj-i.dev>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+declare(strict_types=1);
+
+namespace PhpProfiler\Lib\FFI;
+
+final class CannotCastCDataException extends \Exception
+{
+}

src/Lib/FFI/CannotGetTypeForCDataException.php

@@ -0,0 +1,28 @@
+<?php
+
+/**
+ * This file is part of the sj-i/php-profiler package.
+ *
+ * (c) sji <sji@sj-i.dev>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+declare(strict_types=1);
+
+namespace PhpProfiler\Lib\FFI;
+
+use Throwable;
+
+final class CannotGetTypeForCDataException extends \Exception
+{
+    public function __construct(
+        string $message = "",
+        int $code = 0,
+        ?Throwable $previous = null,
+        public string $type = '',
+    ) {
+        parent::__construct($message, $code, $previous);
+    }
+}

src/Lib/FFI/CannotLoadCHeaderException.php

@@ -0,0 +1,18 @@
+<?php
+
+/**
+ * This file is part of the sj-i/php-profiler package.
+ *
+ * (c) sji <sji@sj-i.dev>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+declare(strict_types=1);
+
+namespace PhpProfiler\Lib\FFI;
+
+final class CannotLoadCHeaderException extends \Exception
+{
+}

src/Lib/File/NativeFileReader.php

@@ -14,6 +14,7 @@
 namespace PhpProfiler\Lib\File;
 
 use FFI;
+use PhpProfiler\Lib\FFI\CannotAllocateBufferException;
 
 class NativeFileReader implements FileReaderInterface
 {
@@ -32,7 +33,8 @@ public function __construct()
 
     public function readAll(string $path): string
     {
-        $buffer = $this->ffi->new("unsigned char[4096]");
+        $buffer = $this->ffi->new("unsigned char[4096]")
+            ?? throw new CannotAllocateBufferException('cannot allocate buffer');
 
         $fd = $this->ffi->open($path, 0);
         $result = "";

src/Lib/Libc/Sys/Ptrace/PtraceX64.php

@@ -15,6 +15,8 @@
 
 use FFI\CData;
 use FFI\CInteger;
+use PhpProfiler\Lib\FFI\CannotAllocateBufferException;
+use PhpProfiler\Lib\FFI\CannotCastCDataException;
 use PhpProfiler\Lib\Libc\Addressable;
 
 class PtraceX64 implements Ptrace
@@ -95,15 +97,19 @@ public function ptrace(
     ): int {
         if (is_null($addr) or is_int($addr)) {
             /** @var CInteger */
-            $addr_holder = \FFI::new('long');
+            $addr_holder = \FFI::new('long')
+                ?? throw new CannotAllocateBufferException('cannot allocate buffer');
             $addr_holder->cdata = (int)$addr;
-            $addr = \FFI::cast('void *', $addr_holder);
+            $addr = \FFI::cast('void *', $addr_holder)
+                ?? throw new CannotCastCDataException('cannot cast buffer');
         }
         if (is_null($data) or is_int($data)) {
             /** @var CInteger */
-            $data_holder = \FFI::new('long');
+            $data_holder = \FFI::new('long')
+                ?? throw new CannotAllocateBufferException('cannot allocate buffer');
             $data_holder->cdata = (int)$data;
-            $data = \FFI::cast('void *', $data_holder);
+            $data = \FFI::cast('void *', $data_holder)
+                ?? throw new CannotCastCDataException('cannot cast buffer');
         }
 
         $addr_pointer = $addr instanceof Addressable ? $addr->toVoidPointer() : $addr;

src/Lib/PhpInternals/ZendTypeReader.php

@@ -15,6 +15,8 @@
 
 use FFI;
 use FFI\CData;
+use PhpProfiler\Lib\FFI\CannotGetTypeForCDataException;
+use PhpProfiler\Lib\FFI\CannotLoadCHeaderException;
 
 final class ZendTypeReader
 {
@@ -52,7 +54,8 @@ private function loadHeader(string $php_version): FFI
             if ($php_version === self::V81) {
                 $php_version = self::V80;
             }
-            $this->ffi = FFI::load(__DIR__ . "/Headers/{$php_version}.h");
+            $this->ffi = FFI::load(__DIR__ . "/Headers/{$php_version}.h")
+                ?? throw new CannotLoadCHeaderException('cannot load headers for zend engine');
         }
         return $this->ffi;
     }
@@ -66,7 +69,11 @@ public function readAs(string $type, CData $cdata): ZendTypeCData
     public function sizeOf(string $type): int
     {
         $ffi = $this->loadHeader($this->php_version);
-        $type = $ffi->type($type);
-        return FFI::sizeof($type);
+        $cdata_type = $ffi->type($type)
+            ?? throw new CannotGetTypeForCDataException(
+                message: 'cannot get type for a C Data',
+                type: $type
+            );
+        return FFI::sizeof($cdata_type);
     }
 }

src/Lib/Process/MemoryReader/MemoryReader.php

@@ -15,6 +15,7 @@
 
 use FFI;
 use FFI\CData;
+use PhpProfiler\Lib\FFI\CannotAllocateBufferException;
 
 final class MemoryReader implements MemoryReaderInterface
 {
@@ -39,9 +40,12 @@ public function __construct()
                          unsigned long riovcnt,
                          unsigned long flags);
        ', 'libc.so.6');
-        $this->local_iov = $this->ffi->new('struct iovec');
-        $this->remote_iov = $this->ffi->new('struct iovec');
-        $this->remote_base = $this->ffi->new('long');
+        $this->local_iov = $this->ffi->new('struct iovec')
+            ?? throw new CannotAllocateBufferException('cannot allocate buffer');
+        $this->remote_iov = $this->ffi->new('struct iovec')
+            ?? throw new CannotAllocateBufferException('cannot allocate buffer');
+        $this->remote_base = $this->ffi->new('long')
+            ?? throw new CannotAllocateBufferException('cannot allocate buffer');
     }
 
     /**
@@ -50,7 +54,8 @@ public function __construct()
      */
     public function read(int $pid, int $remote_address, int $size): CData
     {
-        $buffer = $this->ffi->new("unsigned char[{$size}]");
+        $buffer = $this->ffi->new("unsigned char[{$size}]")
+            ?? throw new CannotAllocateBufferException('cannot allocate buffer');
 
         /**
          * @var FFI\Libc\iovec $this->local_iov