Skip to content

Fix panic caused by nil pointer dereference in Query Frontend#6609

Merged
yeya24 merged 1 commit into
cortexproject:masterfrom
rapphil:rapphil-fix-panic-query-frontend
Feb 26, 2025
Merged

Fix panic caused by nil pointer dereference in Query Frontend#6609
yeya24 merged 1 commit into
cortexproject:masterfrom
rapphil:rapphil-fix-panic-query-frontend

Conversation

@rapphil
Copy link
Copy Markdown
Contributor

@rapphil rapphil commented Feb 25, 2025

What this PR does:

Fix latent bug in which a nil pointer dereference can happen in case the response received by the Frontend contains an error.

Which issue(s) this PR fixes:
Fixes #

Checklist

  • Tests updated
  • Documentation added - NA
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

@rapphil rapphil force-pushed the rapphil-fix-panic-query-frontend branch from 81ae649 to e108c89 Compare February 25, 2025 22:02
Signed-off-by: Raphael Silva <rapphil@gmail.com>
@rapphil rapphil force-pushed the rapphil-fix-panic-query-frontend branch from e108c89 to 52d96d6 Compare February 25, 2025 22:10
@rapphil rapphil changed the title Fix panic caused by nil pointer dereference Fix panic caused by nil pointer dereference in Query Frontend Feb 25, 2025
Copy link
Copy Markdown
Contributor

@danielblando danielblando left a comment

Choose a reason for hiding this comment

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

Thanks

Copy link
Copy Markdown
Contributor

@yeya24 yeya24 left a comment

Choose a reason for hiding this comment

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

Thanks

@yeya24 yeya24 merged commit 9623c9f into cortexproject:master Feb 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants