Skip to content
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

rTorrent: Resolve memory access crash #310

Merged
merged 1 commit into from
Jan 16, 2024

Conversation

stickz
Copy link
Contributor

@stickz stickz commented Jan 3, 2024

rTorrent requires the usage of aligned memory access on all platforms (x86 and ARM) to prevent a memory access crash from happening. As such, we need to configure libtorrent with --enable-aligned to enable the feature.

This is also more efficient for CPU usage at the cost of slightly more memory consumption. However, due to the relatively small usage of memory, all platforms will benefit.

More information can be found here: rakshasa/libtorrent#244

rTorrent requires the usage of aligned memory access on all platforms (x86 and ARM) to prevent a memory access crash from happening. As such, we need to configure libtorrent with `--enable-aligned` to enable the feature.

This is also more efficient for CPU usage at the cost of slightly more memory consumption.  However, due to the relatively small usage of memory, all platforms will benefit.

More information can be found here: rakshasa/libtorrent#244
@stickz
Copy link
Contributor Author

stickz commented Jan 14, 2024

@crazy-max There has been an issue opened #315 about rTorrent segmentation faults during hash checking. This PR might resolve that problem. Could we get this pushed to docker edge, so the user can confirm a resolution or continue testing?

@crazy-max crazy-max merged commit e3784e3 into crazy-max:master Jan 16, 2024
7 checks passed
@stickz stickz deleted the rtorrent-crash-fix branch January 16, 2024 05:29
XxAcielxX added a commit to XxAcielxX/docker-rutorrent that referenced this pull request Apr 28, 2024
## Changes:
* Update to `ruTorrent v4.3.0`

## Base Image:
* rTorrent: Add TCP socket configurations
([322](crazy-max/docker-rtorrent-rutorrent#322))
* Add php82-fileinfo package
([325](crazy-max/docker-rtorrent-rutorrent#325))
* rTorrent: Set of patches to fix memory leaks
([308](crazy-max/docker-rtorrent-rutorrent#308))
* rTorrent: Fix memory access crash
([310](crazy-max/docker-rtorrent-rutorrent#310))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants