On 10/02/15 14:31, Lester Caine wrote: > in interbase/ibase_blobs.c > >> zval *blob_arg, *string_arg; >> ibase_blob *ib_blob; >> >> RESET_ERRMSG; >> >> if (ZEND_NUM_ARGS() != 2 || zend_get_parameters_ex(2, &blob_arg, >> &string_arg) == FAILURE) { >> WRONG_PARAM_COUNT; >> } >> >> ZEND_FETCH_RESOURCE(ib_blob, ibase_blob *, blob_arg, -1, "Interbase >> blob", le_blob); > > I've got that if changes to > >> if (FAILURE == zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "rr", >> &blob_arg, &string_arg)) { >> return; >> } > > But I suspect I need to change the *string_arg to a character string so > 'rc' and add a string_len field. At least that is what I think I'm > seeing from the samples I have found.
OK I've got a patch for master, but I know that the changes are not complete! http://hg.lsces.org.uk/hg/php-src/rev/8ec9101f59b6 I've still got a question about the &string_arg on line 1.9 and if it should be changed to a string but that change needs working through the following code. Also is what I'm doing here php7 only? so should there be wrappers for a PHP5 build? I think that all this leaves is all the complaints about 'NULL' and integer initialization makes integer from pointer without a cast and assignment makes integer from pointer without a cast -- Lester Caine - G8HFL ----------------------------- Contact - http://lsces.co.uk/wiki/?page=contact L.S.Caine Electronic Services - http://lsces.co.uk EnquirySolve - http://enquirysolve.com/ Model Engineers Digital Workshop - http://medw.co.uk Rainbow Digital Media - http://rainbowdigitalmedia.co.uk -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php