fix: _remove_preset_flag crash on NULL image

[sentry]

The function _remove_preset_flag in src/common/history.c was susceptible to a NULL pointer dereference. The dt_image_t *image variable, obtained from dt_image_cache_get, was dereferenced without a NULL check. If dt_image_cache_get returned NULL (e.g., if the image was not found in the cache), this would lead to a segfault.

This commit adds a NULL check for the image pointer immediately after the call to dt_image_cache_get. If image is NULL, the function now returns early, preventing the crash. This aligns with existing patterns in the codebase for handling potential NULL returns from dt_image_cache_get.

Fixes ANSEL-1K

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud