Skip to content

[8.19](backport #47950) [meraki] improve defensive checks to prevent panics#47952

Merged
gpop63 merged 1 commit into8.19from
mergify/bp/8.19/pr-47950
Dec 7, 2025
Merged

[8.19](backport #47950) [meraki] improve defensive checks to prevent panics#47952
gpop63 merged 1 commit into8.19from
mergify/bp/8.19/pr-47950

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Dec 7, 2025

Overview

Fix nil pointer dereference panic in getDeviceSwitchports when the Meraki API returns switch port data for devices not present in the local devices map. + more safety checks to prevent panics

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works. Where relevant, I have used the stresstest.sh script to run them under stress conditions and race detector to verify their stability.
  • I have added an entry in ./changelog/fragments using the changelog tool.
    This is an automatic backport of pull request [meraki] improve defensive checks to prevent panics #47950 done by Mergify.
* check if device exists in the map

* add fragment

* add more safety checks

* Fix more potential nil panic(s)

* Fix more potential nil panic(s)

* update fragment

---------

Co-authored-by: subham sarkar <subham.sarkar@elastic.co>
(cherry picked from commit 1c8215c)
@mergify mergify bot requested a review from a team as a code owner December 7, 2025 16:23
@mergify mergify bot added the backport label Dec 7, 2025
@mergify mergify bot requested review from VihasMakwana and orestisfl and removed request for a team December 7, 2025 16:23
@mergify mergify bot assigned gpop63 Dec 7, 2025
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Dec 7, 2025
@github-actions github-actions bot added the Team:Obs-InfraObs Label for the Observability Infrastructure Monitoring team label Dec 7, 2025
@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Dec 7, 2025
@gpop63 gpop63 merged commit 4e3987b into 8.19 Dec 7, 2025
33 checks passed
@gpop63 gpop63 deleted the mergify/bp/8.19/pr-47950 branch December 7, 2025 20:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport Team:Obs-InfraObs Label for the Observability Infrastructure Monitoring team

1 participant