Skip to content

Conversation

petk
Copy link
Member

@petk petk commented Sep 29, 2017

This patch fixed bug #75279 and it's for PHP-7.2 and master branches. Related to 4fbff82

@@ -110,7 +110,7 @@ PHPAPI int php_load_extension(char *filename, int type, int start_now)
char *libpath;
zend_module_entry *module_entry;
zend_module_entry *(*get_module)(void);
int error_type, slash_suffix = NULL;
int error_type, slash_suffix;
Copy link
Member

Choose a reason for hiding this comment

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

I think this change will cause line 154 may use of uninitialized value, should be change to slash_suffix=0

Copy link
Member Author

Choose a reason for hiding this comment

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

Done. Then it should be added to master branch also (I've updated above message to target both branches. Thanks.

@petk petk force-pushed the patch-compile-warning branch from 751716f to 8d87f6f Compare September 29, 2017 12:34
@php-pulls php-pulls merged commit 8d87f6f into php:PHP-7.2 Sep 30, 2017
@petk petk deleted the patch-compile-warning branch September 30, 2017 06:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants