Fix two small memory leaks in the camera import dialog

[jeffc]

These were identified and fixed by AI tools as part of #21319 . Created a separate PR at the request of the reviewers there.

The leak in camera_control.c was caused when a caller-allocated list was sorted in dt_camctl_import(). When the caller later went to free the list, it no longer had access to all of the list elements. Fixed by allocating and later freeing a scope-local shallow copy of the list.

The leak in camera_jobs.c was caused by a shallow call to free, which orphaned the underlying dynamically-allocated string data. Fixed by changing g_list_free() to g_list_free_full().

[jeffc]

Reverting to draft until I figure out build failures. My test build must have been using a cached result

[victoryforce]

@jeffc Your commit contains a part of your feature adding PR code working with a new member of the dt_camera_files_t structure that does not yet exist within this PR.

[jeffc]

That's what I get for trying to quickly split the PR on my lunch break. I fixed the build (removed references to size) and updated the PR description.

[jenshannoschwalm]

Ok & nice, squashing all three commits and force_push would be even better :-)

[TurboGit]

Also, please never merge master branch into a topic branch. The workflow is either to rebase or do nothing if there is no conflicts.

[jeffc]

Ok & nice, squashing all three commits and force_push would be even better :-)

Done!

Also, please never merge master branch into a topic branch. The workflow is either to rebase or do nothing if there is no conflicts.

Fixed

[TurboGit]

Thanks!