Commit e96860ac authored by Simon Latapie's avatar Simon Latapie Committed by Thomas Guillem

subtitle: fix a potential memory leak

Fixes the Coverity defect 403210.

Breaking the switch case will end up freeing psz_orig and continue the loop.
Signed-off-by: Thomas Guillem's avatarThomas Guillem <thomas@gllm.fr>
parent 4dedfb2a
......@@ -1884,7 +1884,7 @@ static int ParseJSS( vlc_object_t *p_obj, subs_properties_t *p_props,
case 'S':
shift = isalpha( (unsigned char)psz_text[2] ) ? 6 : 2 ;
if ( shift > line_length )
continue;
break;
if( sscanf( &psz_text[shift], "%d", &h ) )
{
......@@ -1923,7 +1923,7 @@ static int ParseJSS( vlc_object_t *p_obj, subs_properties_t *p_props,
case 'T':
shift = isalpha( (unsigned char)psz_text[2] ) ? 8 : 2 ;
if ( shift > line_length )
continue;
break;
sscanf( &psz_text[shift], "%d", &p_props->jss.i_time_resolution );
break;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment