[vlc-commits] [Git][videolan/vlc][master] subtitle: do not use negative jacosub time resolutions

[Steve Lhomme (@robUx4)]

Steve Lhomme pushed to branch master at VideoLAN / VLC


Commits:
5e37a14f by Steve Lhomme at 2026-01-09T12:11:35+00:00
subtitle: do not use negative jacosub time resolutions

The format [^1] doesn't say it's not allowed but it doesn't make much sense.

Fixes #29321

[^1]: http://unicorn.us.com/jacosub/jscripts.html

- - - - -


1 changed file:

- modules/demux/subtitle.c


Changes:

=====================================
modules/demux/subtitle.c
=====================================
@@ -1922,7 +1922,7 @@ static int ParseJSS( vlc_object_t *p_obj, subs_properties_t *p_props,
                     break;
 
                 sscanf( &psz_text[shift], "%d", &p_props->jss.i_time_resolution );
-                if( !p_props->jss.i_time_resolution )
+                if( !p_props->jss.i_time_resolution || p_props->jss.i_time_resolution < 0 )
                     p_props->jss.i_time_resolution = 30;
                 break;
             }



View it on GitLab: https://code.videolan.org/videolan/vlc/-/commit/5e37a14fc8ff8f79abcc843984acf74e1d841948

-- 
View it on GitLab: https://code.videolan.org/videolan/vlc/-/commit/5e37a14fc8ff8f79abcc843984acf74e1d841948
You're receiving this email because of your account on code.videolan.org.


VideoLAN code repository instance