php-git icon indicating copy to clipboard operation
php-git copied to clipboard

Crash in git_transport_register

Open MagicalTux opened this issue 10 years ago • 0 comments

git_transport_register will register a callback, then free said callback before returning, resulting in a crash when said callback is to be used.

This patch removes the call to php_git2_cb_free(), and also implements a new helper php_git2_cb_init_copy() that will duplicate fci/fcc (to be freed in php_git2_cb_free if needed).

Ideally we should also free the callback in the unregister method - but considering most transports are likely to survive until the end of execution of the script that might have a lower priority.

MagicalTux avatar Jun 25 '15 09:06 MagicalTux