LibGfx/JPEG: Don't reject SOF2 image with successive approximations

It means full SOF2 JPEG support, yay!
Author: https://github.com/LucasChollet Commit: https://github.com/SerenityOS/serenity/commit/f42d850211 Pull-request: https://github.com/SerenityOS/serenity/pull/17907 Issue: https://github.com/SerenityOS/serenity/issues/4039 Reviewed-by: https://github.com/gmta Reviewed-by: https://github.com/nico ✅ Reviewed-by: https://github.com/trflynn89 ✅
2023-02-28 00:40:50 -05:00 · 2023-02-28 00:40:50 -05:00 · f42d850211 · 2024-07-17 17:06:59 +09:00
commit f42d850211
parent fbad9a70fc
1 changed files with 1 additions and 2 deletions
--- a/Userland/Libraries/LibGfx/ImageFormats/JPEGLoader.cpp
+++ b/Userland/Libraries/LibGfx/ImageFormats/JPEGLoader.cpp
@ -752,8 +752,7 @@ static ErrorOr<void> read_start_of_scan(Stream& stream, JPEGLoadingContext& cont
        current_scan.successive_approximation_high,
        current_scan.successive_approximation_low);

-    // FIXME: Support SOF2 jpegs with current_scan.successive_approximation != 0
-    if (current_scan.spectral_selection_start > 63 || current_scan.spectral_selection_end > 63 || successive_approximation != 0) {
+    if (current_scan.spectral_selection_start > 63 || current_scan.spectral_selection_end > 63 || current_scan.successive_approximation_high > 13 || current_scan.successive_approximation_low > 13) {
        dbgln_if(JPEG_DEBUG, "ERROR! Start of Selection: {}, End of Selection: {}, Successive Approximation High: {}, Successive Approximation Low: {}!",
            current_scan.spectral_selection_start,
            current_scan.spectral_selection_end,