-
Notifications
You must be signed in to change notification settings - Fork 7.9k
PowerPC64 support in safe_address function #734
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Hmm, I don't have a PowerPC64, someone can review this? or should we just commit it? |
"r"(offset)); | ||
|
||
if (overflow) { | ||
printf("Possible integer overflow in memory allocation (%zu * %zu + %zu)\n", nmemb, size, offset); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please use one of the zend_error_
functions here.
From a quick test build (5.6.0RC4 on RHEL-7 ppc64), those PR (734-737) increase number of failed tests (from 6 to 62) diff of "FAILED TEST SUMMARY":
|
@vitordelima could you please fix the problems? |
Sure, I will take a look at it. |
Add a ppc64-specific implementation of the safe_address function with overflow checking.
looks like this needs a rebase |
Add a ppc64-specific implementation of the safe_address function with overflow
checking.