Re: svn commit: r1887497 - in /apr/apr/branches/1.7.x: ./ CHANGES mmap/unix/mmap.c test/data/mmap_large_datafile.txt test/testfnmatch.c test/testmmap.c

2022-06-30 Thread William A Rowe Jr
IMNSO opinion, this seems like a somewhat safe 1.8 change. What you want to ask is how someone who "discovers" apr.so.1 (.8) would react with 1.7 code, and if that's clean and using only public API's, it makes sense to me. That's not the case for other changes, but for this one it shouldn't shock

Re: svn commit: r1887497 - in /apr/apr/branches/1.7.x: ./ CHANGES mmap/unix/mmap.c test/data/mmap_large_datafile.txt test/testfnmatch.c test/testmmap.c

2022-06-30 Thread Yann Ylavic
On Wed, Jun 29, 2022 at 4:49 PM William A Rowe Jr wrote: > > The eventual change still is present for apr > 1.8 adopters. > > Thoughts? ylavic are you willing to revert? It seems that no one is using apr_mmap_create() with an offset != 0 or the bug would have been reported (PR 65158 proved to be

Re: svn commit: r1887497 - in /apr/apr/branches/1.7.x: ./ CHANGES mmap/unix/mmap.c test/data/mmap_large_datafile.txt test/testfnmatch.c test/testmmap.c

2022-06-29 Thread William A Rowe Jr
This change does introduce an API change, and a somewhat possibly dangerous structure size alteration, and seems to be out of bounds for apr 1.7.x branch, as identified in the commit message. Anyone using a build crossing runtimes between apr 1.7.0 and 1.7.1 may experience unintended