Skip to content

Fix crash in camera tethering#13255

Merged
TurboGit merged 1 commit intodarktable-org:masterfrom
victoryforce:fix-for-12022
Jan 2, 2023
Merged

Fix crash in camera tethering#13255
TurboGit merged 1 commit intodarktable-org:masterfrom
victoryforce:fix-for-12022

Conversation

@victoryforce
Copy link
Collaborator

This fix adds a NULL check to prevent further NULL dereferencing.

Resolves #12022.

@TurboGit TurboGit added this to the 4.2.1 milestone Jan 2, 2023
@TurboGit TurboGit added bugfix pull request fixing a bug priority: medium core features are degraded in a way that is still mostly usable, software stutters scope: hardware support dealing with drivers and external devices: GPU, printers release notes: pending labels Jan 2, 2023
@TurboGit TurboGit self-requested a review January 2, 2023 12:10
@victoryforce
Copy link
Collaborator Author

To facilitate the review. The problem is that the first parameter of the gp_widget_get_child_by_name function is not checked for NULL before dereferencing: https://github.com/gphoto/libgphoto2/blob/master/libgphoto2/gphoto2-widget.c#L646

Copy link
Member

@TurboGit TurboGit left a comment

Choose a reason for hiding this comment

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

Thanks!

@TurboGit TurboGit merged commit 8549ebe into darktable-org:master Jan 2, 2023
@TurboGit
Copy link
Member

TurboGit commented Jan 2, 2023

Merged in master and 4.2 branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix pull request fixing a bug priority: medium core features are degraded in a way that is still mostly usable, software stutters scope: hardware support dealing with drivers and external devices: GPU, printers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

crash on switching back to tethering mode possibly after camera disconnected

2 participants