Skip to content

Prevent reading key on SFTP stat bool - #31531

Merged
skjnldsv merged 1 commit into
masterfrom
fix/sftp-offset
Mar 11, 2022
Merged

Prevent reading key on SFTP stat bool#31531
skjnldsv merged 1 commit into
masterfrom
fix/sftp-offset

Conversation

@skjnldsv

Copy link
Copy Markdown
Member

Signed-off-by: John Molakvoæ <skjnldsv@protonmail.com>
@skjnldsvskjnldsv added this to the Nextcloud 24 milestone Mar 11, 2022
@skjnldsv
skjnldsv requested a review from a teamMarch 11, 2022 09:55
@skjnldsvskjnldsv self-assigned this Mar 11, 2022
@skjnldsv
skjnldsv requested review from PVince81, come-nc and icewind1991 and removed request for a teamMarch 11, 2022 09:55
@skjnldsv

Copy link
Copy Markdown
MemberAuthor

/backport to stable23

@skjnldsv

Copy link
Copy Markdown
MemberAuthor

/backport to stable22

@skjnldsv

Copy link
Copy Markdown
MemberAuthor

/backport to stable21

@PVince81PVince81 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@CarlSchwanCarlSchwan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not the first time we had this issue :/

@skjnldsvskjnldsv added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Mar 11, 2022
@skjnldsv
skjnldsv merged commit bcb1fc5 into masterMar 11, 2022
@skjnldsv
skjnldsv deleted the fix/sftp-offset branch March 11, 2022 15:02
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to releaseReady to be released and/or waiting for tests to finishbugfeature: external storage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Error: Trying to access array offset on value of type bool at /var/www/nextcloud/apps/files_external/lib/Lib/Storage/SFTP.php on line 331 and 335

3 participants

@skjnldsv@PVince81@CarlSchwan