Skip to content

fix: prevent crash when tapping marker without snippet on iOS#568

Merged
jokerttu merged 1 commit intomainfrom
fix/not-guarded-marker-snippet-string-on-ios
Apr 13, 2026
Merged

fix: prevent crash when tapping marker without snippet on iOS#568
jokerttu merged 1 commit intomainfrom
fix/not-guarded-marker-snippet-string-on-ios

Conversation

@jokerttu
Copy link
Copy Markdown
Contributor

Adds nil guard on iOS implementaiton for marker.snippet in handleMarkerClick: and handleMarkerInfoWindowTapped: to prevent NULL pointer dereference crash when tapping markers without a snippet set.

Fixes #559

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I signed the CLA.
  • I listed at least one issue that this PR fixes in the description above.
  • I updated/added relevant documentation
  • I added new tests to check the change I am making
  • All existing and new tests are passing.

Copy link
Copy Markdown
Contributor

@illuminati1911 illuminati1911 left a comment

Choose a reason for hiding this comment

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

LGTM

@jokerttu jokerttu merged commit 72069ca into main Apr 13, 2026
13 checks passed
@jokerttu jokerttu deleted the fix/not-guarded-marker-snippet-string-on-ios branch April 13, 2026 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

iOS crash: NULL pointer dereference in handleMarkerClick due to nil marker.snippet

3 participants