Skip to content

fix: multirepo potential nil pointer dereference#658

Merged
kommendorkapten merged 1 commit intotheupdateframework:masterfrom
MrDan4es:multirepo_nil_pointer_repo
Nov 20, 2024
Merged

fix: multirepo potential nil pointer dereference#658
kommendorkapten merged 1 commit intotheupdateframework:masterfrom
MrDan4es:multirepo_nil_pointer_repo

Conversation

@MrDan4es
Copy link
Copy Markdown
Contributor

This PR prevent potential panic on NIL pointer when we trying to download file (from user input repo name for example).

@MrDan4es MrDan4es requested a review from a team as a code owner November 20, 2024 10:10
@kommendorkapten
Copy link
Copy Markdown
Member

Thanks for fixing this. Would you mind amend the commit with a signoff?

@MrDan4es MrDan4es force-pushed the multirepo_nil_pointer_repo branch from 31f7051 to 59b3c0d Compare November 20, 2024 10:33
@MrDan4es
Copy link
Copy Markdown
Contributor Author

Done

Signed-off-by: mrdan4es <mrdan4es@gmail.com>
@MrDan4es MrDan4es force-pushed the multirepo_nil_pointer_repo branch from 59b3c0d to f0cdc4f Compare November 20, 2024 10:49
Copy link
Copy Markdown
Contributor

@rdimitrov rdimitrov left a comment

Choose a reason for hiding this comment

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

This looks nice! 🙌 Thank you 🚀

Copy link
Copy Markdown
Member

@kommendorkapten kommendorkapten left a comment

Choose a reason for hiding this comment

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

Thanks!

@kommendorkapten kommendorkapten merged commit 990cdb8 into theupdateframework:master Nov 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants