On 11/23/2017 9:16 PM, Paul B Mahol wrote:

> +typedef struct LV2Context {
> +    const AVClass *class;
> +    char *plugin_uri;
> +    char *options;
> +
> +    unsigned long nb_inputs;
> +    unsigned long nb_inputcontrols;
> +    unsigned long nb_outputs;

Why are you using longs instead of stdint?

> +    table->uris = av_realloc(table->uris, ++table->n_uris * sizeof(char*));
> +    table->uris[table->n_uris - 1] = av_malloc(len + 1);

Every single allocation in this whole file is completely unchecked. That's not 
OK.

- Derek
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to