2018-08-22 18:00 GMT+02:00, Paul B Mahol <onemda at gmail.com>: > + switch (avctx->bits_per_coded_sample) { > + case 12: > + s->stride = 3LL * FFALIGN(avctx->width, 8) >> 1; > + break; > + default: > + return AVERROR_INVALIDDATA; > + } Why are the condition and the error needed? Carl Eugen