-
Notifications
You must be signed in to change notification settings - Fork 8k
Fix #78987 - Memory problems running finfo::buffer #7188
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
|
Duplicate of #7187 |
|
Could you please fix the failing test? |
|
I’ll look into it |
|
Any news on this? |
|
Sorry, I didn’t forget but it was a busy period. I’ll work on it in the next days. |
|
@weltling, what do you think? |
|
@cmb69 the submitted patch doesn't seem appropriate for 7.4, a test is missing and there's a lot of unrelated white space change. @davisinfo appreciate your efforts to fix this. May i suggest this:
I might have time to fix this closer to next week or alike, there still seems to be room for this change to get into 7.4. Thanks |
|
Backported the upstream solution in fa96c7e. Thanks. |
|
Is this fix in 7.4.24 or not yet in a public 7.4.x release? |
|
To answer my own question: looks like it's in the 7.4.25 release notes! Thanks! |
Fix #78987 - Memory problems running finfo::buffer