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
commit "utf8.h, et.al.: Clean up some casts" causes "../t/re/pat_advanced.t .. panic: my_snprintf buffer overflow" #15156
Comments
From @bulk88Created by @bulk88Tried a DEBUGGING build, first DEBUGGING smoke in a while, with VC 2003, bisected to I've made 2 branches to show the last good commit and the first bad -------------------------------------------- Test Summary Report C:\perl521\src\win32> buffer after vsnprintf fills it and fails/truncates With this call stack _vsnprintf returns -1 eventually causing the panic my @f = ( Perl Info
|
From @tonycozOn Fri Jan 29 10:40:41 2016, bulk88 wrote:
This looks like a bug in the patch for #126045. The newer macros add the casts so the callers don't need them, but the macros used on older MSVC don't include the cast. Another issue is the test I added triggers when _MSC_VER isn't defined too, so non-MSVC builds are failing with the same error. I'll push a fix for it soon. Tony |
The RT System itself - Status changed from 'new' to 'open' |
@tonycoz - Status changed from 'open' to 'resolved' |
Migrated from rt.perl.org#127426 (status was 'resolved')
Searchable as RT127426$
The text was updated successfully, but these errors were encountered: