libssh2_sftp_symlink: wrong order of arguments
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
调研方向
Start by reading the libssh2_sftp_symlink_ex implementation in src/sftp.c near line 3687 and compare its argument handling with the SFTP extension behavior described in the issue. Check related symlink tests and callers to understand compatibility expectations; done means the function handles the argument order correctly without breaking existing behavior, with tests covering the expected paths.
由索引模型根据 Issue 内容生成。
描述
I believe this function implementation has never worked correctly!?
#define libssh2_sftp_symlink(sftp, orig, linkpath) \
libssh2_sftp_symlink_ex((sftp), orig, (unsigned int)strlen(orig), \
linkpath, (unsigned int)strlen(linkpath), \
LIBSSH2_SFTP_SYMLINK)
Because:
https://www.sftp.net/spec/openssh-sftp-extensions.txt
When OpenSSH's sftp-server was implemented, the order of the arguments
to the SSH_FXP_SYMLINK method was inadvertently reversed. Unfortunately,
the reversal was not noticed until the server was widely deployed. Since
fixing this to follow the specification would cause incompatibility, the
current order was retained. For correct operation, clients should send
SSH_FXP_SYMLINK as follows:
uint32 id
string targetpath
string linkpath
In my code, I'm simply reversing the arguments, but I think this should better be fixed at the source, inside:
https://github.com/libssh2/libssh2/blob/a951e06640c8432eba70f01efe85b10ec5f80119/src/sftp.c#L3687
- 主要语言
- C
- 星标
- 1.6k
- 派生
- 646
- PR 合并指标
- 30 天内没有已合并 PR
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 没有贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
libssh2/libssh2 的其他 Issue
-
难度 2/5 1-3 小时 新手友好度 74/100
libssh2/libssh2#2676 · 1 条评论 ·
维护者通常 1 天内回复
-
AEAD ciphers (chacha20-poly1305, AES-GCM) leak 16 bytes of uninitialized memory per outgoing encrypted packet可能已有人在做 关联的 PR 仍在进行中或已合并。 未关闭memory-leak
难度 2/5 1-3 小时 新手友好度 85/100
libssh2/libssh2#2023 · 5 条评论 ·
维护者通常 1 天内回复
-
Q&A
难度 4/5 3-5 天 新手友好度 45/100
维护者通常 1 天内回复
-
难度 4/5 3-5 天 新手友好度 35/100
libssh2/libssh2#1925 · 13 条评论 · 32 个 reaction ·
维护者通常 1 天内回复
-
To avoid interfering with auth, send keepalives at the end of the interval instead of the beginning可能已有人在做 @MatrixManAtYrService 于 236 天前认领。 未关闭bug
难度 2/5 1-3 小时 新手友好度 35/100
libssh2/libssh2#1810 · 1 条评论 ·
维护者通常 1 天内回复
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 78/100
zephyrproject-rtos/zephyr#123423 ·
维护者通常 2 天内回复
-
bug
难度 2/5 1-3 小时 新手友好度 83/100
nextbsd/nextbsd-userland#434 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 77/100
维护者通常 1 天内回复
-
[YSQL] Local-node initdb reads postgres.bki, but YB clusters bootstrap only from yb_postgres.bki未关闭area/ysql kind/enhancement priority/low status/awaiting-triage
难度 2/5 1-3 小时 新手友好度 72/100
yugabyte/yugabyte-db#35763 ·
维护者通常 1 天内回复
-
category:port-update
难度 2/5 1-3 小时 新手友好度 66/100
维护者通常 2 天内回复