On 11/22/2017 10:24 AM, Marc-Antoine ARNAUD wrote: > New patch version which fixe the last remark. [...] > + char depth_string[8]; > + snprintf(depth_string, sizeof(depth_string), "%d", (1<<geq->depth) - 1); > + geq->expr_str[A] = av_strdup(depth_string); Missing return value check for av_strdup. - Derek