check return code

This commit is contained in:
veejay
2023-09-04 21:40:04 +02:00
parent 0d04bc77f0
commit 72a3341510
2 changed files with 18 additions and 10 deletions

View File

@@ -1489,12 +1489,14 @@ int sample_get_all_effect_arg(int s1, int position, int *args, int arg_len, int
sample = sample_get(s1); sample = sample_get(s1);
if( arg_len == 0) if( arg_len == 0)
return 1; return 1;
if (!sample) if (!sample)
return -1; return 0;
if (position >= SAMPLE_MAX_EFFECTS) if (position >= SAMPLE_MAX_EFFECTS)
return -1; return 0;
if (arg_len < 0 || arg_len > SAMPLE_MAX_PARAMETERS) if (arg_len < 0 || arg_len > SAMPLE_MAX_PARAMETERS)
return -1; return 0;
if( sample->effect_chain[position]->kf ) if( sample->effect_chain[position]->kf )
{ {

View File

@@ -1221,6 +1221,8 @@ int vj_tag_new(int type, char *filename, int stream_nr, editlist * el, int pix_f
if( _vj_tag_new_clone(tag,channel) == 0 ) { if( _vj_tag_new_clone(tag,channel) == 0 ) {
free(tag->source_name); free(tag->source_name);
if(tag->method_filename)
free(tag->method_filename);
free(tag); free(tag);
return -1; return -1;
} }
@@ -1228,6 +1230,8 @@ int vj_tag_new(int type, char *filename, int stream_nr, editlist * el, int pix_f
default: default:
veejay_msg(0, "Stream type %d invalid", type ); veejay_msg(0, "Stream type %d invalid", type );
free(tag->source_name); free(tag->source_name);
if(tag->method_filename)
free(tag->method_filename);
free(tag); free(tag);
return -1; return -1;
} }
@@ -1261,6 +1265,8 @@ int vj_tag_new(int type, char *filename, int stream_nr, editlist * el, int pix_f
{ {
veejay_msg(0, "Unable to store stream %d - Internal Error", tag->id); veejay_msg(0, "Unable to store stream %d - Internal Error", tag->id);
free(tag->source_name); free(tag->source_name);
if(tag->method_filename)
free(tag->method_filename);
free(tag); free(tag);
return -1; return -1;
} }
@@ -2336,15 +2342,15 @@ int vj_tag_get_all_effect_args(int t1, int position, int *args,
int i = 0; int i = 0;
vj_tag *tag = vj_tag_get(t1); vj_tag *tag = vj_tag_get(t1);
if (!tag) if (!tag)
return -1; return 0;
if (arg_len == 0 ) if (arg_len == 0 )
return 1; return 1;
if (!args) if (!args)
return -1; return 0;
if (position < 0 || position >= SAMPLE_MAX_EFFECTS) if (position < 0 || position >= SAMPLE_MAX_EFFECTS)
return -1; return 0;
if (arg_len < 0 || arg_len > SAMPLE_MAX_PARAMETERS) if (arg_len < 0 || arg_len > SAMPLE_MAX_PARAMETERS)
return -1; return 0;
if( tag->effect_chain[position]->kf_status ) if( tag->effect_chain[position]->kf_status )
{ {