Skip to content

Camera discovery fixups#13197

Merged
HTRamsey merged 3 commits intomasterfrom
pr-camera-fixup
Aug 2, 2025
Merged

Camera discovery fixups#13197
HTRamsey merged 3 commits intomasterfrom
pr-camera-fixup

Conversation

@julianoes
Copy link
Contributor

@julianoes julianoes commented Jul 22, 2025

Follow up fixes to #12627 #13100.

This fixes a segfault in:

src/VideoManager/VideoManager.cc:551
MavlinkCameraControl *pCamera = _activeVehicle->cameraManager()->currentCameraInstance();
@julianoes julianoes marked this pull request as ready for review July 23, 2025 04:25
@julianoes julianoes requested a review from DonLakeFlyer July 23, 2025 04:25
@HTRamsey HTRamsey merged commit 28c834b into master Aug 2, 2025
19 checks passed
@HTRamsey HTRamsey deleted the pr-camera-fixup branch August 2, 2025 04:52
new-tonAA pushed a commit to new-tonAA/qgroundcontrol that referenced this pull request Aug 12, 2025
* VideoManager: fix crash on camera disconnect

This fixes a segfault in:

src/VideoManager/VideoManager.cc:551
MavlinkCameraControl *pCamera = _activeVehicle->cameraManager()->currentCameraInstance();

* FlightMap: drive-by bugfix

---------

Co-authored-by: Holden Ramsey <68555040+HTRamsey@users.noreply.github.com>
julianoes added a commit that referenced this pull request Sep 1, 2025
* VideoManager: fix crash on camera disconnect

This fixes a segfault in:

src/VideoManager/VideoManager.cc:551
MavlinkCameraControl *pCamera = _activeVehicle->cameraManager()->currentCameraInstance();

* FlightMap: drive-by bugfix

---------

Co-authored-by: Holden Ramsey <68555040+HTRamsey@users.noreply.github.com>
DonLakeFlyer pushed a commit that referenced this pull request Sep 15, 2025
* VideoManager: fix crash on camera disconnect

This fixes a segfault in:

src/VideoManager/VideoManager.cc:551
MavlinkCameraControl *pCamera = _activeVehicle->cameraManager()->currentCameraInstance();

* FlightMap: drive-by bugfix

---------

Co-authored-by: Holden Ramsey <68555040+HTRamsey@users.noreply.github.com>
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.

2 participants