Skip to content

libssh: make atime and mtime cap the timestamp instead of wrap - #18989

Closed
bagder wants to merge 1 commit into
masterfrom
bagder/libssh-date
Closed

libssh: make atime and mtime cap the timestamp instead of wrap#18989
bagder wants to merge 1 commit into
masterfrom
bagder/libssh-date

Conversation

@bagder

@bagder bagder commented Oct 9, 2025

Copy link
Copy Markdown
Member

The libssh API uses a 32 bit type for datestamp, so instead of just force-typecast it, make sure it gets capped at UINT_MAX if the value is larger.

Reported-by: Joshua Rogers

The libssh API uses a 32 bit type for datestamp, so instead of just
force-typecast it, make sure it gets capped at UINT_MAX if the value is
larger.

Reported-by: Joshua Rogers
@bagder bagder added the SCP/SFTP label Oct 9, 2025
@bagder
bagder marked this pull request as ready for review October 10, 2025 06:25
@bagder bagder closed this in c049c37 Oct 10, 2025
@bagder
bagder deleted the bagder/libssh-date branch October 10, 2025 06:26
vszakats added a commit that referenced this pull request Apr 5, 2026
… check

Follow-up to 8c8eeba #21214 (wrong silencing)
Follow-up to c049c37 #18989 (add redundant check)
Follow-up to c988ec9 #9328 (make check fail)
Follow-up to 44a02d2 #9324 (add original check)

Closes #21225
outcast36 pushed a commit to greearb/curl that referenced this pull request Jun 3, 2026
… check

Follow-up to 8c8eeba curl#21214 (wrong silencing)
Follow-up to c049c37 curl#18989 (add redundant check)
Follow-up to c988ec9 curl#9328 (make check fail)
Follow-up to 44a02d2 curl#9324 (add original check)

Closes curl#21225
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

1 participant