|
|
php.net | support | documentation | report a bug | advanced search | search howto | statistics | random bug | login |
[2020-01-27 18:05 UTC] lisachenko dot it at gmail dot com
Description: ------------ If we define a C callback handler in PHP and this handler should return a pointer (not a simple value like int, float, etc), then FFI doesn't check if correct value was returned or not. For example when we throw our custom exception within callback body. This typically results in exception from FFI: Incompatible types when assigning to type 'struct some_native_type*' from PHP 'unknown' Exception is thrown because there is no check for Z_TYPE(retval) != IS_UNDEF inside FFI's function zend_ffi_callback_trampoline before call to zend_ffi_zval_to_cdata(ret, ret_type, &retval) Expected result: ---------------- Exception is thrown within callback and not from FFI itself about missing type conversion Actual result: -------------- Incompatible types when assigning to type 'struct some_native_type*' from PHP 'unknown' PatchesPull Requests
Pull requests:
HistoryAllCommentsChangesGit/SVN commits
|
|||||||||||||||||||||||||||
Copyright © 2001-2024 The PHP GroupAll rights reserved. |
Last updated: Thu Nov 21 15:01:30 2024 UTC |
Test example to reproduce this case: use FFI\CData; $php = FFI::cdef(" typedef unsigned int fake_struct; typedef fake_struct* (*zend_write_func_t)(const char *str, size_t str_length); extern zend_write_func_t zend_write; "); echo "Before", PHP_EOL; $originalHandler = clone $php->zend_write; $php->zend_write = function($str, $len): CData { throw new \RuntimeException('Not allowed'); }; try { echo "After", PHP_EOL; } catch (\Throwable $exception) { // Do not output anything here, as handler is overridden } finally { $php->zend_write = $originalHandler; } echo get_class($exception), ': ', $exception->getMessage(), PHP_EOL; // Output: // Before // FFI\Exception: Incompatible types when assigning to type 'uint32_t*' from PHP 'unknown' // Expected: // Before // RuntimeException: Not allowed