-
Notifications
You must be signed in to change notification settings - Fork 15
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
Host runs out of memory resulting in mscp being killed #8
Comments
Hi, Could you please show the output of mscp with |
@upa
Also, another observation is that if the directory already exists at the destination, it messes up the location. This is the command I used: On the second run it created ocalcache dir on remote host:
|
Hi, Thank you for the screenshot and logs. I have determined the reason of OOM. I will fix it soon.
This is an expected behavior. scp also does so.
|
@upa For the second issue mscp seems to be creating the dir with a wrong name. The expected dir is "localcache" and it ends up creating "ocalcache" |
Oh, I see. will investigate. |
Coping a file under / (root) to a remote directory causes corrupted remote path (Issue #8).
When a source file path is /FILE, its dest path would be dst/ILE. This commit fixes this issue (#8).
This commit makes struct path allocation use strndup(). It reduices the memory footprint for struct path per file (issue #8).
This commit makes struct path allocation use strndup(). It reduices the memory footprint for struct path per file (issue #8).
The dev branch now includes the fixes for both issues (the truncated remote dir name and OOM). Could you clone and build the dev branch, and test it in your environment? The OOM issue depends on the number of files being copied. I guess the fixed version should be suitable for almost all cases including yours. If it is ok, I will merge it into the main branch and make a new release. |
Coping a file under / (root) to a remote directory causes corrupted remote path (Issue #8).
When a source file path is /FILE, its dest path would be dst/ILE. This commit fixes this issue (#8).
This commit makes struct path allocation use strndup(). It reduices the memory footprint for struct path per file (issue #8).
I created the new release v0.1.3 including the fixes for this issue, and it is available on homebrew and ubuntu ppa. I believe the issues are resolved in your env, but if not, please make a new issue with detailed information. Thanks. |
System configuration:
MSCP stack trace:
The text was updated successfully, but these errors were encountered: