diff options
author | Viktor Szakats <commit@vsz.me> | 2023-02-12 12:52:59 +0000 |
---|---|---|
committer | Viktor Szakats <commit@vsz.me> | 2023-02-12 12:52:59 +0000 |
commit | 5309e32141a144187dcdc97e61cf8f39c4288dea (patch) | |
tree | c021224312bd3088c12150e1dfe90b520a7b8b77 /docs/KNOWN_BUGS | |
parent | 82348603e8d134c334acd8becfeb2865d5e139c2 (diff) | |
download | curl-5309e32141a144187dcdc97e61cf8f39c4288dea.tar.gz |
windows: always use curl's basename() implementation
The `basename()` [1][2] implementation provided by mingw-w64 [3] makes
assumptions about input encoding and may break with non-ASCII strings.
`basename()` was auto-detected with CMake, autotools and since
68fa9bf3f5d7b4fcbb57619f70cb4aabb79a51f6 (2022-10-13), also in
`Makefile.mk` after syncing its behaviour with the mainline build
methods. A similar patch for curl-for-win broke official Windows
builds earlier, in release 7.83.1_4 (2022-06-15).
This patch forces all Windows builds to use curl's internal
`basename()` implementation to avoid such problems.
[1]: https://pubs.opengroup.org/onlinepubs/9699919799/utilities/basename.html
[2]: https://www.man7.org/linux/man-pages/man3/basename.3.html
[3]: https://sourceforge.net/p/mingw-w64/mingw-w64/ci/master/tree/mingw-w64-crt/misc/basename.c
Reported-by: UnicornZhang on Github
Assisted-by: Cherish98 on Github
Reviewed-by: Daniel Stenberg
Fixes #10261
Closes #10475
Diffstat (limited to 'docs/KNOWN_BUGS')
0 files changed, 0 insertions, 0 deletions