Skip to content

fix: prevent potential nil pointer dereference#8756

Merged
mnencia merged 1 commit intocloudnative-pg:mainfrom
PascalBourdier:potential-nil-dereference
Oct 10, 2025
Merged

fix: prevent potential nil pointer dereference#8756
mnencia merged 1 commit intocloudnative-pg:mainfrom
PascalBourdier:potential-nil-dereference

Conversation

@PascalBourdier
Copy link
Contributor

@PascalBourdier PascalBourdier commented Oct 6, 2025

Prevent possible nil pointer dereference when an error occurs inside the hibernation reconciler.

@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Oct 6, 2025
@cnpg-bot cnpg-bot added backport-requested ◀️ This pull request should be backported to all supported releases release-1.25 release-1.26 release-1.27 labels Oct 6, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Oct 6, 2025

❗ By default, the pull request is configured to backport to all release branches.

  • To stop backporting this pr, remove the label: backport-requested ◀️ or add the label 'do not backport'
  • To stop backporting this pr to a certain release branch, remove the specific branch label: release-x.y

@PascalBourdier PascalBourdier force-pushed the potential-nil-dereference branch from 1836afb to 736fabb Compare October 6, 2025 09:49
@dosubot dosubot bot added the bug 🐛 Something isn't working label Oct 6, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Oct 9, 2025
@mnencia mnencia force-pushed the potential-nil-dereference branch 3 times, most recently from d76d692 to 2b79325 Compare October 9, 2025 18:22
@mnencia
Copy link
Member

mnencia commented Oct 9, 2025

/test

@github-actions
Copy link
Contributor

github-actions bot commented Oct 9, 2025

@mnencia, here's the link to the E2E on CNPG workflow run: https://github.com/cloudnative-pg/cloudnative-pg/actions/runs/18385443564

@mnencia mnencia force-pushed the potential-nil-dereference branch from 2b79325 to 4b26ec1 Compare October 9, 2025 19:25
@cnpg-bot cnpg-bot added the ok to merge 👌 This PR can be merged label Oct 9, 2025
improve code to avoid panic

Signed-off-by: Pascal Bourdier <pascal.bourdier@gmail.com>
@mnencia mnencia force-pushed the potential-nil-dereference branch from 4b26ec1 to 9b26391 Compare October 10, 2025 07:01
@mnencia mnencia changed the title fix: potential nil dereference fix: prevent potential nil pointer dereference Oct 10, 2025
@mnencia mnencia merged commit 5788e02 into cloudnative-pg:main Oct 10, 2025
33 checks passed
cnpg-bot pushed a commit that referenced this pull request Oct 10, 2025
Prevent possible nil pointer dereference when an error occurs inside the
hibernation reconciler.

Signed-off-by: Pascal Bourdier <pascal.bourdier@gmail.com>
(cherry picked from commit 5788e02)
cnpg-bot pushed a commit that referenced this pull request Oct 10, 2025
Prevent possible nil pointer dereference when an error occurs inside the
hibernation reconciler.

Signed-off-by: Pascal Bourdier <pascal.bourdier@gmail.com>
(cherry picked from commit 5788e02)
cnpg-bot pushed a commit that referenced this pull request Oct 10, 2025
Prevent possible nil pointer dereference when an error occurs inside the
hibernation reconciler.

Signed-off-by: Pascal Bourdier <pascal.bourdier@gmail.com>
(cherry picked from commit 5788e02)
THE-BRAHMA pushed a commit to THE-BRAHMA/cloudnative-pg that referenced this pull request Oct 30, 2025
Prevent possible nil pointer dereference when an error occurs inside the
hibernation reconciler.

Signed-off-by: Pascal Bourdier <pascal.bourdier@gmail.com>
Signed-off-by: theBrahma <office.utpal.brahma@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-requested ◀️ This pull request should be backported to all supported releases bug 🐛 Something isn't working lgtm This PR has been approved by a maintainer no-issue ok to merge 👌 This PR can be merged release-1.25 release-1.26 release-1.27 size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants