René's URL Explorer Experiment


Title: bpo-31940: Reject faulty lchown implementations by asottile · Pull Request #4267 · python/cpython · GitHub

Open Graph Title: bpo-31940: Reject faulty lchown implementations by asottile · Pull Request #4267 · python/cpython

X Title: bpo-31940: Reject faulty lchown implementations by asottile · Pull Request #4267 · python/cpython

Description: Verified by compiling with alpine using the following Dockerfile and test script: FROM alpine RUN apk update && \ apk add expat-dev libressl-dev zlib-dev ncurses-dev bzip2-dev xz-dev \ sqlite-dev libffi-dev tcl-dev linux-headers gdbm-dev readline-dev \ gcc nano bash git python3 alpine-sdk #!/usr/bin/env bash set -euxo pipefail docker build -t python-build . docker run \ --rm -ti \ -v "$PWD/cpython:/src:ro" \ python-build bash -exc '\ cp -r /src /tmp/src && \ cd /tmp/src && \ ./configure || (cat config.log && exit 1) && \ make -j8 && \ ./python -m test.test_shutil \ ' Before + ./python -m test.test_shutil ..................s.................Es.E.....EssEE.....EE..ss..........s.................Fs......s.s.. ====================================================================== ERROR: test_copy2_symlinks (__main__.TestShutil) ---------------------------------------------------------------------- Traceback (most recent call last): File "/tmp/src/Lib/test/test_shutil.py", line 531, in test_copy2_symlinks os.lchmod(src_link, stat.S_IRWXU | stat.S_IRWXO) OSError: [Errno 95] Not supported: '/tmp/tmpcfxc61cw/baz' ====================================================================== ERROR: test_copy_symlinks (__main__.TestShutil) ---------------------------------------------------------------------- Traceback (most recent call last): File "/tmp/src/Lib/test/test_shutil.py", line 508, in test_copy_symlinks os.lchmod(src_link, stat.S_IRWXU | stat.S_IRWXO) OSError: [Errno 95] Not supported: '/tmp/tmplde2crj7/baz' ====================================================================== ERROR: test_copymode_symlink_to_symlink (__main__.TestShutil) ---------------------------------------------------------------------- Traceback (most recent call last): File "/tmp/src/Lib/test/test_shutil.py", line 318, in test_copymode_symlink_to_symlink os.lchmod(src_link, stat.S_IRWXO|stat.S_IRWXG) OSError: [Errno 95] Not supported: '/tmp/tmpr3v0c24m/baz' ====================================================================== ERROR: test_copystat_symlinks (__main__.TestShutil) ---------------------------------------------------------------------- Traceback (most recent call last): File "/tmp/src/Lib/test/test_shutil.py", line 364, in test_copystat_symlinks os.lchmod(src_link, stat.S_IRWXO) OSError: [Errno 95] Not supported: '/tmp/tmplqh68wha/baz' ====================================================================== ERROR: test_copytree_dangling_symlinks (__main__.TestShutil) ---------------------------------------------------------------------- Traceback (most recent call last): File "/tmp/src/Lib/test/test_shutil.py", line 892, in test_copytree_dangling_symlinks shutil.copytree(src_dir, dst_dir, symlinks=True) File "/tmp/src/Lib/shutil.py", line 359, in copytree raise Error(errors) shutil.Error: [('/tmp/tmpj76xsejs/test.txt', '/tmp/tmpg6s1ihql/destination3/test.txt', "[Errno 95] Not supported: '/tmp/tmpg6s1ihql/destination3/test.txt'")] ====================================================================== ERROR: test_copytree_symlink_dir (__main__.TestShutil) ---------------------------------------------------------------------- Traceback (most recent call last): File "/tmp/src/Lib/test/test_shutil.py", line 911, in test_copytree_symlink_dir shutil.copytree(src_dir, dst_dir, symlinks=True) File "/tmp/src/Lib/shutil.py", line 359, in copytree raise Error(errors) shutil.Error: [('/tmp/tmpsa0vkix_/link_to_dir', '/tmp/tmpta37k_s9/destination2/link_to_dir', "[Errno 95] Not supported: '/tmp/tmpta37k_s9/destination2/link_to_dir'")] ====================================================================== ERROR: test_copytree_symlinks (__main__.TestShutil) ---------------------------------------------------------------------- Traceback (most recent call last): File "/tmp/src/Lib/test/test_shutil.py", line 651, in test_copytree_symlinks os.lchmod(src_link, stat.S_IRWXU | stat.S_IRWXO) OSError: [Errno 95] Not supported: '/tmp/tmphrkx18bj/src/sub/link' ====================================================================== FAIL: test_unzip_zipfile (__main__.TestShutil) ---------------------------------------------------------------------- Traceback (most recent call last): File "/tmp/src/Lib/test/test_shutil.py", line 1118, in test_unzip_zipfile subprocess.check_output(zip_cmd, stderr=subprocess.STDOUT) subprocess.CalledProcessError: Command '['unzip', '-t', '/tmp/tmppyfabhkq/archive.zip']' returned non-zero exit status 1. During handling of the above exception, another exception occurred: Traceback (most recent call last): File "/tmp/src/Lib/test/test_shutil.py", line 1122, in test_unzip_zipfile self.fail(msg.format(exc, details)) AssertionError: Command '['unzip', '-t', '/tmp/tmppyfabhkq/archive.zip']' returned non-zero exit status 1. **Unzip Output** unzip: unrecognized option: t BusyBox v1.26.2 (2017-10-04 13:37:41 GMT) multi-call binary. Usage: unzip [-lnopq] FILE[.zip] [FILE]... [-x FILE...] [-d DIR] Extract FILEs from ZIP archive -l List contents (with -q for short form) -n Never overwrite files (default: ask) -o Overwrite -p Print to stdout -q Quiet -x FILE Exclude FILEs -d DIR Extract into DIR ---------------------------------------------------------------------- Ran 102 tests in 0.289s FAILED (failures=1, errors=7, skipped=10) After + ./python -m test.test_shutil ..................s..................s.......s.s...........ss..........s.................Fs......s.s.. ====================================================================== FAIL: test_unzip_zipfile (__main__.TestShutil) ---------------------------------------------------------------------- Traceback (most recent call last): File "/tmp/src/Lib/test/test_shutil.py", line 1118, in test_unzip_zipfile subprocess.check_output(zip_cmd, stderr=subprocess.STDOUT) subprocess.CalledProcessError: Command '['unzip', '-t', '/tmp/tmp9zf_mymx/archive.zip']' returned non-zero exit status 1. During handling of the above exception, another exception occurred: Traceback (most recent call last): File "/tmp/src/Lib/test/test_shutil.py", line 1122, in test_unzip_zipfile self.fail(msg.format(exc, details)) AssertionError: Command '['unzip', '-t', '/tmp/tmp9zf_mymx/archive.zip']' returned non-zero exit status 1. **Unzip Output** unzip: unrecognized option: t BusyBox v1.26.2 (2017-10-04 13:37:41 GMT) multi-call binary. Usage: unzip [-lnopq] FILE[.zip] [FILE]... [-x FILE...] [-d DIR] Extract FILEs from ZIP archive -l List contents (with -q for short form) -n Never overwrite files (default: ask) -o Overwrite -p Print to stdout -q Quiet -x FILE Exclude FILEs -d DIR Extract into DIR ---------------------------------------------------------------------- Ran 102 tests in 0.253s FAILED (failures=1, skipped=10) https://bugs.python.org/issue31940

