Title: inaccurate block size in socket._sendfile_use_sendfile · Issue #112341 · python/cpython · GitHub
Open Graph Title: inaccurate block size in socket._sendfile_use_sendfile · Issue #112341 · python/cpython
X Title: inaccurate block size in socket._sendfile_use_sendfile · Issue #112341 · python/cpython
Description: Bug report Bug description: socket._sendfile_use_sendfile(self, file, offset=0, count=None) intent to use the file size reamains from current position as block size, to a underlying sendfile syscall, when count is not specified. But as t...
Open Graph Description: Bug report Bug description: socket._sendfile_use_sendfile(self, file, offset=0, count=None) intent to use the file size reamains from current position as block size, to a underlying sendfile syscal...
X Description: Bug report Bug description: socket._sendfile_use_sendfile(self, file, offset=0, count=None) intent to use the file size reamains from current position as block size, to a underlying sendfile syscal...
Opengraph URL: https://github.com/python/cpython/issues/112341
X: @github
Domain: github.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"inaccurate block size in socket._sendfile_use_sendfile","articleBody":"# Bug report\r\n\r\n### Bug description:\r\n\r\n`socket._sendfile_use_sendfile(self, file, offset=0, count=None)` intent to use the file size reamains from current position as block size, to a underlying `sendfile` syscall, when count is not specified.\r\n\r\nBut as the codes below shows, it will always use the whole file size as block size even if offset is non zero, so the blocksize is larger than actuall file bytes remains from offset:\r\n\r\nhttps://github.com/python/cpython/blob/89ddea4886942b0c27a778a0ad3f0d5ac5f518f0/Lib/socket.py#L363\r\n\r\nOn modern platforms, if someone specified the count which is larger than the actual file size, `sendfile` will accept it, and only sending the remaining data and returns the bytes which have been sent.\r\n\r\nSo the bug (if it is a bug) have no effect for now, and it's so minor, but I think we should fix it however. And if we want using `TransmitFile` as a `sendfile` implenentation on Windows, the inaccurate block size will leads to an error, which is mentioned in https://github.com/python/cpython/issues/102898#issuecomment-1824554063, and the fix will be a must have.\r\n\r\n### CPython versions tested on:\r\n\r\n3.8, 3.9, 3.10, 3.11, 3.12, 3.13, CPython main branch\r\n\r\n### Operating systems tested on:\r\n\r\nmacOS\r\n\r\n\u003c!-- gh-linked-prs --\u003e\r\n### Linked PRs\r\n* gh-112342\r\n\u003c!-- /gh-linked-prs --\u003e\r\n","author":{"url":"https://github.com/aisk","@type":"Person","name":"aisk"},"datePublished":"2023-11-23T16:20:07.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":2},"url":"https://github.com/112341/cpython/issues/112341"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:da8e635c-7109-0710-7d87-51ef5fa5d10a |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | 8AB8:E2378:7267969:A050906:6A4FD757 |
| html-safe-nonce | b9f00aa6eca90826e873ae30fe4cc55144c18838414bec00ba1318954bbcffde |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiI4QUI4OkUyMzc4OjcyNjc5Njk6QTA1MDkwNjo2QTRGRDc1NyIsInZpc2l0b3JfaWQiOiIxMjgzNDUyMjUyOTQ3NjM4NjMiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | 2aa1ea08f301c2b6c00b8a6e40116a25640c9559c912baed7bdfb196106c7fcf |
| hovercard-subject-tag | issue:2008527162 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/python/cpython/112341/issue_layout |
| twitter:image | https://opengraph.githubassets.com/ccbc6c9d44c887b2c0a1f3e15d94d1a731291331338297fee3e7bf4b89780267/python/cpython/issues/112341 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/ccbc6c9d44c887b2c0a1f3e15d94d1a731291331338297fee3e7bf4b89780267/python/cpython/issues/112341 |
| og:image:alt | Bug report Bug description: socket._sendfile_use_sendfile(self, file, offset=0, count=None) intent to use the file size reamains from current position as block size, to a underlying sendfile syscal... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | aisk |
| hostname | github.com |
| expected-hostname | github.com |
| None | 19b98c3d9767bb8a56238a89ff8401d1cbdf3bc5c353799aacf49fe6c76b4b7c |
| turbo-cache-control | no-preview |
| go-import | github.com/python/cpython git https://github.com/python/cpython.git |
| octolytics-dimension-user_id | 1525981 |
| octolytics-dimension-user_login | python |
| octolytics-dimension-repository_id | 81598961 |
| octolytics-dimension-repository_nwo | python/cpython |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 81598961 |
| octolytics-dimension-repository_network_root_nwo | python/cpython |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 142593e66413bbae27fd485859993597d5a3b104 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width