On Sat, Jan 21, 2012 at 03:18:53PM +0000, Paul B Mahol wrote:
> On 1/21/12, Kostya Shishkov <[email protected]> wrote:
> > On Fri, Jan 20, 2012 at 11:36:26PM +0000, Paul B Mahol wrote:
> >> Fixed patch (with additional fixes) attached.
> 
> [...]
> 
> >> +    if (buf_size < header_size + ncolors * XWD_CMAP_SIZE + avctx->height
> >> * lsize) {
> >> +        av_log(avctx, AV_LOG_ERROR, "input buffer too small\n");
> >> +        return AVERROR_INVALIDDATA;
> >> +    }
> >> +
> >> +    if (pixformat != XWD_Z_PIXMAP) {
> >> +        av_log(avctx, AV_LOG_ERROR, "pixmap format %d unsupported\n",
> >> pixformat);
> >> +        return AVERROR_PATCHWELCOME;
> >
> > nit: av_log_ask_for_sample ?
> 
> You can create one with "xwd --xy ..."
> 
> IMHO that is not worth implementing.

thank you for clarification
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to