Open Graph Description: Verified by compiling with alpine using the following Dockerfile and test script: FROM alpine RUN apk update && \ apk add expat-dev libressl-dev zlib-dev ncurses-dev bzip2-dev xz-dev \ ...

X Description: Verified by compiling with alpine using the following Dockerfile and test script: FROM alpine RUN apk update && \ apk add expat-dev libressl-dev zlib-dev ncurses-dev bzip2-dev x...

Opengraph URL: https://github.com/python/cpython/pull/4267

X: @github

direct link

Domain: github.com

route-pattern/:user_id/:repository/pull/:id/files(.:format)
route-controllerpull_requests
route-actionfiles
fetch-noncev2:229350ad-008c-d04c-7b28-50996bbb4bc1
current-catalog-service-hashae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b
request-idB812:645E3:15CFC6:2006C6:6A4DFD95
html-safe-nonced0fad1668a46561b514994e3ea3007f3934acc38d7822af9b74f3a4f54a8678b
visitor-payloadeyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJCODEyOjY0NUUzOjE1Q0ZDNjoyMDA2QzY6NkE0REZEOTUiLCJ2aXNpdG9yX2lkIjoiNTg2MDQwNTY5NjYxMzE4Njk2NSIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9
visitor-hmac907df8f6a14ea42171a9e3fd4661407a4277e7eba5381deee4d86cd4e0cae06f
hovercard-subject-tagpull_request:150662013
github-keyboard-shortcutsrepository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot
google-site-verificationApib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I
octolytics-urlhttps://collector.github.com/github/collect
analytics-location///pull_requests/show/files
fb:app_id1401488693436528
apple-itunes-appapp-id=1477376905, app-argument=https://github.com/python/cpython/pull/4267/files
twitter:imagehttps://avatars.githubusercontent.com/u/1810591?s=400&v=4
twitter:cardsummary_large_image
og:imagehttps://avatars.githubusercontent.com/u/1810591?s=400&v=4
og:image:altVerified by compiling with alpine using the following Dockerfile and test script: FROM alpine RUN apk update && \ apk add expat-dev libressl-dev zlib-dev ncurses-dev bzip2-dev xz-dev \ ...
og:site_nameGitHub
og:typeobject
hostnamegithub.com
expected-hostnamegithub.com
None5818716c93c6a2925b815402541a32814e43a7b1261c322b0c2df75224289566
turbo-cache-controlno-preview
diff-viewunified
go-importgithub.com/python/cpython git https://github.com/python/cpython.git
octolytics-dimension-user_id1525981
octolytics-dimension-user_loginpython
octolytics-dimension-repository_id81598961
octolytics-dimension-repository_nwopython/cpython
octolytics-dimension-repository_publictrue
octolytics-dimension-repository_is_forkfalse
octolytics-dimension-repository_network_root_id81598961
octolytics-dimension-repository_network_root_nwopython/cpython
turbo-body-classeslogged-out env-production page-responsive full-width
disable-turbotrue
browser-stats-urlhttps://api.github.com/_private/browser/stats
browser-errors-urlhttps://api.github.com/_private/browser/errors
release4314b1df11fa8a565684f3a72dc971e3785da365
ui-targetfull
theme-color#1e2327
color-schemelight dark

