arshidkv12 · GitHub

Girgias

Comment on lines +1449 to +1452

if (!*dest) {
zend_argument_must_not_be_empty_error(2);
return FAILURE;
}

Copy link Copy Markdown

Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Again this is not the call site.


/* {{{ php_copy_file_ctx */
PHPAPI zend_result php_copy_file_ctx(const char *src, const char *dest, int src_flags, php_stream_context *ctx)
ZEND_ATTRIBUTE_NONNULL_ARGS(1,2) PHPAPI zend_result php_copy_file_ctx(const char *src, const char *dest, int src_flags, php_stream_context *ctx)

Copy link Copy Markdown

Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The attribute should be back propagated to php_copy_file_ex and php_copy_file

Read the original on github.com ↗