Links:

Skip to contenthttps://github.com/python/cpython/pull/4267/files#start-of-content
https://github.com/
Sign in https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fpython%2Fcpython%2Fpull%2F4267%2Ffiles
GitHub CopilotWrite better code with AIhttps://github.com/features/copilot
GitHub Copilot appDirect agents from issue to mergehttps://github.com/features/ai/github-app
MCP RegistryNewIntegrate external toolshttps://github.com/mcp
ActionsAutomate any workflowhttps://github.com/features/actions
CodespacesInstant dev environmentshttps://github.com/features/codespaces
IssuesPlan and track workhttps://github.com/features/issues
Code ReviewManage code changeshttps://github.com/features/code-review
GitHub Advanced SecurityFind and fix vulnerabilitieshttps://github.com/security/advanced-security
Code securitySecure your code as you buildhttps://github.com/security/advanced-security/code-security
Secret protectionStop leaks before they starthttps://github.com/security/advanced-security/secret-protection
Why GitHubhttps://github.com/why-github
Documentationhttps://docs.github.com
Bloghttps://github.blog
Changeloghttps://github.blog/changelog
Marketplacehttps://github.com/marketplace
View all featureshttps://github.com/features
Enterpriseshttps://github.com/enterprise
Small and medium teamshttps://github.com/team
Startupshttps://github.com/enterprise/startups
Nonprofitshttps://github.com/solutions/industry/nonprofits
App Modernizationhttps://github.com/solutions/use-case/app-modernization
DevSecOpshttps://github.com/solutions/use-case/devsecops
DevOpshttps://github.com/solutions/use-case/devops
CI/CDhttps://github.com/solutions/use-case/ci-cd
View all use caseshttps://github.com/solutions/use-case
Healthcarehttps://github.com/solutions/industry/healthcare
Financial serviceshttps://github.com/solutions/industry/financial-services
Manufacturinghttps://github.com/solutions/industry/manufacturing
Governmenthttps://github.com/solutions/industry/government
View all industrieshttps://github.com/solutions/industry
View all solutionshttps://github.com/solutions
AIhttps://github.com/resources/articles?topic=ai
Software Developmenthttps://github.com/resources/articles?topic=software-development
DevOpshttps://github.com/resources/articles?topic=devops
Securityhttps://github.com/resources/articles?topic=security
View all topicshttps://github.com/resources/articles
Customer storieshttps://github.com/customer-stories
Events & webinarshttps://github.com/resources/events
Ebooks & reportshttps://github.com/resources/whitepapers
Business insightshttps://github.com/solutions/executive-insights
GitHub Skillshttps://skills.github.com
Documentationhttps://docs.github.com
Customer supporthttps://support.github.com
Community forumhttps://github.com/orgs/community/discussions
Trust centerhttps://github.com/trust-center
Partnershttps://github.com/partners
View all resourceshttps://github.com/resources
GitHub SponsorsFund open source developershttps://github.com/sponsors
Security Labhttps://securitylab.github.com
Maintainer Communityhttps://maintainers.github.com
Acceleratorhttps://github.com/accelerator
GitHub Starshttps://stars.github.com
Archive Programhttps://archiveprogram.github.com
Topicshttps://github.com/topics
Trendinghttps://github.com/trending
Collectionshttps://github.com/collections
Enterprise platformAI-powered developer platformhttps://github.com/enterprise
GitHub Advanced SecurityEnterprise-grade security featureshttps://github.com/security/advanced-security
Copilot for BusinessEnterprise-grade AI featureshttps://github.com/features/copilot/copilot-business
Premium SupportEnterprise-grade 24/7 supporthttps://github.com/premium-support
Pricinghttps://github.com/pricing
Search syntax tipshttps://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax
documentationhttps://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax
Sign in https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fpython%2Fcpython%2Fpull%2F4267%2Ffiles
Sign up https://github.com/signup?ref_cta=Sign+up&ref_loc=header+logged+out&ref_page=%2F%3Cuser-name%3E%2F%3Crepo-name%3E%2Fpull_requests%2Fshow%2Ffiles&source=header-repo&source_repo=python%2Fcpython
Reloadhttps://github.com/python/cpython/pull/4267/files
Reloadhttps://github.com/python/cpython/pull/4267/files
Reloadhttps://github.com/python/cpython/pull/4267/files
Please reload this pagehttps://github.com/python/cpython/pull/4267/files
python https://github.com/python
cpythonhttps://github.com/python/cpython
Please reload this pagehttps://github.com/python/cpython/pull/4267/files
Notifications https://github.com/login?return_to=%2Fpython%2Fcpython
Fork 34.8k https://github.com/login?return_to=%2Fpython%2Fcpython
Star 73.6k https://github.com/login?return_to=%2Fpython%2Fcpython
Code https://github.com/python/cpython
Issues 5k+ https://github.com/python/cpython/issues
Pull requests 2.3k https://github.com/python/cpython/pulls
Actions https://github.com/python/cpython/actions
Projects https://github.com/python/cpython/projects
Security and quality 0 https://github.com/python/cpython/security
Insights https://github.com/python/cpython/pulse
Code https://github.com/python/cpython
Issues https://github.com/python/cpython/issues
Pull requests https://github.com/python/cpython/pulls
Actions https://github.com/python/cpython/actions
Projects https://github.com/python/cpython/projects
Security and quality https://github.com/python/cpython/security
Insights https://github.com/python/cpython/pulse
Sign up for GitHub https://github.com/signup?return_to=%2Fpython%2Fcpython%2Fissues%2Fnew%2Fchoose
terms of servicehttps://docs.github.com/terms
privacy statementhttps://docs.github.com/privacy
Sign inhttps://github.com/login?return_to=%2Fpython%2Fcpython%2Fissues%2Fnew%2Fchoose
asottilehttps://github.com/asottile
python:masterhttps://github.com/python/cpython/tree/master
asottile:fix_shutil_copystat_alpine_bpo-31940https://github.com/asottile/cpython/tree/fix_shutil_copystat_alpine_bpo-31940
Conversation 33 https://github.com/python/cpython/pull/4267
Commits 1 https://github.com/python/cpython/pull/4267/commits
Checks 0 https://github.com/python/cpython/pull/4267/checks
Files changed https://github.com/python/cpython/pull/4267/files
Please reload this pagehttps://github.com/python/cpython/pull/4267/files
bpo-31940: Reject faulty lchown implementations https://github.com/python/cpython/pull/4267/files#top
Show all changes 1 commit https://github.com/python/cpython/pull/4267/files
ad495d0 bpo-31940: Reject faulty lchown implementations asottile Nov 4, 2017 https://github.com/python/cpython/pull/4267/commits/ad495d0b10255c488a1afc3eb12b0f49b19919fd
Clear filters https://github.com/python/cpython/pull/4267/files
Please reload this pagehttps://github.com/python/cpython/pull/4267/files
Please reload this pagehttps://github.com/python/cpython/pull/4267/files
2017-11-03-21-54-32.bpo-31940.fG1wnQ.rst https://github.com/python/cpython/pull/4267/files#diff-64afa9fa3c8444f607f09357c716f84b9b8e08722f452134974afe42c9574569
configure https://github.com/python/cpython/pull/4267/files#diff-90d08e583c4c9c6f391b2ae90f819f600a6326928ea9512c9e0c6d98e9f29ac2
configure.ac https://github.com/python/cpython/pull/4267/files#diff-49473dca262eeab3b4a43002adb08b4db31020d190caaad1594b47f1d5daa810
Misc/NEWS.d/next/Build/2017-11-03-21-54-32.bpo-31940.fG1wnQ.rsthttps://github.com/python/cpython/pull/4267/files#diff-64afa9fa3c8444f607f09357c716f84b9b8e08722f452134974afe42c9574569
View file https://github.com/asottile/cpython/blob/ad495d0b10255c488a1afc3eb12b0f49b19919fd/Misc/NEWS.d/next/Build/2017-11-03-21-54-32.bpo-31940.fG1wnQ.rst
Open in desktop https://desktop.github.com
https://github.co/hiddenchars
https://github.com/python/cpython/pull/4267/{{ revealButtonHref }}
rm-youhttps://github.com/rm-you
Nov 18, 2017https://github.com/python/cpython/pull/4267/files#r151828341
Learn morehttps://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment
Please reload this pagehttps://github.com/python/cpython/pull/4267/files
prometheanfirehttps://github.com/prometheanfire
Dec 4, 2017https://github.com/python/cpython/pull/4267/files#r154734357
Learn morehttps://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment
Please reload this pagehttps://github.com/python/cpython/pull/4267/files
asottilehttps://github.com/asottile
Dec 4, 2017https://github.com/python/cpython/pull/4267/files#r154735252
Learn morehttps://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment
Please reload this pagehttps://github.com/python/cpython/pull/4267/files
configurehttps://github.com/python/cpython/pull/4267/files#diff-90d08e583c4c9c6f391b2ae90f819f600a6326928ea9512c9e0c6d98e9f29ac2
View file https://github.com/asottile/cpython/blob/ad495d0b10255c488a1afc3eb12b0f49b19919fd/configure
Open in desktop https://desktop.github.com
https://github.co/hiddenchars
https://github.com/python/cpython/pull/4267/{{ revealButtonHref }}
https://github.com/python/cpython/pull/4267/files#diff-90d08e583c4c9c6f391b2ae90f819f600a6326928ea9512c9e0c6d98e9f29ac2
https://github.com/python/cpython/pull/4267/files#diff-90d08e583c4c9c6f391b2ae90f819f600a6326928ea9512c9e0c6d98e9f29ac2
https://github.com/python/cpython/pull/4267/files#diff-90d08e583c4c9c6f391b2ae90f819f600a6326928ea9512c9e0c6d98e9f29ac2
rm-youhttps://github.com/rm-you
Nov 18, 2017https://github.com/python/cpython/pull/4267/files#r151828352
Learn morehttps://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment
Please reload this pagehttps://github.com/python/cpython/pull/4267/files
rm-youhttps://github.com/rm-you
Nov 18, 2017https://github.com/python/cpython/pull/4267/files#r151828398
Learn morehttps://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment
Please reload this pagehttps://github.com/python/cpython/pull/4267/files
asottilehttps://github.com/asottile
Nov 18, 2017https://github.com/python/cpython/pull/4267/files#r151828536
Learn morehttps://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment
Please reload this pagehttps://github.com/python/cpython/pull/4267/files
rm-youhttps://github.com/rm-you
Nov 18, 2017https://github.com/python/cpython/pull/4267/files#r151828451
Learn morehttps://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment
Please reload this pagehttps://github.com/python/cpython/pull/4267/files
https://github.com/python/cpython/pull/4267/files#diff-90d08e583c4c9c6f391b2ae90f819f600a6326928ea9512c9e0c6d98e9f29ac2
configure.achttps://github.com/python/cpython/pull/4267/files#diff-49473dca262eeab3b4a43002adb08b4db31020d190caaad1594b47f1d5daa810
View file https://github.com/asottile/cpython/blob/ad495d0b10255c488a1afc3eb12b0f49b19919fd/configure.ac
Open in desktop https://desktop.github.com
https://github.co/hiddenchars
https://github.com/python/cpython/pull/4267/{{ revealButtonHref }}
https://github.com/python/cpython/pull/4267/files#diff-49473dca262eeab3b4a43002adb08b4db31020d190caaad1594b47f1d5daa810
https://github.com/python/cpython/pull/4267/files#diff-49473dca262eeab3b4a43002adb08b4db31020d190caaad1594b47f1d5daa810
https://github.com/python/cpython/pull/4267/files#diff-49473dca262eeab3b4a43002adb08b4db31020d190caaad1594b47f1d5daa810
rm-youhttps://github.com/rm-you
Nov 18, 2017https://github.com/python/cpython/pull/4267/files#r151828456
Learn morehttps://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment
Please reload this pagehttps://github.com/python/cpython/pull/4267/files
https://github.com/python/cpython/pull/4267/files#diff-49473dca262eeab3b4a43002adb08b4db31020d190caaad1594b47f1d5daa810
https://github.com
Termshttps://docs.github.com/site-policy/github-terms/github-terms-of-service
Privacyhttps://docs.github.com/site-policy/privacy-policies/github-privacy-statement
Securityhttps://github.com/security
Statushttps://www.githubstatus.com/
Communityhttps://github.community/
Docshttps://docs.github.com/
Contacthttps://support.github.com?tags=dotcom-footer

Viewport: width=device-width


URLs of crawlers that visited me.